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

Unified Diff: ash/wm/panel_layout_manager.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
« no previous file with comments | « ash/wm/frame_painter.cc ('k') | ash/wm/panel_layout_manager_unittest.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: ash/wm/panel_layout_manager.cc
diff --git a/ash/wm/panel_layout_manager.cc b/ash/wm/panel_layout_manager.cc
index 84dfdc6ad05545e75aaf3a95413d1e9ae879fb03..52599f361c2e992affe9b14fe8663acf9863babd 100644
--- a/ash/wm/panel_layout_manager.cc
+++ b/ash/wm/panel_layout_manager.cc
@@ -250,7 +250,7 @@ void PanelLayoutManager::Relayout() {
return;
AutoReset<bool> auto_reset_in_layout(&in_layout_, true);
- int launcher_top = launcher_->widget()->GetWindowScreenBounds().y();
+ int launcher_top = launcher_->widget()->GetWindowBoundsInScreen().y();
aura::Window* active_panel = NULL;
for (PanelList::iterator iter = panel_windows_.begin();
iter != panel_windows_.end(); ++iter) {
@@ -353,8 +353,8 @@ void PanelLayoutManager::UpdateCallout(aura::Window* active_panel) {
void PanelLayoutManager::ShowCalloutHelper(aura::Window* active_panel) {
DCHECK(active_panel);
- gfx::Rect bounds = active_panel->GetRootWindowBounds();
- gfx::Rect callout_bounds = callout_widget_->GetWindowScreenBounds();
+ gfx::Rect bounds = active_panel->GetBoundsInRootWindow();
+ gfx::Rect callout_bounds = callout_widget_->GetWindowBoundsInScreen();
callout_bounds.set_x(
bounds.x() + (bounds.width() - callout_bounds.width()) / 2);
callout_bounds.set_y(bounds.bottom());
« no previous file with comments | « ash/wm/frame_painter.cc ('k') | ash/wm/panel_layout_manager_unittest.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698