commit | 02b33c1f9c7145169cfda0caf4b5eb0c50e471b5 | [log] [tgz] |
---|---|---|
author | Leonard Grey <lgrey@chromium.org> | Fri Jul 20 15:36:31 2018 |
committer | Commit Bot <commit-bot@chromium.org> | Fri Jul 20 15:36:31 2018 |
tree | 007718aa96733ac51787b2d2429ab74563877d3b | |
parent | 38c375648d86dd68e2b50713cbbe991481f68521 [diff] |
Views: Don't unnecessarily invalidate layout when setting app menu button margin Currently, we invalidate the app menu button's layout unconditionally when setting its margin. Since invalidating layout walks up the view hierarchy, this essentially keeps ToolbarView's layout dirty all the time. This change exits early if the new margin is the same as the current margin. Bug: 858944, 835983 Change-Id: If94d3c7d29cb1476fdb268a8ece35187d0b43f4f Reviewed-on: https://chromium-review.googlesource.com/1144222 Reviewed-by: Elly Fong-Jones <ellyjones@chromium.org> Commit-Queue: Leonard Grey <lgrey@chromium.org> Cr-Commit-Position: refs/heads/master@{#576873}
Chromium is an open-source browser project that aims to build a safer, faster, and more stable way for all users to experience the web.
The project's web site is https://www.chromium.org.
Documentation in the source is rooted in docs/README.md.
Learn how to Get Around the Chromium Source Code Directory Structure .