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

Conversation

@GaryQian
Copy link
Contributor

@GaryQian GaryQian commented May 1, 2020

Reverts #17612

We are seeing resurgence of the duplication bug: flutter/flutter#55730

We should re-apply this for now and try to remove it at a later date.

Copy link
Member

@cbracken cbracken left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM stamp from a Japanese personal seal

@cbracken
Copy link
Member

cbracken commented May 1, 2020

@GaryQian does the upcoming 1.17 release have this bug?

@GaryQian
Copy link
Contributor Author

GaryQian commented May 1, 2020

It likely does, but I don't have a repro, so will see if customer sees it fixed.

@GaryQian
Copy link
Contributor Author

GaryQian commented May 1, 2020

If it does repro, it would only be on a small amount of Samsung devices, as I have verified it does not occur on most of the common ones.

@GaryQian
Copy link
Contributor Author

GaryQian commented May 1, 2020

Analyzer failure fixed by flutter/flutter#56139.

@GaryQian GaryQian merged commit 7a05fcb into master May 1, 2020
engine-flutter-autoroll added a commit to engine-flutter-autoroll/flutter that referenced this pull request May 1, 2020
engine-flutter-autoroll added a commit to engine-flutter-autoroll/flutter that referenced this pull request May 1, 2020
engine-flutter-autoroll added a commit to engine-flutter-autoroll/flutter that referenced this pull request May 2, 2020
engine-flutter-autoroll added a commit to engine-flutter-autoroll/flutter that referenced this pull request May 5, 2020
@cbracken cbracken deleted the revert-17612-samsung branch June 24, 2020 04:37
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants