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

ui/aura/monitor_manager.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 "ui/aura/monitor_manager.h"
 
 #include <stdio.h>
 
+#include "base/logging.h"
 #include "ui/aura/env.h"
-#include "ui/aura/monitor.h"
+#include "ui/aura/monitor_aura.h"
 #include "ui/aura/root_window.h"
 #include "ui/aura/root_window_host.h"
 #include "ui/gfx/rect.h"
+#include "ui/gfx/monitor_observer.h"
 
 namespace aura {
 namespace {
 // Default bounds for a monitor.
 static const int kDefaultHostWindowX = 200;
 static const int kDefaultHostWindowY = 200;
 static const int kDefaultHostWindowWidth = 1280;
 static const int kDefaultHostWindowHeight = 1024;
 }  // namespace
 
 // static
 bool MonitorManager::use_fullscreen_host_window_ = false;
 
 // static
-Monitor* MonitorManager::CreateMonitorFromSpec(const std::string& spec) {
+MonitorAura* MonitorManager::CreateMonitorFromSpec(const std::string& spec) {
   gfx::Rect bounds(kDefaultHostWindowX, kDefaultHostWindowY,
                    kDefaultHostWindowWidth, kDefaultHostWindowHeight);
   int x = 0, y = 0, width, height;
-  if (sscanf(spec.c_str(), "%dx%d", &width, &height) == 2) {
+  float scale = 1.0f;
+  if (sscanf(spec.c_str(), "%dx%d*%f", &width, &height, &scale) == 3) {

[Fady Samuel, 2012/04/10 16:58:09]

Where is this spec coming from? Are you reading the value from the
--default-device-scale-factor flag? We would like to be able to force a
"high-DPI" mode on development workstations.

[oshima, 2012/04/10 21:31:46]

The value comes from --aura-host-window-size.
I want this to be per monitor because I need to support multi-density
environment.
It needs to be renamed to something more relevant though.

     bounds.set_size(gfx::Size(width, height));
+  } else if (sscanf(spec.c_str(), "%dx%d", &width, &height) == 2) {
+    bounds.set_size(gfx::Size(width, height));
+  } else if (sscanf(spec.c_str(), "%d+%d-%dx%d*%f", &x, &y, &width, &height,
+                    &scale)
+             == 5) {
+    bounds = gfx::Rect(x, y, width, height);
   } else if (sscanf(spec.c_str(), "%d+%d-%dx%d", &x, &y, &width, &height)
              == 4) {
     bounds = gfx::Rect(x, y, width, height);
   } else if (use_fullscreen_host_window_) {
     bounds = gfx::Rect(aura::RootWindowHost::GetNativeScreenSize());
   }
-  Monitor* monitor = new Monitor();
+  MonitorAura* monitor = new MonitorAura();
   monitor->set_bounds(bounds);
+  monitor->set_device_scale_factor(scale);
   return monitor;
 }
 
 // static
 RootWindow* MonitorManager::CreateRootWindowForPrimaryMonitor() {
   MonitorManager* manager = aura::Env::GetInstance()->monitor_manager();
   RootWindow* root =
       manager->CreateRootWindowForMonitor(manager->GetMonitorAt(0));
   if (use_fullscreen_host_window_)
     root->ConfineCursorToWindow();
   return root;
 }
 
 MonitorManager::MonitorManager() {
 }
 
 MonitorManager::~MonitorManager() {
 }
 
-void MonitorManager::AddObserver(MonitorObserver* observer) {
+void MonitorManager::AddObserver(gfx::MonitorObserver* observer) {
   observers_.AddObserver(observer);
 }
 
-void MonitorManager::RemoveObserver(MonitorObserver* observer) {
+void MonitorManager::RemoveObserver(gfx::MonitorObserver* observer) {
   observers_.RemoveObserver(observer);
 }
 
-void MonitorManager::NotifyBoundsChanged(const Monitor* monitor) {
-  FOR_EACH_OBSERVER(MonitorObserver, observers_,
+void MonitorManager::NotifyBoundsChanged(const gfx::Monitor* monitor) {
+  FOR_EACH_OBSERVER(gfx::MonitorObserver, observers_,
                     OnMonitorBoundsChanged(monitor));
 }
 
-void MonitorManager::NotifyMonitorAdded(Monitor* monitor) {
-  FOR_EACH_OBSERVER(MonitorObserver, observers_,
+void MonitorManager::NotifyMonitorAdded(gfx::Monitor* monitor) {
+  FOR_EACH_OBSERVER(gfx::MonitorObserver, observers_,
                     OnMonitorAdded(monitor));
 }
 
-void MonitorManager::NotifyMonitorRemoved(const Monitor* monitor) {
-  FOR_EACH_OBSERVER(MonitorObserver, observers_,
+void MonitorManager::NotifyMonitorRemoved(const gfx::Monitor* monitor) {
+  FOR_EACH_OBSERVER(gfx::MonitorObserver, observers_,
                     OnMonitorRemoved(monitor));
 }
 
 }  // namespace aura