Refactoring display configuration state to allow generic state objects

ui/display/chromeos/output_configurator.h

 // Copyright 2014 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_DISPLAY_CHROMEOS_OUTPUT_CONFIGURATOR_H_
 #define UI_DISPLAY_CHROMEOS_OUTPUT_CONFIGURATOR_H_
 
 #include <map>
 #include <string>
 #include <vector>
 
 #include "base/basictypes.h"
 #include "base/event_types.h"
 #include "base/memory/scoped_ptr.h"
 #include "base/observer_list.h"
 #include "base/timer/timer.h"
 #include "third_party/cros_system_api/dbus/service_constants.h"
 #include "ui/display/chromeos/native_display_observer.h"
 #include "ui/display/display_constants.h"
 #include "ui/display/display_export.h"
 
-// Forward declarations for Xlib and Xrandr.
-// This is so unused X definitions don't pollute the namespace.
-typedef unsigned long XID;
-typedef XID RROutput;
-typedef XID RRCrtc;
-typedef XID RRMode;
+namespace gfx {
+class Point;
+class Size;
+}
 
 namespace ui {
-
+class DisplayMode;
+class DisplaySnapshot;
 class NativeDisplayDelegate;
 
-// This class interacts directly with the underlying Xrandr API to manipulate
-// CTRCs and Outputs.
+// This class interacts directly with the system display configurator.
 class DISPLAY_EXPORT OutputConfigurator : public NativeDisplayObserver {
  public:
   typedef uint64_t OutputProtectionClientId;
   static const OutputProtectionClientId kInvalidClientId = 0;
 
-  struct ModeInfo {
-    ModeInfo();
-    ModeInfo(int width, int height, bool interlaced, float refresh_rate);
-
-    int width;
-    int height;
-    bool interlaced;
-    float refresh_rate;
-  };
-
-  typedef std::map<RRMode, ModeInfo> ModeInfoMap;
-
   struct CoordinateTransformation {
     // Initialized to the identity transformation.
     CoordinateTransformation();
 
     float x_scale;
     float x_offset;
     float y_scale;
     float y_offset;
   };
 
-  // Information about an output's current state.
-  struct OutputSnapshot {
-    OutputSnapshot();
-    ~OutputSnapshot();
+  struct DisplayState {
+    DisplayState();
 
-    RROutput output;
-
-    // CRTC that should be used for this output. Not necessarily the CRTC
-    // that XRandR reports is currently being used.
-    RRCrtc crtc;
-
-    // Mode currently being used by the output.
-    RRMode current_mode;
-
-    // "Best" mode supported by the output.
-    RRMode native_mode;
-
-    // Mode used when displaying the same desktop on multiple outputs.
-    RRMode mirror_mode;
-
-    // User-selected mode for the output.
-    RRMode selected_mode;
-
-    // Output's origin on the framebuffer.
-    int x;
-    int y;
-
-    // Output's physical dimensions.
-    uint64 width_mm;
-    uint64 height_mm;
-
-    bool is_aspect_preserving_scaling;
-
-    // The type of output.
-    OutputType type;
-
-    // Map from mode IDs to details about the corresponding modes.
-    ModeInfoMap mode_infos;
+    DisplaySnapshot* display;  // Not owned.
 
     // XInput device ID or 0 if this output isn't a touchscreen.
     int touch_device_id;
 
     CoordinateTransformation transform;
 
-    // Display id for this output.
-    int64 display_id;
+    // User-selected mode for the output.
+    const DisplayMode* selected_mode;
 
-    bool has_display_id;
+    // Mode used when displaying the same desktop on multiple outputs.
+    const DisplayMode* mirror_mode;
+  };
 
-    // This output's index in the array returned by XRandR. Stable even as
-    // outputs are connected or disconnected.
-    int index;
-  };
+  typedef std::vector<DisplayState> DisplayStateList;
 
   class Observer {
    public:
     virtual ~Observer() {}
 
     // Called after the display mode has been changed. |output| contains the
     // just-applied configuration. Note that the X server is no longer grabbed
     // when this method is called, so the actual configuration could've changed
     // already.
     virtual void OnDisplayModeChanged(
-        const std::vector<OutputSnapshot>& outputs) {}
+        const std::vector<DisplayState>& outputs) {}
 
     // Called after a display mode change attempt failed. |failed_new_state| is
     // the new state which the system failed to enter.
     virtual void OnDisplayModeChangeFailed(OutputState failed_new_state) {}
   };
 
   // Interface for classes that make decisions about which output state
   // should be used.
   class StateController {
    public:
     virtual ~StateController() {}
 
     // Called when displays are detected.
     virtual OutputState GetStateForDisplayIds(
         const std::vector<int64>& display_ids) const = 0;
 
-    // Queries the resolution (|width|x|height|) in pixels
-    // to select output mode for the given display id.
+    // Queries the resolution (|size|) in pixels to select output mode for the
+    // given display id.
     virtual bool GetResolutionForDisplayId(int64 display_id,
-                                           int* width,
-                                           int* height) const = 0;
+                                           gfx::Size* size) const = 0;
   };
 
   // Interface for classes that implement software based mirroring.
   class SoftwareMirroringController {
    public:
     virtual ~SoftwareMirroringController() {}
 
     // Called when the hardware mirroring failed.
     virtual void SetSoftwareMirroring(bool enabled) = 0;
   };
 
   class TouchscreenDelegate {
    public:
     virtual ~TouchscreenDelegate() {}
 
     // Searches for touchscreens among input devices,
     // and tries to match them up to screens in |outputs|.
     // |outputs| is an array of detected screens.
     // If a touchscreen with same resolution as an output's native mode
     // is detected, its id will be stored in this output.
-    virtual void AssociateTouchscreens(
-        std::vector<OutputSnapshot>* outputs) = 0;
+    virtual void AssociateTouchscreens(std::vector<DisplayState>* outputs) = 0;
 
     // Configures XInput's Coordinate Transformation Matrix property.
     // |touch_device_id| the ID of the touchscreen device to configure.
     // |ctm| contains the desired transformation parameters.  The offsets
     // in it should be normalized so that 1 corresponds to the X or Y axis
     // size for the corresponding offset.
     virtual void ConfigureCTM(int touch_device_id,
                               const CoordinateTransformation& ctm) = 0;
   };
 
@@ skipping 23 matching lines @@
 
   // Gap between screens so cursor at bottom of active display doesn't
   // partially appear on top of inactive display. Higher numbers guard
   // against larger cursors, but also waste more memory.
   // For simplicity, this is hard-coded to avoid the complexity of always
   // determining the DPI of the screen and rationalizing which screen we
   // need to use for the DPI calculation.
   // See crbug.com/130188 for initial discussion.
   static const int kVerticalGap = 60;
 
-  // Returns a pointer to the ModeInfo struct in |output| corresponding to
-  // |mode|, or NULL if the struct isn't present.
-  static const ModeInfo* GetModeInfo(const OutputSnapshot& output, RRMode mode);
-
   // Returns the mode within |output| that matches the given size with highest
   // refresh rate. Returns None if no matching output was found.
-  static RRMode FindOutputModeMatchingSize(const OutputSnapshot& output,
-                                           int width,
-                                           int height);
+  static const DisplayMode* FindDisplayModeMatchingSize(
+      const DisplaySnapshot& output,
+      const gfx::Size& size);
 
   OutputConfigurator();
   virtual ~OutputConfigurator();
 
   OutputState output_state() const { return output_state_; }
   chromeos::DisplayPowerState power_state() const { return power_state_; }
-  const std::vector<OutputSnapshot>& cached_outputs() const {
+  const std::vector<DisplayState>& cached_outputs() const {
     return cached_outputs_;
   }
 
   void set_state_controller(StateController* controller) {
     state_controller_ = controller;
   }
   void set_mirroring_controller(SoftwareMirroringController* controller) {
     mirroring_controller_ = controller;
   }
 
@@ skipping 93 matching lines @@
   // outputs' |mirror_mode| fields by looking for a mode in |internal_output|
   // and |external_output| having the same resolution. Returns false if a shared
   // mode wasn't found or created.
   //
   // |try_panel_fitting| allows creating a panel-fitting mode for
   // |internal_output| instead of only searching for a matching mode (note that
   // it may lead to a crash if |internal_info| is not capable of panel fitting).
   //
   // |preserve_aspect| limits the search/creation only to the modes having the
   // native aspect ratio of |external_output|.
-  bool FindMirrorMode(OutputSnapshot* internal_output,
-                      OutputSnapshot* external_output,
+  bool FindMirrorMode(DisplayState* internal_output,
+                      DisplayState* external_output,
                       bool try_panel_fitting,
                       bool preserve_aspect);
 
   // Configures outputs.
   void ConfigureOutputs();
 
   // Notifies observers about an attempted state change.
   void NotifyObservers(bool success, OutputState attempted_state);
 
   // Switches to the state specified in |output_state| and |power_state|.
@@ skipping 12 matching lines @@
   bool EnterState(OutputState output_state,
                   chromeos::DisplayPowerState power_state);
 
   // Returns the output state that should be used with |cached_outputs_| while
   // in |power_state|.
   OutputState ChooseOutputState(chromeos::DisplayPowerState power_state) const;
 
   // Computes the relevant transformation for mirror mode.
   // |output| is the output on which mirror mode is being applied.
   // Returns the transformation or identity if computations fail.
-  CoordinateTransformation GetMirrorModeCTM(
-      const OutputConfigurator::OutputSnapshot& output);
+  CoordinateTransformation GetMirrorModeCTM(const DisplayState& output);
 
-  // Computes the relevant transformation for extended mode.
-  // |output| is the output on which extended mode is being applied.
-  // |width| and |height| are the width and height of the combined framebuffer.
+  // Computes the relevant transformation for extended mode. |output| is the
+  // output on which extended mode is being applied. |new_origin| is the
+  // position of the output on the framebuffer. |framebuffer_size| is the
+  // size of the combined framebuffer.
   // Returns the transformation or identity if computations fail.
   CoordinateTransformation GetExtendedModeCTM(
-      const OutputConfigurator::OutputSnapshot& output,
-      int framebuffer_width,
-      int frame_buffer_height);
+      const DisplayState& output,
+      const gfx::Point& new_origin,
+      const gfx::Size& framebuffer_size);
 
   // Returns the ratio between mirrored mode area and native mode area:
   // (mirror_mode_width * mirrow_mode_height) / (native_width * native_height)
-  float GetMirroredDisplayAreaRatio(
-      const OutputConfigurator::OutputSnapshot& output);
+  float GetMirroredDisplayAreaRatio(const DisplayState& output);
 
   // Applies output protections according to requests.
   bool ApplyProtections(const DisplayProtections& requests);
 
   StateController* state_controller_;
   SoftwareMirroringController* mirroring_controller_;
   scoped_ptr<NativeDisplayDelegate> native_display_delegate_;
   scoped_ptr<TouchscreenDelegate> touchscreen_delegate_;
 
   // Used to enable modes which rely on panel fitting.
@@ skipping 15 matching lines @@
   bool configure_display_;
 
   // The current display state.
   OutputState output_state_;
 
   // The current power state.
   chromeos::DisplayPowerState power_state_;
 
   // Most-recently-used output configuration. Note that the actual
   // configuration changes asynchronously.
-  std::vector<OutputSnapshot> cached_outputs_;
+  DisplayStateList cached_outputs_;
 
   ObserverList<Observer> observers_;
 
   // The timer to delay configuring outputs. See also the comments in
   // Dispatch().
   scoped_ptr<base::OneShotTimer<OutputConfigurator> > configure_timer_;
 
   // Id for next output protection client.
   OutputProtectionClientId next_output_protection_client_id_;
 
   // Output protection requests of each client.
   ProtectionRequests client_protection_requests_;
 
   DISALLOW_COPY_AND_ASSIGN(OutputConfigurator);
 };
 
-typedef std::vector<OutputConfigurator::OutputSnapshot> OutputSnapshotList;
-
 }  // namespace ui
 
 #endif  // UI_DISPLAY_CHROMEOS_OUTPUT_CONFIGURATOR_H_