Skip to content

Call mark* methods before attaching child - #25239

Merged
goderbauer merged 1 commit into
flutter:masterfrom
goderbauer:repaintBoudnaries
Dec 13, 2018
Merged

goderbauer merged 1 commit into
flutter:masterfrom
goderbauer:repaintBoudnaries

Conversation

@goderbauer

Copy link
Copy Markdown
Member

This allows children to assume that their parents are already in a valid state. In particular, markNeedsCompositingBitsUpdate makes assumptions about the parent's dirty state that may not be true if a child is attached before the parent has been marked as dirty. See #24029 (comment) for a detailed description of the root cause for that bug.

Fixes #24029.

@goderbauer
goderbauer requested a review from Hixie December 12, 2018 05:36
@zoechi zoechi added the framework flutter/packages/flutter repository. See also f: labels. label Dec 12, 2018
@Hixie

Hixie commented Dec 13, 2018

Copy link
Copy Markdown
Contributor

LGTM

@goderbauer
goderbauer merged commit 5c1f1d7 into flutter:master Dec 13, 2018
@goderbauer
goderbauer deleted the repaintBoudnaries branch December 13, 2018 00:45
goderbauer added a commit to goderbauer/flutter that referenced this pull request Dec 14, 2018
This reverts commit 5c1f1d7.

Reverting to see if that fixes flakiness of flutter_gallery__transition_perf_with_semantics in devicelab.
goderbauer added a commit that referenced this pull request Dec 14, 2018
This reverts commit 5c1f1d7.

Reverting to see if that fixes flakiness of flutter_gallery__transition_perf_with_semantics in devicelab.
goderbauer added a commit to goderbauer/flutter that referenced this pull request Dec 15, 2018
This reverts commit ea7d086.

The revert did not fix the flakiness.
goderbauer added a commit that referenced this pull request Dec 15, 2018
This reverts commit ea7d086.

The revert did not fix the flakiness.
@goderbauer goderbauer added the c: crash Stack traces logged to the console label Feb 22, 2019
@github-actions github-actions Bot locked as resolved and limited conversation to collaborators Aug 7, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

c: crash Stack traces logged to the console framework flutter/packages/flutter repository. See also f: labels.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

markNeedsCompositingBitsUpdate fails after re-attaching a RepaintBoundary

4 participants