Commit 3010d2e5 authored by Patti's avatar Patti Committed by Commit Bot

Omnibox/Views: Fix cutout position while the omnibox popup is open in touch.

The rounded omnibox drop-down cuts out a hole for the omnibox to show through
when it's open to avoid having to teleport the LocationBarView between Widgets.
r550927 adjusted the positioning and size of the drop-down, but regressed by
positioning the cutout incorrectly. This patch re-aligns it.

Bug: 833281
Change-Id: If95dad702b54fa44e6e8e6df4b959401510993df
Reviewed-on: https://chromium-review.googlesource.com/1013020Reviewed-by: default avatarJustin Donnelly <jdonnelly@chromium.org>
Commit-Queue: Patti <patricialor@chromium.org>
Cr-Commit-Position: refs/heads/master@{#551198}
parent 87c43b29
...@@ -107,8 +107,7 @@ void RoundedOmniboxResultsFrame::Layout() { ...@@ -107,8 +107,7 @@ void RoundedOmniboxResultsFrame::Layout() {
gfx::Rect top_bounds(bounds); gfx::Rect top_bounds(bounds);
top_bounds.set_height(GetNonResultSectionHeight()); top_bounds.set_height(GetNonResultSectionHeight());
// Height is already accounted for in GetNonResultSectionHeight(); top_bounds.Inset(kLocationBarAlignmentInsets);
top_bounds.set_width(bounds.width() - kLocationBarAlignmentInsets.width());
top_background_->SetBoundsRect(top_bounds); top_background_->SetBoundsRect(top_bounds);
gfx::Rect results_bounds(bounds); gfx::Rect results_bounds(bounds);
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment