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

Unified Diff: chrome/browser/ui/views/find_bar_host.cc

Issue 10795013: Rename bounds accessors to be intuitive and consistent (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: . Created 8 years, 5 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/find_bar_host.cc
diff --git a/chrome/browser/ui/views/find_bar_host.cc b/chrome/browser/ui/views/find_bar_host.cc
index 70923b524925a0c0d8db5dfdd586dad2f951b778..ec069ce6e0678a9639e7c665d9fbf3ba563d42b4 100644
--- a/chrome/browser/ui/views/find_bar_host.cc
+++ b/chrome/browser/ui/views/find_bar_host.cc
@@ -220,7 +220,7 @@ bool FindBarHost::GetFindBarWindowInfo(gfx::Point* position,
return false;
}
- gfx::Rect window_rect = host()->GetWindowScreenBounds();
+ gfx::Rect window_rect = host()->GetWindowBoundsInScreen();
if (position)
*position = window_rect.origin();
if (fully_visible)
@@ -331,7 +331,7 @@ void FindBarHost::UnregisterAccelerators() {
// private:
void FindBarHost::GetWidgetPositionNative(gfx::Rect* avoid_overlapping_rect) {
- gfx::Rect frame_rect = host()->GetTopLevelWidget()->GetWindowScreenBounds();
+ gfx::Rect frame_rect = host()->GetTopLevelWidget()->GetWindowBoundsInScreen();
content::WebContentsView* tab_view =
find_bar_controller_->tab_contents()->web_contents()->GetView();
gfx::Rect webcontents_rect = tab_view->GetViewBounds();

Powered by Google App Engine
This is Rietveld 408576698