Skip to content

Revert "Automatic focus highlight mode for FocusManager (#37825)"#38866

Merged
gspencergoog merged 1 commit intoflutter:masterfrom
gspencergoog:revert_highlight
Aug 21, 2019
Merged

Revert "Automatic focus highlight mode for FocusManager (#37825)"#38866
gspencergoog merged 1 commit intoflutter:masterfrom
gspencergoog:revert_highlight

Conversation

@gspencergoog
Copy link
Contributor

This reverts commit a11d731 because of a regression in
flutter_gallery_ios32__transition_perf and 90th_percentile_frame_build_time_millis.

See issue #38860.

This reverts commit a11d731 because of a regression in
flutter_gallery_ios32__transition_perf and 90th_percentile_frame_build_time_millis.

See issue flutter#38860.
@fluttergithubbot fluttergithubbot added f: material design flutter/packages/flutter/material repository. framework flutter/packages/flutter repository. See also f: labels. labels Aug 20, 2019
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

@gspencergoog
Copy link
Contributor Author

Note that reverting this is likely to cause a 3-5% increase in hello_world size, and a 15% regression in velocity_tracker_bench_aot_frontend (Moto G4).

@gspencergoog
Copy link
Contributor Author

I'm still trying to determine which is worse: the regression in flutter_gallery_transition_perf that resulted from the change, or the 5% regression in hello world binary size that will result in reverting this. I'm hoping to find a fix for the transition perf.

@gspencergoog gspencergoog merged commit f515bf6 into flutter:master Aug 21, 2019
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Aug 4, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

f: material design flutter/packages/flutter/material repository. framework flutter/packages/flutter repository. See also f: labels.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants