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

Conversation

@yjbanov
Copy link
Contributor

@yjbanov yjbanov commented Feb 28, 2021

Remove _getRRect from PathRef. Instead, remember the RRect that was added to the path while accounting for possible corrections in radii.

Fixes flutter/flutter#76885
Also fixes an AssertionError thrown by _getRRect in debug mode

@yjbanov yjbanov force-pushed the path-ref-getRRect branch from 74750df to 1b83888 Compare March 1, 2021 19:10
@yjbanov yjbanov requested a review from ferhatb March 1, 2021 19:10
@yjbanov yjbanov merged commit e763e56 into flutter:master Mar 1, 2021
engine-flutter-autoroll added a commit to engine-flutter-autoroll/flutter that referenced this pull request Mar 1, 2021
yjbanov added a commit to yjbanov/engine that referenced this pull request Mar 1, 2021
This reverts commit e763e56.

The commit broke golden tests.
yjbanov added a commit that referenced this pull request Mar 1, 2021
This reverts commit e763e56.

The commit broke golden tests.
engine-flutter-autoroll added a commit to engine-flutter-autoroll/flutter that referenced this pull request Mar 2, 2021
engine-flutter-autoroll added a commit to engine-flutter-autoroll/flutter that referenced this pull request Mar 2, 2021
engine-flutter-autoroll added a commit to engine-flutter-autoroll/flutter that referenced this pull request Mar 2, 2021
hjfreyer pushed a commit to hjfreyer/engine that referenced this pull request Mar 22, 2021
hjfreyer pushed a commit to hjfreyer/engine that referenced this pull request Mar 22, 2021
chriscraws pushed a commit to chriscraws/engine that referenced this pull request Mar 23, 2021
chriscraws pushed a commit to chriscraws/engine that referenced this pull request Mar 23, 2021
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.

[web]: PathRef.getRRect() could be simpler, faster, and more precise

3 participants