Simplify WorkAreaBounds access logic

ash/screen_ash.cc

 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "ash/screen_ash.h"
 
 #include "ash/display/display_controller.h"
 #include "ash/display/multi_display_manager.h"
 #include "ash/shell.h"
 #include "ash/wm/coordinate_conversion.h"
@@ skipping 20 matching lines @@
 
 ScreenAsh::~ScreenAsh() {
 }
 
 // static
 gfx::Display ScreenAsh::FindDisplayContainingPoint(const gfx::Point& point) {
   return GetDisplayManager()->FindDisplayContainingPoint(point);
 }
 
 // static
 gfx::Rect ScreenAsh::GetMaximizedWindowBoundsInParent(aura::Window* window) {

[sky, 2012/10/01 14:11:48]

Can't you nuke this entirely and folks instead use work_area on Display?

   if (window->GetRootWindow() == Shell::GetPrimaryRootWindow())
-    return Shell::GetInstance()->shelf()->GetMaximizedWindowBounds(window);
+    return Shell::GetInstance()->shelf()->GetWorkAreaBounds(window);
   else
     return GetDisplayBoundsInParent(window);
 }
 
 // static
 gfx::Rect ScreenAsh::GetUnmaximizedWorkAreaBoundsInParent(
     aura::Window* window) {
   if (window->GetRootWindow() == Shell::GetPrimaryRootWindow())
-    return Shell::GetInstance()->shelf()->GetUnmaximizedWorkAreaBounds(window);
+    return Shell::GetInstance()->shelf()->GetWorkAreaBounds(window);
   else
     return GetDisplayWorkAreaBoundsInParent(window);
 }
 
 // static
 gfx::Rect ScreenAsh::GetDisplayBoundsInParent(aura::Window* window) {
   return ConvertRectFromScreen(
       window->parent(),
       gfx::Screen::GetDisplayNearestWindow(window).bounds());
 }
@@ skipping 51 matching lines @@
 
 gfx::Display ScreenAsh::GetDisplayMatching(const gfx::Rect& match_rect) const {
   return GetDisplayManager()->GetDisplayMatching(match_rect);
 }
 
 gfx::Display ScreenAsh::GetPrimaryDisplay() const {
   return DisplayController::GetPrimaryDisplay();
 }
 
 }  // namespace ash