GitLab

FindBarView was already using a LayoutManager, but also overrode Layout() to do
two things: position an invisible FocusForwarderView atop some children, and set
highlight paths on the buttons.  Each of these tasks is better achieved via
another route: buttons can set their own highlight paths, and setting textfield
focus can be done by FindBarView itself as long as we exclude the relevant
children from processing events.

Bug: 1005568
Change-Id: I32c102abeea7fbedf7dbfc57a22ce01e2c976904
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1817290
Commit-Queue: Peter Kasting <pkasting@chromium.org>
Commit-Queue: Peter Boström <pbos@chromium.org>
Auto-Submit: Peter Kasting <pkasting@chromium.org>
Reviewed-by: Peter Boström <pbos@chromium.org>
Cr-Commit-Position: refs/heads/master@{#698714}