Skip to content
This repository was archived by the owner on Feb 25, 2025. It is now read-only.

Remove single window assumption from testing/dart - #40160

Merged
goderbauer merged 2 commits into
flutter-team-archive:mainfrom
goderbauer:migrateDart
Mar 9, 2023
Merged

goderbauer merged 2 commits into
flutter-team-archive:mainfrom
goderbauer:migrateDart

Conversation

@goderbauer

Copy link
Copy Markdown
Contributor

@goderbauer
goderbauer requested a review from loic-sharma March 9, 2023 00:24
final Scene scene = builder.build();

window.render(scene);
PlatformDispatcher.instance.implicitView!.render(scene);

@loic-sharma loic-sharma Mar 9, 2023 •

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The comments on line 40 and lines 42-43 look like test code that was never removed. Consider deleting it if you feel like it :)

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Will do!

@goderbauer goderbauer added the autosubmit Merge PR when tree becomes green via auto submit App label Mar 9, 2023
@goderbauer
goderbauer merged commit 7893f10 into flutter-team-archive:main Mar 9, 2023
@goderbauer
goderbauer deleted the migrateDart branch March 9, 2023 18:10
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

autosubmit Merge PR when tree becomes green via auto submit App

Development

Successfully merging this pull request may close these issues.

2 participants