Skip to content

Roll engine to 0c09bf03d. - #16668

Merged
tvolkert merged 2 commits into
flutter:masterfrom
chinmaygarde:rolltoday
Apr 17, 2018
Merged

tvolkert merged 2 commits into
flutter:masterfrom
chinmaygarde:rolltoday

Conversation

@chinmaygarde

@chinmaygarde chinmaygarde commented Apr 17, 2018 •

Copy link
Copy Markdown
Contributor

This roll includes the following engine commits:

@chinmaygarde
chinmaygarde requested a review from tvolkert April 17, 2018 19:27
@chinmaygarde chinmaygarde changed the title Roll engine to 0c09bf03d1dcde1a6c8cfaa2102949abac7757cd. Roll engine to 0c09bf03d. Apr 17, 2018
@chinmaygarde

Copy link
Copy Markdown
Contributor Author

The commit message is wrong. It need to include more engine commits. Updating.

This roll includes the following engine commits:

* flutter-team-archive/engine@0c09bf03: In the service protocol, if no view is specified to flush tasks on, pick the first one. (flutter#5028)
* flutter-team-archive/engine@ecaefd7a: Roll src/third_party/skia/ 276886160..1cd6098d5 (50 commits) (flutter#5026)
* flutter-team-archive/engine@e6545573: Enable 'Speak Screen' and 'Switch Control' for iOS (flutter#5016)
* flutter-team-archive/engine@3393c179: Fix Android platform channels (flutter#5025)
* flutter-team-archive/engine@1bc0e1bb: On windows, refer to Dart snapshot directly executable. (flutter#5024)
* flutter-team-archive/engine@cf6ca32d: Revert "Roll Dart to fe606f890b0a311da802c78b0af414a3c2087a79" (flutter#5023)
* flutter-team-archive/engine@4b45a53e: Only make ERROR and FATAL log levels visible by default. (flutter#5022)
* flutter-team-archive/engine@98f6c2d5: Fix shell launcher test (dart2 compliant and stop polling) (flutter#5009)
* flutter-team-archive/engine@75851f08: Roll Dart to fe606f890b0a311da802c78b0af414a3c2087a79 (flutter#5020)
* flutter-team-archive/engine@9495a52d: On iOS, try to use ES3, then fall back to ES2. (flutter#5006)
* flutter-team-archive/engine@d6d4eec2: Fix broken Linux build (flutter#5019)
* flutter-team-archive/engine@d15dc76e: Fix broken Chromebot build (flutter#5018)
* flutter-team-archive/engine@c17a6290: Breadcrumbs to the wiki (flutter#5017)
* flutter-team-archive/engine@4eaf2c2f: Return raw (unencoded) bytes in Image.toByteData() (flutter#5008)
* flutter-team-archive/engine@d812a617: Roll src/third_party/skia/ e669bdfad..276886160 (9 commits) (flutter#5014)
* flutter-team-archive/engine@62aeab72: Call SkSurface::flush instead of getting a backend handle in vulkan_swapchain. (flutter#5012)
* flutter-team-archive/engine@f4ebb180: Roll src/third_party/skia/ 3462eb034..e669bdfad (54 commits) (flutter#5007)
* flutter-team-archive/engine@0b7d6be9: Re-land "do not pause rendering when android view loses focus" (flutter#5004)
* flutter-team-archive/engine@91dee598: If the rasterizer has a valid surface context, use that to create a snapshotting render target. (flutter#4979)
* flutter-team-archive/engine@570231b7: Specify the packages file path when running engine dart tests. (flutter#5005)
* flutter-team-archive/engine@0b9cef5a: Minor update to README (flutter#5003)
* flutter-team-archive/engine@c386418f: Promote line height in text style created from paragraph style (flutter#4995)
* flutter-team-archive/engine@4e0fbb6f: If the test specifies a .dill file, dont make the engine interpret is as source. (flutter#5002)
* flutter-team-archive/engine@58e84c8b: Re-land "Support multiple shells in a single process. (flutter#4932)" (flutter#4998)
* flutter-team-archive/engine@37e5df05: Remove only use of SkPaint::kGenA8FromLCD (flutter#4994)
* flutter-team-archive/engine@b7358b33: libtxt: cache font families that are remapped to the default font family (flutter#4990)
* flutter-team-archive/engine@a530035b: libtxt: fix leaks in Skia object reference counting (flutter#4988)
* flutter-team-archive/engine@0c74fc93: [fuchsia] Fix use-after-free (flutter#4987)
* flutter-team-archive/engine@b6d2dde9: [fuchsia] Fix build (flutter#4986)
* flutter-team-archive/engine@1dd12649: [async] Adapt to libasync API changes. (flutter#4980)
@chinmaygarde

Copy link
Copy Markdown
Contributor Author

Done commit messages amended.

@tvolkert

Copy link
Copy Markdown
Contributor

You're going to need to remove the references to EncodingFormat, as was done in dd32d1b

@tvolkert tvolkert 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.

LGTM once comment is addressed.

@chinmaygarde

Copy link
Copy Markdown
Contributor Author

Done.

@tvolkert
tvolkert merged commit e9177ba into flutter:master Apr 17, 2018
@tvolkert

Copy link
Copy Markdown
Contributor

This increased the hot reload time in --no-preview-dart-2 by 10x (Dart 2 mode was unaffected). It's also worth note that the previous engine roll (that was reverted) did not regress those benchmarks, so the culprit must be in one of the commits herein that wasn't in the original engine roll.

@a-siva @Hixie how much do we still care about --no-preview-dart-2? Is it going away very soon?

@a-siva

a-siva commented Apr 18, 2018

Copy link
Copy Markdown
Contributor

measuring against --no-preview-dart2 was a good comparison point for us to figure out how much of a regression Dart2 introduced.
Not sure why a 10x regression would happen, I don't see any Dart SDK rolls in the list you have except for the roll/revert.

@tvolkert

Copy link
Copy Markdown
Contributor

These are the only commits that weren't in the first roll:

flutter-team-archive/engine@0c09bf0: In the service protocol, if no view is specified to flush tasks on, pick the first one.
flutter-team-archive/engine@ecaefd7: Roll src/third_party/skia/ 276886160..1cd6098d5 (50 commits)
flutter-team-archive/engine@e654557: Enable 'Speak Screen' and 'Switch Control' for iOS
flutter-team-archive/engine@3393c17: Fix Android platform channels

@Hixie

Hixie commented Apr 18, 2018

Copy link
Copy Markdown
Contributor

Weird, no idea why any of those would affect Dart1.

We're about to pull the plug on Dart1 though so I wouldn't worry about it.

DaveShuckerow pushed a commit to DaveShuckerow/flutter that referenced this pull request May 14, 2018
This roll includes the following engine commits:

* flutter-team-archive/engine@0c09bf03: In the service protocol, if no view is specified to flush tasks on, pick the first one. (flutter#5028)
* flutter-team-archive/engine@ecaefd7a: Roll src/third_party/skia/ 276886160..1cd6098d5 (50 commits) (flutter#5026)
* flutter-team-archive/engine@e6545573: Enable 'Speak Screen' and 'Switch Control' for iOS (flutter#5016)
* flutter-team-archive/engine@3393c179: Fix Android platform channels (flutter#5025)
* flutter-team-archive/engine@1bc0e1bb: On windows, refer to Dart snapshot directly executable. (flutter#5024)
* flutter-team-archive/engine@cf6ca32d: Revert "Roll Dart to fe606f890b0a311da802c78b0af414a3c2087a79" (flutter#5023)
* flutter-team-archive/engine@4b45a53e: Only make ERROR and FATAL log levels visible by default. (flutter#5022)
* flutter-team-archive/engine@98f6c2d5: Fix shell launcher test (dart2 compliant and stop polling) (flutter#5009)
* flutter-team-archive/engine@75851f08: Roll Dart to fe606f890b0a311da802c78b0af414a3c2087a79 (flutter#5020)
* flutter-team-archive/engine@9495a52d: On iOS, try to use ES3, then fall back to ES2. (flutter#5006)
* flutter-team-archive/engine@d6d4eec2: Fix broken Linux build (flutter#5019)
* flutter-team-archive/engine@d15dc76e: Fix broken Chromebot build (flutter#5018)
* flutter-team-archive/engine@c17a6290: Breadcrumbs to the wiki (flutter#5017)
* flutter-team-archive/engine@4eaf2c2f: Return raw (unencoded) bytes in Image.toByteData() (flutter#5008)
* flutter-team-archive/engine@d812a617: Roll src/third_party/skia/ e669bdfad..276886160 (9 commits) (flutter#5014)
* flutter-team-archive/engine@62aeab72: Call SkSurface::flush instead of getting a backend handle in vulkan_swapchain. (flutter#5012)
* flutter-team-archive/engine@f4ebb180: Roll src/third_party/skia/ 3462eb034..e669bdfad (54 commits) (flutter#5007)
* flutter-team-archive/engine@0b7d6be9: Re-land "do not pause rendering when android view loses focus" (flutter#5004)
* flutter-team-archive/engine@91dee598: If the rasterizer has a valid surface context, use that to create a snapshotting render target. (flutter#4979)
* flutter-team-archive/engine@570231b7: Specify the packages file path when running engine dart tests. (flutter#5005)
* flutter-team-archive/engine@0b9cef5a: Minor update to README (flutter#5003)
* flutter-team-archive/engine@c386418f: Promote line height in text style created from paragraph style (flutter#4995)
* flutter-team-archive/engine@4e0fbb6f: If the test specifies a .dill file, dont make the engine interpret is as source. (flutter#5002)
* flutter-team-archive/engine@58e84c8b: Re-land "Support multiple shells in a single process. (flutter#4932)" (flutter#4998)
* flutter-team-archive/engine@37e5df05: Remove only use of SkPaint::kGenA8FromLCD (flutter#4994)
* flutter-team-archive/engine@b7358b33: libtxt: cache font families that are remapped to the default font family (flutter#4990)
* flutter-team-archive/engine@a530035b: libtxt: fix leaks in Skia object reference counting (flutter#4988)
* flutter-team-archive/engine@0c74fc93: [fuchsia] Fix use-after-free (flutter#4987)
* flutter-team-archive/engine@b6d2dde9: [fuchsia] Fix build (flutter#4986)
* flutter-team-archive/engine@1dd12649: [async] Adapt to libasync API changes. (flutter#4980)
@github-actions github-actions Bot locked as resolved and limited conversation to collaborators Aug 11, 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