Refactor screen/monitor so that gfx::Screen returns monitor object.

ui/aura/monitor_manager.cc

Index: ui/aura/monitor_manager.cc
diff --git a/ui/aura/monitor_manager.cc b/ui/aura/monitor_manager.cc
index c884c6ddf1a0b3d77fcf61d612695968e1d6362e..9a786021a13046f21fcd68a424919c776a3da368 100644
--- a/ui/aura/monitor_manager.cc
+++ b/ui/aura/monitor_manager.cc
@@ -8,9 +8,10 @@
 
 #include "base/logging.h"
 #include "ui/aura/env.h"
-#include "ui/aura/monitor.h"
+#include "ui/aura/monitor_observer.h"
 #include "ui/aura/root_window.h"
 #include "ui/aura/root_window_host.h"
+#include "ui/gfx/monitor.h"
 #include "ui/gfx/rect.h"
 
 namespace aura {
@@ -26,7 +27,7 @@ static const int kDefaultHostWindowHeight = 1024;
 bool MonitorManager::use_fullscreen_host_window_ = false;
 
 // static
-Monitor* MonitorManager::CreateMonitorFromSpec(const std::string& spec) {
+gfx::Monitor* MonitorManager::CreateMonitorFromSpec(const std::string& spec) {
   gfx::Rect bounds(kDefaultHostWindowX, kDefaultHostWindowY,
                    kDefaultHostWindowWidth, kDefaultHostWindowHeight);
   int x = 0, y = 0, width, height;
@@ -39,8 +40,7 @@ Monitor* MonitorManager::CreateMonitorFromSpec(const std::string& spec) {
   } else if (use_fullscreen_host_window_) {
     bounds = gfx::Rect(aura::RootWindowHost::GetNativeScreenSize());
   }
-  Monitor* monitor = new Monitor();
-  monitor->set_bounds(bounds);
+  gfx::Monitor* monitor = new gfx::Monitor(bounds);
   monitor->set_device_scale_factor(scale);
   DVLOG(1) << "Monitor bounds=" << bounds.ToString() << ", scale=" << scale;
   return monitor;
@@ -70,17 +70,17 @@ void MonitorManager::RemoveObserver(MonitorObserver* observer) {
   observers_.RemoveObserver(observer);
 }
 
-void MonitorManager::NotifyBoundsChanged(const Monitor* monitor) {
+void MonitorManager::NotifyBoundsChanged(const gfx::Monitor* monitor) {

[Ben Goodger (Google), 2012/04/24 21:17:08]

const gfx::Monitor& monitor

[oshima, 2012/04/25 02:38:01]

Done.

   FOR_EACH_OBSERVER(MonitorObserver, observers_,
                     OnMonitorBoundsChanged(monitor));
 }
 
-void MonitorManager::NotifyMonitorAdded(Monitor* monitor) {
+void MonitorManager::NotifyMonitorAdded(gfx::Monitor* monitor) {

[Ben Goodger (Google), 2012/04/24 21:17:08]

etc

[oshima, 2012/04/25 02:38:01]

Done.

   FOR_EACH_OBSERVER(MonitorObserver, observers_,
                     OnMonitorAdded(monitor));
 }
 
-void MonitorManager::NotifyMonitorRemoved(const Monitor* monitor) {
+void MonitorManager::NotifyMonitorRemoved(const gfx::Monitor* monitor) {

[Ben Goodger (Google), 2012/04/24 21:17:08]

etc

[oshima, 2012/04/25 02:38:01]

Done.

   FOR_EACH_OBSERVER(MonitorObserver, observers_,
                     OnMonitorRemoved(monitor));
 }