| commit | 5764e3e320c872bfbb1b5dbaf212de1458e5d42f | |
| author | jonross <jonross@chromium.org> | |
| Wed, 5 Aug 2015 18:00:37 +0000 (5 11:00 -0700) | ||
| committer | Commit bot <commit-bot@chromium.org> | |
| Wed, 5 Aug 2015 18:01:02 +0000 (5 18:01 +0000) | ||
| tree | 9417dcb0320572101c97b232d79fd6b0fbb27403 | treesnapshot (tar.gz zip) |
| parent | 33a1119ad30c3cf698a930219cd24b508c025e7b | commitdiff |
Fix Spacing in OmniboxResultView
OmniboxResultView's Widget was returning the default ThemeProvider. With the
layout parameters now themeable, this was leading to incorrect values being used
for layout.
OmniboxResultView has been updated to refer to LocationBarView's ThemeProvider
in order to get the appropriate values.
A TODO was added to review the dependency on LocaitonBarView. This will be
addressed when the omnibox suggestion popup is updated for Material Design.
TEST=Manual testing on device, ran interactive_ui_tests
BUG=515378
Review URL: https://codereview.chromium.org/1262413005
Cr-Commit-Position: refs/heads/master@{#341921}
OmniboxResultView's Widget was returning the default ThemeProvider. With the
layout parameters now themeable, this was leading to incorrect values being used
for layout.
OmniboxResultView has been updated to refer to LocationBarView's ThemeProvider
in order to get the appropriate values.
A TODO was added to review the dependency on LocaitonBarView. This will be
addressed when the omnibox suggestion popup is updated for Material Design.
TEST=Manual testing on device, ran interactive_ui_tests
BUG=515378
Review URL: https://codereview.chromium.org/1262413005
Cr-Commit-Position: refs/heads/master@{#341921}
| chrome/browser/ui/views/omnibox/omnibox_result_view.cc | diffblobblamehistory |