mash: Migrate ShelfLayoutManager ScreenUtil usage to ash common types.

ash/common/wm/wm_screen_util.h

 // Copyright 2016 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.
 
 #ifndef ASH_COMMON_WM_WM_SCREEN_UTIL_H_
 #define ASH_COMMON_WM_WM_SCREEN_UTIL_H_
 
 #include "ash/ash_export.h"
 
 namespace gfx {
 class Rect;
 }
 
 namespace ash {
 
 class WmWindow;
 
 namespace wm {
 
 ASH_EXPORT gfx::Rect GetDisplayWorkAreaBoundsInParent(WmWindow* window);
 ASH_EXPORT gfx::Rect GetDisplayWorkAreaBounds(WmWindow* window);
 ASH_EXPORT gfx::Rect GetDisplayBoundsInParent(WmWindow* window);
 ASH_EXPORT gfx::Rect GetMaximizedWindowBoundsInParent(WmWindow* window);
 
+// Returns the physical display bounds containing the shelf that
+// shares the same display as |window|. Physical displays can
+// differ from logical displays in unified desktop mode.
+// TODO(oshima): If we need to expand the unified desktop support to
+// general use, we should consider always using physical display in
+// window layout instead of root window, and keep the logical
+// display only in display management code.
+ASH_EXPORT gfx::Rect GetShelfDisplayBoundsForWindow(WmWindow* window);

[James Cook, 2016/08/05 23:51:25]

nit: I think this should be renamed to something like
GetDisplayBoundsWithShelf(WmWindow* context) or
GetDisplayBoundsWithShelfForWindow(). I keep thinking it has to do with the
shelf's bounds, and it doesn't.

[msw, 2016/08/06 00:08:13]

Done. I revised the comment too.

+
 }  // namespace wm
 }  // namespace ash
 
 #endif  // ASH_COMMON_WM_WM_SCREEN_UTIL_H_