This repository was archived by the owner on Feb 25, 2025. It is now read-only.
fix ax unique id flake #23518
Merged
fix ax unique id flake #23518
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Contributor
Author
|
I also applied the same patch to the chromium upstream |
Contributor
|
Do you have a link to the upstream review? |
Contributor
Author
gw280
approved these changes
Jan 13, 2021
chunhtai
force-pushed
the
issues/73512-unskip
branch
from
ddf0682 to
3872589
Compare
chunhtai
added
the
waiting for tree to go green
engine-flutter-autoroll
added a commit
to engine-flutter-autoroll/flutter
that referenced
this pull request
Jan 14, 2021
engine-flutter-autoroll
added a commit
to engine-flutter-autoroll/flutter
that referenced
this pull request
Jan 14, 2021
engine-flutter-autoroll
added a commit
to engine-flutter-autoroll/flutter
that referenced
this pull request
Jan 14, 2021
engine-flutter-autoroll
added a commit
to engine-flutter-autoroll/flutter
that referenced
this pull request
Jan 14, 2021
engine-flutter-autoroll
added a commit
to engine-flutter-autoroll/flutter
that referenced
this pull request
Jan 14, 2021
cbracken
pushed a commit
to flutter/flutter
that referenced
this pull request
Jan 14, 2021
* d7b25f9 [web] Reland: Enable the new rich paragraph implementation (flutter/engine#23659) * 507a887 fix ax unique id flake (flutter/engine#23518) * e5a4c7c Roll Fuchsia Linux SDK from BUsKF6z4t... to a7ezEWPM5... (flutter/engine#23668) * 7d1e4e3 Roll Skia from 0e4a29af9db2 to af9b58e287b5 (10 revisions) (flutter/engine#23670) * 635f64a Roll Skia from af9b58e287b5 to f435ada424df (1 revision) (flutter/engine#23671) * 6957611 Roll Fuchsia Mac SDK from BsUY1yjWh... to BoBy7Eobf... (flutter/engine#23672) * 37131f1 Use non-deprecated SkImageFilter factory functions (flutter/engine#23649) * 38f08ff Roll Fuchsia Linux SDK from a7ezEWPM5... to erwxDS3kf... (flutter/engine#23676) * a56183c Roll Skia from f435ada424df to 8f78d5528438 (7 revisions) (flutter/engine#23680) * 9d75996 Revert "[web] Enable the new rich paragraph implementation (#23162)" (flutter/engine#23687) * b030bc8 Roll Dart SDK from 724d9e5e7d71 to 3629a798353d (1 revision) (flutter/engine#23681) * a8a7534 Support Mice in iPadOS 13.4+ (flutter/engine#23636) * 3f691f0 [web] Fix semantic node order for webkit (flutter/engine#23601) * 70d08c6 Roll Skia from 8f78d5528438 to 890b2b406a60 (9 revisions) (flutter/engine#23692) * 5459565 Roll Fuchsia Mac SDK from BoBy7Eobf... to DVYrV15dq... (flutter/engine#23693) * 3fd4c63 [web] Fix letter spacing for rich paragraphs (flutter/engine#23683) * effb529 Added missing export for the flutter engine group. (flutter/engine#23689)
gspencergoog
pushed a commit
to gspencergoog/engine
that referenced
this pull request
Jan 20, 2021
hjfreyer
pushed a commit
to hjfreyer/engine
that referenced
this pull request
Mar 22, 2021
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
4 participants
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Description
This seems to be an upstream bug. the internal counter in ax unique id can contain arbitrary int when the unique id is called with max value, if the internal counter is > than max value, the ax_unique id will not work properly.
Related Issues
fixes flutter/flutter#73512
Tests
I added the following tests:
see files
Checklist
Before you create this PR confirm that it meets all requirements listed below by checking the relevant checkboxes (
[x]). This will ensure a smooth and quick review process.Reviewer Checklist
Breaking Change
Did any tests fail when you ran them? Please read handling breaking changes.