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

Conversation

@arbreng
Copy link
Contributor

@arbreng arbreng commented Aug 21, 2020

Description

Unfortunately #20532 caused a few elevation issues on fuchsia. This PR fixes them.

Related Issues

https://bugs.fuchsia.dev/p/fuchsia/issues/detail?id=53062
https://bugs.fuchsia.dev/p/fuchsia/issues/detail?id=53063

Tests

Tested: Ran all unittests, ran internal shell and checked web/cast views specifically.

@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.

@arbreng
Copy link
Contributor Author

arbreng commented Aug 21, 2020

Landing on red b/c this fixes a P0 fuchsia issue

@arbreng arbreng merged commit bb57fb4 into flutter:master Aug 21, 2020
@arbreng arbreng deleted the fix-elevation branch August 21, 2020 18:57
@zanderso
Copy link
Member

Landing this on red doesn't seem like a good idea since some Fuchsia tests are part of the current redness.

@arbreng
Copy link
Contributor Author

arbreng commented Aug 21, 2020

Landing this on red doesn't seem like a good idea since some Fuchsia tests are part of the current redness.

Yeah, I landed this out of order with #20697 because I'm in a bit too much of a hurry...

The tree should be stable one that one goes through CI though

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants