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

Conversation

@robert-ancell
Copy link
Contributor

No description provided.

@auto-assign auto-assign bot requested a review from gw280 April 23, 2020 22:24
@robert-ancell robert-ancell changed the title Linux shell platform tasks Run Flutter platform tasks in GLib main loop Apr 23, 2020
@robert-ancell robert-ancell force-pushed the linux-shell-platform-tasks branch from 2fda348 to 4d27d3b Compare April 23, 2020 22:26
@robert-ancell
Copy link
Contributor Author

Builds on #17363, which is ready to land once LUCI is happy.

@robert-ancell robert-ancell force-pushed the linux-shell-platform-tasks branch from 4d27d3b to 0560da6 Compare April 23, 2020 22:32
@robert-ancell
Copy link
Contributor Author

cc @stuartmorgan

Copy link
Contributor

@stuartmorgan-g stuartmorgan-g left a comment

Choose a reason for hiding this comment

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

Setting back to "request changes". @chinmaygarde I think you cleared my review instead of yours.

@chinmaygarde chinmaygarde self-requested a review April 27, 2020 21:50
@robert-ancell robert-ancell force-pushed the linux-shell-platform-tasks branch from 0560da6 to 2c62f30 Compare April 28, 2020 02:54
@robert-ancell
Copy link
Contributor Author

Thanks for reviews @chinmaygarde @stuartmorgan, all issues should be resolved now.

@robert-ancell robert-ancell force-pushed the linux-shell-platform-tasks branch from 2c62f30 to 814bb86 Compare April 28, 2020 03:51
Copy link
Contributor

@stuartmorgan-g stuartmorgan-g left a comment

Choose a reason for hiding this comment

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

LGTM (with the caveat that I'm not familiar with the GTK portion of this API, so if there's a subtle bug there I'm definitely not going to have caught it)

@robert-ancell robert-ancell merged commit fda26fc into flutter:master Apr 28, 2020
@robert-ancell robert-ancell deleted the linux-shell-platform-tasks branch April 28, 2020 21:14
engine-flutter-autoroll added a commit to engine-flutter-autoroll/flutter that referenced this pull request Apr 28, 2020
engine-flutter-autoroll added a commit to engine-flutter-autoroll/flutter that referenced this pull request May 5, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants