Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(1044)

Unified Diff: chrome/browser/ui/views/autocomplete/autocomplete_popup_contents_view.cc

Issue 6462022: It turns out I had the sense of the GetLocalBounds bool wrong everywhere, so ... (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src/
Patch Set: Created 9 years, 10 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: chrome/browser/ui/views/autocomplete/autocomplete_popup_contents_view.cc
===================================================================
--- chrome/browser/ui/views/autocomplete/autocomplete_popup_contents_view.cc (revision 74228)
+++ chrome/browser/ui/views/autocomplete/autocomplete_popup_contents_view.cc (working copy)
@@ -977,7 +977,7 @@
shader->unref();
gfx::Path path;
- MakeContentsPath(&path, GetLocalBounds());
+ MakeContentsPath(&path, GetContentsBounds());
canvas->AsCanvasSkia()->drawPath(path, paint);
// Now we paint the border, so it will be alpha-blended atop the contents.
@@ -990,7 +990,7 @@
UpdateBlurRegion();
// Size our children to the available content area.
- gfx::Rect contents_rect = GetLocalBounds();
+ gfx::Rect contents_rect = GetContentsBounds();
int child_count = GetChildViewCount();
int top = contents_rect.y();
for (int i = 0; i < child_count; ++i) {
@@ -1115,7 +1115,7 @@
// Translate the contents rect into widget coordinates, since that's what
// DwmEnableBlurBehindWindow expects a region in.
- gfx::Rect contents_rect = GetLocalBounds();
+ gfx::Rect contents_rect = GetContentsBounds();
gfx::Point origin(contents_rect.origin());
views::View::ConvertPointToWidget(this, &origin);
contents_rect.set_origin(origin);
@@ -1174,7 +1174,7 @@
}
gfx::Rect AutocompletePopupContentsView::CalculateTargetBounds(int h) {
- gfx::Rect location_bar_bounds(location_bar_->GetLocalBounds());
+ gfx::Rect location_bar_bounds(location_bar_->GetContentsBounds());
const views::Border* border = location_bar_->border();
if (border) {
// Adjust for the border so that the bubble and location bar borders are

Powered by Google App Engine
This is Rietveld 408576698