Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Reland "Add non-rendering operation culling to DisplayListBuilder" (#41463) #43831

Merged
merged 1 commit into from
Jul 19, 2023

Conversation

flar
Copy link
Contributor

@flar flar commented Jul 19, 2023

Fixes: flutter/flutter#129862

This reverts commit 886a14e.

The framework test that was blocking the previous reland has been fixed to not rely on non-visible (cullable) operations.

@chinmaygarde chinmaygarde changed the title Reland "add non-rendering operation culling to DisplayListBuilder" (#41463) Reland "Add non-rendering operation culling to DisplayListBuilder" (#41463) Jul 19, 2023
@flar
Copy link
Contributor Author

flar commented Jul 19, 2023

This version of the PR is nearly identical to the previous version except that I had to merge the changes in DisplayListBuilder::saveLayer from the fix for #43787 - all other changes rebased without any conflicts...

@flar flar added the autosubmit Merge PR when tree becomes green via auto submit App label Jul 19, 2023
@auto-submit auto-submit bot merged commit bc7a207 into flutter:main Jul 19, 2023
engine-flutter-autoroll added a commit to engine-flutter-autoroll/flutter that referenced this pull request Jul 19, 2023
fluttermirroringbot pushed a commit to flutter/flutter that referenced this pull request Jul 20, 2023
…130953)

flutter/engine@7671e2f...eff70f7

2023-07-19 skia-flutter-autoroll@skia.org Roll Dart SDK from 936824d49aa7 to 677bbf64d4d7 (1 revision) (flutter/engine#43833)
2023-07-19 flar@google.com Reland "Add non-rendering operation culling to DisplayListBuilder" (#41463) (flutter/engine#43831)
2023-07-19 skia-flutter-autoroll@skia.org Roll Skia from bf75ae2f6eec to 8413c82dea43 (1 revision) (flutter/engine#43832)

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 bdero@google.com,rmistry@google.com,zra@google.com 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://bugs.chromium.org/p/skia/issues/entry?template=Autoroller+Bug

Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+doc/main/autoroll/README.md
harryterkelsen pushed a commit to harryterkelsen/engine that referenced this pull request Jul 20, 2023
…lutter#41463) (flutter#43831)

Fixes: flutter/flutter#129862

This reverts commit 886a14e.

The framework test that was blocking the previous reland has been [fixed](flutter/flutter#130751) to not rely on non-visible (cullable) operations.
LouiseHsu pushed a commit to LouiseHsu/flutter that referenced this pull request Jul 31, 2023
…lutter#130953)

flutter/engine@7671e2f...eff70f7

2023-07-19 skia-flutter-autoroll@skia.org Roll Dart SDK from 936824d49aa7 to 677bbf64d4d7 (1 revision) (flutter/engine#43833)
2023-07-19 flar@google.com Reland "Add non-rendering operation culling to DisplayListBuilder" (flutter#41463) (flutter/engine#43831)
2023-07-19 skia-flutter-autoroll@skia.org Roll Skia from bf75ae2f6eec to 8413c82dea43 (1 revision) (flutter/engine#43832)

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 bdero@google.com,rmistry@google.com,zra@google.com 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://bugs.chromium.org/p/skia/issues/entry?template=Autoroller+Bug

Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+doc/main/autoroll/README.md
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
autosubmit Merge PR when tree becomes green via auto submit App e: impeller
Projects
Status: Done
Archived in project
Development

Successfully merging this pull request may close these issues.

Reland non-rendering operation culling in DisplayListBuilder.
2 participants