Skip to content

Reenable fade transition for Material page transition - #13048

Merged
Hixie merged 1 commit into
flutter:masterfrom
passsy:material_page_transition_with_fade
Nov 22, 2017
Merged

Hixie merged 1 commit into
flutter:masterfrom
passsy:material_page_transition_with_fade

Conversation

@passsy

@passsy passsy commented Nov 16, 2017

Copy link
Copy Markdown
Contributor

Fixes #12877 by reverting #9472

@Hixie

Hixie commented Nov 16, 2017

Copy link
Copy Markdown
Contributor

What are the performance implications of this patch?

@Hixie

Hixie commented Nov 22, 2017

Copy link
Copy Markdown
Contributor

I guess we'll find out when it lands.

@Hixie

Hixie commented Nov 22, 2017

Copy link
Copy Markdown
Contributor

Thanks! Landing.

@Hixie
Hixie merged commit e73d406 into flutter:master Nov 22, 2017
@Hixie

Hixie commented Nov 23, 2017

Copy link
Copy Markdown
Contributor

This massively regressed a number of our benchmarks. I'm going to revert this again for now.

Hixie added a commit to Hixie/flutter that referenced this pull request Nov 23, 2017
…r#13048)"

This reverts commit e73d406.

It caused major performance regressions.
@Hixie

Hixie commented Nov 23, 2017

Copy link
Copy Markdown
Contributor

See #13172 for the revert.

@xster

xster commented Nov 23, 2017

Copy link
Copy Markdown
Contributor

#13173

Hixie added a commit that referenced this pull request Nov 23, 2017
…" (#13172)

This reverts commit e73d406.

It caused major performance regressions.
@passsy

passsy commented Nov 23, 2017

Copy link
Copy Markdown
Contributor Author

Great this project has benchmarks!

Hixie pushed a commit to Hixie/flutter that referenced this pull request Dec 19, 2017
Hixie added a commit that referenced this pull request Dec 20, 2017
DaveShuckerow pushed a commit to DaveShuckerow/flutter that referenced this pull request May 14, 2018
DaveShuckerow pushed a commit to DaveShuckerow/flutter that referenced this pull request May 14, 2018
…r#13048)" (flutter#13172)

This reverts commit e73d406.

It caused major performance regressions.
DaveShuckerow pushed a commit to DaveShuckerow/flutter that referenced this pull request May 14, 2018
engine-flutter-autoroll added a commit that referenced this pull request Oct 10, 2019
[email protected]:flutter/engine.git/compare/5162413111b8...cef6751

git log 5162413..cef6751 --no-merges --oneline
2019-10-10 [email protected] Revert "Test child isolates are terminated when root is shutdown (#13048)" (flutter-team-archive/engine#13067)
2019-10-10 [email protected] Test child isolates are terminated when root is shutdown (flutter-team-archive/engine#13048)
2019-10-10 [email protected] Allow embedders to specify arbitrary data to the isolate on launch. (flutter-team-archive/engine#13047)


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] on the revert to ensure that a human
is aware of the problem.

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/+/master/autoroll/README.md
Inconnu08 pushed a commit to Inconnu08/flutter that referenced this pull request Nov 26, 2019
[email protected]:flutter/engine.git/compare/5162413111b8...cef6751

git log 5162413..cef6751 --no-merges --oneline
2019-10-10 [email protected] Revert "Test child isolates are terminated when root is shutdown (flutter#13048)" (flutter-team-archive/engine#13067)
2019-10-10 [email protected] Test child isolates are terminated when root is shutdown (flutter-team-archive/engine#13048)
2019-10-10 [email protected] Allow embedders to specify arbitrary data to the isolate on launch. (flutter-team-archive/engine#13047)


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] on the revert to ensure that a human
is aware of the problem.

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/+/master/autoroll/README.md
@github-actions github-actions Bot locked as resolved and limited conversation to collaborators Aug 12, 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.

Bring back fade transition to Material page transition

4 participants