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

Conversation

@flar
Copy link
Contributor

@flar flar commented Nov 11, 2020

The event size did not match the setup. Currently this should not cause any issues because we do the paint culling based on values in Preroll, but soon we will switch to culling during Paint() and these mismatched values will cause the test to fail.

See #22336

@flar flar requested review from chinmaygarde and gw280 November 11, 2020 22:47
@flutter-dashboard
Copy link

It looks like this pull request may not have tests. Please make sure to add tests before merging. If you need an exemption to this rule, contact Hixie on the #hackers channel in Chat.

Reviewers: Read the Tree Hygiene page and make sure this patch meets those guidelines before LGTMing.

@flar
Copy link
Contributor Author

flar commented Nov 11, 2020

This is a fix to a test, not new code, so no new tests are needed.

@flar flar added the waiting for tree to go green This PR is approved and tested, but waiting for the tree to be green to land. label Nov 12, 2020
@fluttergithubbot fluttergithubbot merged commit 69021ce into flutter:master Nov 12, 2020
engine-flutter-autoroll added a commit to engine-flutter-autoroll/flutter that referenced this pull request Nov 13, 2020
engine-flutter-autoroll added a commit to engine-flutter-autoroll/flutter that referenced this pull request Nov 13, 2020
engine-flutter-autoroll added a commit to engine-flutter-autoroll/flutter that referenced this pull request Nov 13, 2020
chaselatta pushed a commit to chaselatta/engine that referenced this pull request Nov 30, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

cla: yes waiting for tree to go green This PR is approved and tested, but waiting for the tree to be green to land.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants