-
Notifications
You must be signed in to change notification settings - Fork 6k
Fix android new engine in group #49659
Conversation
|
It looks like this pull request may not have tests. Please make sure to add tests before merging. If you need an exemption to this rule, contact "@test-exemption-reviewer" in the #hackers channel in Chat (don't just cc them here, they won't see it! Use Discord!). If you are not sure if you need tests, consider this rule of thumb: the purpose of a test is to make sure someone doesn't accidentally revert the fix. Ask yourself, is there anything in your PR that you feel it is important we not accidentally revert back to how it was before your fix? Reviewers: Read the Tree Hygiene page and make sure this patch meets those guidelines before LGTMing. |
|
We need more information about what issue this is fixing. Along with a test. |
|
The triagers think this is fine though as all other uses in this file are the same. If the CI passes, I think this is good to go. |
|
Yup, if tests pass (I believe they should) this is LGTM from me. I'll watch it. |
jonahwilliams
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
…sions) (#142278) Manual roll requested by [email protected] flutter/engine@4b145d0...9142fc4 2024-01-25 98614782+auto-submit[bot]@users.noreply.github.com Reverts "Optimizations for TLHC frame rate and jank" (flutter/engine#50062) 2024-01-25 [email protected] [Impeller] Fix advanced blend alpha issue, improve blend goldens. (flutter/engine#50035) 2024-01-25 [email protected] Fix android new engine in group (flutter/engine#49659) 2024-01-25 [email protected] Remove fuchsia from allowed_hosts post-migration (flutter/engine#50013) 2024-01-25 [email protected] Roll Skia from 801bb44f2b09 to cc5124317c18 (1 revision) (flutter/engine#50054) 2024-01-25 [email protected] Fix a typo in ChannelBuffers documentation (flutter/engine#49928) 2024-01-25 [email protected] Add include suggested by clang-tidy (flutter/engine#49898) 2024-01-25 [email protected] Refactor macOS trackpad gesture tracking (flutter/engine#49424) 2024-01-25 [email protected] winding order from tesellator.h to formats.h (flutter/engine#49865) 2024-01-25 [email protected] Roll webp to 1.3.2 (flutter/engine#49883) 2024-01-25 [email protected] Optimizations for TLHC frame rate and jank (flutter/engine#50033) 2024-01-25 [email protected] Roll Skia from 29b545e4356b to 801bb44f2b09 (1 revision) (flutter/engine#50051) 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 [email protected],[email protected],[email protected] on the revert to ensure that a human is aware of the problem. To file a bug in Flutter: https://github.com/flutter/flutter/issues/new/choose To report a problem with the AutoRoller itself, please file a bug: https://issues.skia.org/issues/new?component=1389291&template=1850622 Documentation for the AutoRoller is here: https://skia.googlesource.com/buildbot/+doc/main/autoroll/README.md
Fix android
FlutterFragmentActivity.createFlutterFragment()new engine in group.Pre-launch Checklist
///).