MonitorManager to manage multiple monitors.

MonitorManager to manage multiple monitors.
 This is clone of issue 9689027, which I can't access due to some internal error.

BUG=115510
TEST=none

Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=126685

[oshima, 2012-03-14 16:31:30]

Ben, can you approve this?

This is basically same as 9689027, but modified to merge erg's gfx::Screen
change. I had to create another CL because while I was merging the change,
the original issue somehow become unaccessible (don't know why, but I keep
getting error)

http://codereview.chromium.org/9699013/diff/4002/ash/shell.h
File ash/shell.h (right):

http://codereview.chromium.org/9699013/diff/4002/ash/shell.h#newcode163
ash/shell.h:163: const gfx::Insets& insets);
This will be moved to with MonitorManager in followup CL.

[Ben Goodger (Google), 2012-03-14 17:01:26]

http://codereview.chromium.org/9699013/diff/4002/ash/wm/shelf_layout_manager_...
File ash/wm/shelf_layout_manager_unittest.cc (right):

http://codereview.chromium.org/9699013/diff/4002/ash/wm/shelf_layout_manager_...
ash/wm/shelf_layout_manager_unittest.cc:110:
manager->GetMonitorNearestWindow(Shell::GetRootWindow());
I suspect we'll want some convenience function that automates this, probably in
aura. Can we define in the aura namespace:

GetMonitorNearestWindow(aura::Window* window) with these lines in it, and just
call it from places like this?

[oshima, 2012-03-14 17:19:43]

http://codereview.chromium.org/9699013/diff/4002/ash/wm/shelf_layout_manager_...
File ash/wm/shelf_layout_manager_unittest.cc (right):

http://codereview.chromium.org/9699013/diff/4002/ash/wm/shelf_layout_manager_...
ash/wm/shelf_layout_manager_unittest.cc:110:
manager->GetMonitorNearestWindow(Shell::GetRootWindow());
On 2012/03/14 17:01:26, Ben Goodger (Google) wrote:
> I suspect we'll want some convenience function that automates this, probably
in
> aura. Can we define in the aura namespace:
> 
> GetMonitorNearestWindow(aura::Window* window) with these lines in it, and just
> call it from places like this?

I'm thinking of refacotring gfx::Screen to return monitor, replaces
GetMonitorXXXNearestYYY with GetMonitorNearestYYY()->GetXXX().
That is, this will be

const ui::Monitor* monitor = gfx::Screen::GetMonitorNearestWindow(window);

Since this requires changes to all platforms, I'd like to do it in separate CL. 
Let me know what you think.

[Ben Goodger (Google), 2012-03-14 17:23:16]

SG. LGTM for this