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

Started coalescing updates to viewport metrics after foregrounding the app.#16270

Merged
gaaclarke merged 7 commits into
flutter-team-archive:masterfrom
gaaclarke:coallesce-after-foreground
Jan 31, 2020
Merged

Started coalescing updates to viewport metrics after foregrounding the app.#16270
gaaclarke merged 7 commits into
flutter-team-archive:masterfrom
gaaclarke:coallesce-after-foreground

Conversation

@gaaclarke

Copy link
Copy Markdown
Contributor

Relevant issue: flutter/flutter#41679

@gaaclarke gaaclarke marked this pull request as ready for review January 30, 2020 23:53
@auto-assign auto-assign Bot requested a review from iskakaushik January 30, 2020 23:53

@chinmaygarde chinmaygarde left a comment

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.

Can you add a headerdoc brief to the FlutterCoalescer that describes why this is necessary and when it will be used? It is not entirely obvious from the implementation.

@gaaclarke

Copy link
Copy Markdown
Contributor Author

Can you add a headerdoc brief to the FlutterCoalescer that describes why this is necessary and when it will be used? It is not entirely obvious from the implementation.

done

@gaaclarke gaaclarke merged commit 71de21c into flutter-team-archive:master Jan 31, 2020
@gaaclarke

Copy link
Copy Markdown
Contributor Author

Submitting on red (it's a purple flake).

engine-flutter-autoroll added a commit to engine-flutter-autoroll/flutter that referenced this pull request Jan 31, 2020
engine-flutter-autoroll added a commit to engine-flutter-autoroll/flutter that referenced this pull request Jan 31, 2020
engine-flutter-autoroll added a commit to engine-flutter-autoroll/flutter that referenced this pull request Jan 31, 2020
engine-flutter-autoroll added a commit to engine-flutter-autoroll/flutter that referenced this pull request Jan 31, 2020
engine-flutter-autoroll added a commit to engine-flutter-autoroll/flutter that referenced this pull request Jan 31, 2020
engine-flutter-autoroll added a commit to flutter/flutter that referenced this pull request Jan 31, 2020
flutter-team-archive/engine@804dca6...6b8c150

git log 804dca6..6b8c150 --first-parent --oneline
2020-01-31 gw280@google.com Use static constexpr for iphone_xs_times (flutter-team-archive/engine#16264)
2020-01-31 30870216+gaaclarke@users.noreply.github.com Started coalescing updates to viewport metrics after foregrounding the app. (flutter-team-archive/engine#16270)
2020-01-31 chinmaygarde@google.com Remove superfluous INFO logs from //flutter/runtime. (flutter-team-archive/engine#16280)


If this roll has caused a breakage, revert this CL and stop the roller
using the controls here:
https://autoroll.skia.org/r/flutter-engine-flutter-autoroll
Please CC cbracken@google.com on the revert to ensure that a human
is aware of the problem.

To report a problem with the AutoRoller itself, please file a bug:
https://bugs.chromium.org/p/skia/issues/entry?template=Autoroller+Bug

Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+/master/autoroll/README.md
NoamDev pushed a commit to NoamDev/engine that referenced this pull request Feb 27, 2020
NoamDev added a commit to NoamDev/engine that referenced this pull request Feb 27, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

Development

Successfully merging this pull request may close these issues.

3 participants