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

ui/aura/monitor_aura.h

 // 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.
 
 #ifndef UI_AURA_MONITOR_H_
 #define UI_AURA_MONITOR_H_
 #pragma once
 
 #include "base/basictypes.h"
 #include "ui/aura/aura_export.h"
 #include "ui/gfx/insets.h"
 #include "ui/gfx/rect.h"
 
+namespace gfx {
+class Monitor;
+}
+
 namespace aura {
 
-// Note: The screen and monitor currently uses pixels coordinate
-// system.  ENABLE_DIP macro (which is enabled with enable_dip=1 gyp
-// flag) will make this inconsistent with views' coordinate system
-// because views will use DIP coordinate system, which uses
-// (1.0/device_scale_factor) scale of the pixel coordinate system.
-// TODO(oshima): Change aura/screen to DIP coordinate system and
-// update this comment.
-class AURA_EXPORT Monitor {
+class AURA_EXPORT MonitorAura {

[Ben Goodger (Google), 2012/04/13 19:57:19]

What's the purpose of this class when you have gfx::Monitor? It looks pretty
much identical.

[oshima, 2012/04/13 21:18:41]

WorkArea is computed from bounds and insets on Aura, which is set by WM.
Alternatively, I can compute the insets on the fly when bounds changes.
I'm fine with either way, so let me know which way you prefer.

  public:
-  Monitor();
-  ~Monitor();
+  MonitorAura();
+  explicit MonitorAura(const gfx::Rect& bounds);
+  virtual ~MonitorAura();
 
   // Sets/gets monitor's bounds in |gfx::screen|'s coordinates,
   // which is relative to the primary screen's origin.
   void set_bounds(const gfx::Rect& bounds) { bounds_ = bounds;}
   const gfx::Rect& bounds() const { return bounds_; };
 
   // Sets/gets monitor's size.
   void set_size(const gfx::Size& size) { bounds_.set_size(size); }
   const gfx::Size& size() const { return bounds_.size(); }
 
   // Sets/gets monitor's workarea insets.
   void set_work_area_insets(const gfx::Insets& insets) {
     work_area_insets_ = insets;
   }
   const gfx::Insets& work_area_insets() const { return work_area_insets_; }
 
-  // Output device's pixel scale factor. This specifies how much the
-  // UI should be scaled when the actual output has more pixels than
-  // standard monitors (which is around 100~120dpi.) For aura, this
-  // value is either 1.0 or 2.0, but may return different values on
-  // other platforms.
-  float GetDeviceScaleFactor() const {
-    return device_scale_factor_;
-  }
-
+  // Sets the device scale factor.
   void set_device_scale_factor(float scale) {
     device_scale_factor_ = scale;
   }
 
-  // Returns the monitor's work area.
-  gfx::Rect GetWorkAreaBounds() const;
+  // Copy the monitor info into |monitor_out|.
+  void CopyTo(gfx::Monitor* monitor_out) const;
 
  private:
   // Insets for the work area.
   gfx::Insets work_area_insets_;
 
   gfx::Rect bounds_;
 
   float device_scale_factor_;
 
-  DISALLOW_COPY_AND_ASSIGN(Monitor);
+  DISALLOW_COPY_AND_ASSIGN(MonitorAura);
 };
 
 }  // namespace aura
 
 #endif  // UI_AURA_MONITOR_H_