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

Conversation

@auto-submit
Copy link
Contributor

@auto-submit auto-submit bot commented Oct 3, 2024

Reverts: #55393

Initiated by: matanlurey

Reason for reverting: Postsubmit turned red (clang tidy and build).

Original PR Author: jonahwilliams

Reviewed By: {chinmaygarde}

This change reverts the following previous change:
Hoist the content context up to the ios_surface so that overlay layers don't recreate a bunch of expensive machinery.

Fixes flutter/flutter#154836

@auto-submit auto-submit bot added the revert of Bot Only: Tracking label for bot. Tracks new revert of pull requests. label Oct 3, 2024
@auto-submit auto-submit bot merged commit 8b9dbdd into main Oct 3, 2024
@auto-submit auto-submit bot deleted the revert_7fbdef03a3ba678cca2ea7e44f3925d1655e585a branch October 3, 2024 18:46
engine-flutter-autoroll added a commit to engine-flutter-autoroll/flutter that referenced this pull request Oct 3, 2024
engine-flutter-autoroll added a commit to engine-flutter-autoroll/flutter that referenced this pull request Oct 3, 2024
engine-flutter-autoroll added a commit to engine-flutter-autoroll/flutter that referenced this pull request Oct 3, 2024
engine-flutter-autoroll added a commit to engine-flutter-autoroll/flutter that referenced this pull request Oct 3, 2024
auto-submit bot pushed a commit to flutter/flutter that referenced this pull request Oct 3, 2024
…156174)

flutter/engine@de1762d...20369c5

2024-10-03 [email protected] [Impeller] A text layout and shaping API for the standalone library. (flutter/engine#55598)
2024-10-03 [email protected] Refactor `et run` (and friends). (flutter/engine#55537)
2024-10-03 [email protected] [Impeller] Do not cull in TextFrameDispatcher::drawDisplayList if the transform has perspective (flutter/engine#55602)
2024-10-03 [email protected] iOS: Fix IOSSurfaceNoopTest (flutter/engine#55621)
2024-10-03 98614782+auto-submit[bot]@users.noreply.github.com Reverts "[Impeller] one aiks context per app. (#55393)" (flutter/engine#55625)
2024-10-03 [email protected] Roll Fuchsia Linux SDK from YKvfbpZ38daiwJ7H0... to TnSpLq0S9XT2rHGl1... (flutter/engine#55622)
2024-10-03 [email protected] [Impeller] one aiks context per app. (flutter/engine#55393)

Also rolling transitive DEPS:
  fuchsia/sdk/core/linux-amd64 from YKvfbpZ38dai to TnSpLq0S9XT2

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
nick9822 pushed a commit to nick9822/flutter that referenced this pull request Dec 18, 2024
…er/engine#55625)

Reverts: flutter/engine#55393
Initiated by: matanlurey
Reason for reverting: Postsubmit turned red (clang tidy and build).
Original PR Author: jonahwilliams

Reviewed By: {chinmaygarde}

This change reverts the following previous change:
Hoist the content context up to the ios_surface so that overlay layers don't recreate a bunch of expensive machinery. 

Fixes flutter#154836
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

platform-ios revert of Bot Only: Tracking label for bot. Tracks new revert of pull requests.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Impeller] Ensure there is one and only one content context per Flutter shell.

2 participants