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

Conversation

@harryterkelsen
Copy link
Contributor

Reverts #55464

The PR caused a large regression in a web benchmark.

@github-actions github-actions bot added the platform-web Code specifically for the web engine label Sep 27, 2024
Copy link
Contributor

@yjbanov yjbanov left a comment

Choose a reason for hiding this comment

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

lgtm

@harryterkelsen harryterkelsen added the autosubmit Merge PR when tree becomes green via auto submit App label Sep 27, 2024
@auto-submit auto-submit bot removed the autosubmit Merge PR when tree becomes green via auto submit App label Sep 27, 2024
@auto-submit
Copy link
Contributor

auto-submit bot commented Sep 27, 2024

auto label is removed for flutter/engine/55501, due to - The status or check suite Mac mac_host_engine has failed. Please fix the issues identified (or deflake) before re-applying this label.

@harryterkelsen harryterkelsen added the autosubmit Merge PR when tree becomes green via auto submit App label Sep 28, 2024
@auto-submit auto-submit bot merged commit f3b11bc into main Sep 28, 2024
@auto-submit auto-submit bot deleted the revert-55464-revert-55456-revert_746149965de9d31687096a857f58bec6fd71ec2b branch September 28, 2024 00:51
engine-flutter-autoroll added a commit to engine-flutter-autoroll/flutter that referenced this pull request Sep 28, 2024
auto-submit bot pushed a commit to flutter/flutter that referenced this pull request Sep 28, 2024
…155865)

flutter/engine@9c8e5cb...f3b11bc

2024-09-28 [email protected] Revert "Reland "[canvaskit] Further improve overlay optimization by splitting pictures"" (flutter/engine#55501)
2024-09-28 [email protected] Migrate `try/finally` calls to `addTearDown`. (flutter/engine#55499)
2024-09-27 [email protected] iOS: Migrate FlutterAppDelegate to ARC (flutter/engine#55472)

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] 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
thejitenpatel pushed a commit to thejitenpatel/flutter that referenced this pull request Oct 1, 2024
…lutter#155865)

flutter/engine@9c8e5cb...f3b11bc

2024-09-28 [email protected] Revert "Reland "[canvaskit] Further improve overlay optimization by splitting pictures"" (flutter/engine#55501)
2024-09-28 [email protected] Migrate `try/finally` calls to `addTearDown`. (flutter/engine#55499)
2024-09-27 [email protected] iOS: Migrate FlutterAppDelegate to ARC (flutter/engine#55472)

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] 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
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

autosubmit Merge PR when tree becomes green via auto submit App platform-web Code specifically for the web engine

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants