Skip to content

Conversation

@dnfield
Copy link
Contributor

@dnfield dnfield commented Mar 14, 2019

Rolls engine to flutter/engine@403337ebb - which contains a fix for what broke in the previous roll.

The bots aren't finished with building that revision quite yet, so this will be red until they do.

As mentioned in the previous PR, the golden update is needed because of flutter/engine#8157

flutter/engine@403337ebb Do not pass short-lived buffers as labels to Dart_TimelineEvent (flutter/engine#8166)
flutter/engine@e514f0bd1 Ensure that typed data is released within SendPlatformMessage scope. (flutter/engine#8155)
flutter/engine@f2b42d6c5 Roll src/third_party/skia f699bf0437db..feb720f746dc (15 commits) (flutter/engine#8165)
flutter/engine@fbca3d34e Roll src/third_party/skia 5412618d503c..f699bf0437db (2 commits) (flutter/engine#8164)
flutter/engine@23cff1a1c Roll src/third_party/skia 8b9b8f8c5846..5412618d503c (1 commits) (flutter/engine#8163)
flutter/engine@bbd58fab5 Roll src/third_party/skia 1386b2d5579d..8b9b8f8c5846 (3 commits) (flutter/engine#8162)
flutter/engine@cdf55a8d7 Roll src/third_party/skia 29d5dec9a078..1386b2d5579d (123 commits) (flutter/engine#8161)
flutter/engine@cae083657 Roll src/third_party/dart 1bd36d694d..7d560f8385 (27 commits)
flutter/engine@fcc66fc23 Suppress deprecation warning for use of Build.CPU_ABI (flutter/engine#8154)
flutter/engine@fdad56f5f Anti-Aliasing for shape layers (flutter/engine#8157)
flutter/engine@906d684a7 Reland ""Add support for trace counters with variable arguments and instrument the raster cache." (flutter/engine#8145)
flutter/engine@3c9381783 Add “full-dart-debug” that disabled optimizations in the Dart VM. (flutter/engine#8147)
flutter/engine@b1b388f1c Encode scroll motion events in the Android touch processor (flutter/engine#8149)
flutter/engine@2daebebf4 Fix text.dart height docs (flutter/engine#8079)
flutter/engine@2e427030f Revert "Disable build_ios task due to lack of credits. (flutter/engine#8150)" (flutter/engine#8153)
flutter/engine@8be2aca56 Roll src/third_party/dart 1bd36d694d..674fd0e060 (48 commits) flutter/engine#8152

Copy link
Contributor

@jonahwilliams jonahwilliams left a comment

Choose a reason for hiding this comment

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

LGTM

@dnfield
Copy link
Contributor Author

dnfield commented Mar 15, 2019

The only failure on the last commit was a flake - going to land this to get the jobs rolling

@dnfield dnfield merged commit a29906d into flutter:master Mar 15, 2019
@Hixie
Copy link
Contributor

Hixie commented Mar 18, 2019

cc @jason-simmons - this had an undesired impact on a number of benchmarks

@Hixie
Copy link
Contributor

Hixie commented Mar 19, 2019

notably several flutter_gallery_ios32__transition_perf benchmarks, including 90th_percentile_frame_rasterizer_time_millis, and flutter_gallery__transition_perf average_frame_rasterizer_time_millis.

@jason-simmons
Copy link
Member

Took a look at the change in flutter_gallery__transition_perf/average_frame_rasterizer_time_millis. It looks like it's associated with enabling anti-aliasing (flutter/engine@fdad56f)

@dnfield dnfield deleted the manual_engine_golden branch March 19, 2019 21:25
@Hixie
Copy link
Contributor

Hixie commented Apr 2, 2019

Filed #30402 for the regressions.

@github-actions github-actions bot locked as resolved and limited conversation to collaborators Aug 7, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants