chromeos: Make OutputConfigurator save latest config.

chromeos/display/output_configurator.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 CHROMEOS_DISPLAY_OUTPUT_CONFIGURATOR_H_
 #define CHROMEOS_DISPLAY_OUTPUT_CONFIGURATOR_H_
 
 #include <map>
 #include <string>
 #include <vector>
@@ skipping 24 matching lines @@
   STATE_DUAL_MIRROR,
   STATE_DUAL_EXTENDED,
 };
 
 // This class interacts directly with the underlying Xrandr API to manipulate
 // CTRCs and Outputs.
 class CHROMEOS_EXPORT OutputConfigurator
     : public base::MessageLoop::Dispatcher,
       public base::MessagePumpObserver {
  public:
+  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();
 
     RROutput output;
 
-    // CRTC that should be used for this output.  Not necessarily the CRTC
+    // 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;
-    int height;
 
     bool is_internal;
     bool is_aspect_preserving_scaling;
 
     // 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;
 
     bool has_display_id;
   };
 
-  struct CoordinateTransformation {
-    // Initialized to the identity transformation.
-    CoordinateTransformation();
-
-    float x_scale;
-    float x_offset;
-    float y_scale;
-    float y_offset;
-  };
-
-  struct CrtcConfig {
-    CrtcConfig();
-    CrtcConfig(RRCrtc crtc, int x, int y, RRMode mode, RROutput output);
-
-    RRCrtc crtc;
-    int x;
-    int y;
-    RRMode mode;
-    RROutput output;
-  };
-
   class Observer {
    public:
     virtual ~Observer() {}
 
     // Called when the change of the display mode finished.  It will usually
     // start the fading in the displays.
     virtual void OnDisplayModeChanged() {}
 
     // Called when the change of the display mode is issued but failed.
     // |failed_new_state| is the new state which the system failed to enter.
@@ skipping 63 matching lines @@
     virtual std::vector<OutputSnapshot> GetOutputs(
         const StateController* state_controller) = 0;
 
     // Gets details corresponding to |mode|.  Parameters may be NULL.
     // Returns true on success.
     virtual bool GetModeDetails(RRMode mode,
                                 int* width,
                                 int* height,
                                 bool* interlaced) = 0;
 
-    // Calls XRRSetCrtcConfig() with the given options but some of our
-    // default output count and rotation arguments.
-    virtual void ConfigureCrtc(CrtcConfig* config) = 0;
+    // Calls XRRSetCrtcConfig() with the given options but some of our default
+    // output count and rotation arguments. Returns true on success.
+    virtual bool ConfigureCrtc(RRCrtc crtc,
+                               RRMode mode,
+                               RROutput output,
+                               int x,
+                               int y) = 0;
 
     // Called to set the frame buffer (underlying XRR "screen") size.  Has
     // a side-effect of disabling all CRTCs.
     virtual void CreateFrameBuffer(
         int width,
         int height,
-        const std::vector<OutputConfigurator::CrtcConfig>& configs) = 0;
+        const std::vector<OutputConfigurator::OutputSnapshot>& 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;
 
     // Sends a D-Bus message to the power manager telling it that the
@@ skipping 119 matching lines @@
  private:
   // Configure outputs.
   void ConfigureOutputs();
 
   // Fires OnDisplayModeChanged() event to the observers.
   void NotifyOnDisplayChanged();
 
   // Switches to the state specified in |output_state| and |power_state|.
   // If the hardware mirroring failed and |mirroring_controller_| is set,
   // it switches to |STATE_DUAL_EXTENDED| and calls |SetSoftwareMirroring()|
-  // to enable software based mirroing.
-  // On success, updates |output_state_| and |power_state_| and returns true.
+  // to enable software based mirroring.
+  // On success, updates |output_state_|, |power_state_|, and |cached_outputs_|
+  // and returns true.
   bool EnterStateOrFallBackToSoftwareMirroring(
       OutputState output_state,
       DisplayPowerState power_state,
       const std::vector<OutputSnapshot>& outputs);
 
   // Switches to the state specified in |output_state| and |power_state|.
-  // On success, updates |output_state_| and |power_state_| and returns true.
+  // On success, updates |output_state_|, |power_state_|, and
+  // |cached_outputs_| and returns true.
   bool EnterState(OutputState output_state,
                   DisplayPowerState power_state,
                   const std::vector<OutputSnapshot>& outputs);
 
   // Returns the output state that should be used with |outputs| connected
   // while in |power_state|.
   OutputState GetOutputState(const std::vector<OutputSnapshot>& outputs,
                              DisplayPowerState power_state) const;
 
   // Computes the relevant transformation for mirror mode.
@@ skipping 29 matching lines @@
   // The base of the event numbers used to represent XRandr events used in
   // decoding events regarding output add/remove.
   int xrandr_event_base_;
 
   // The current display state.
   OutputState output_state_;
 
   // The current power state.
   DisplayPowerState power_state_;
 
+  // Most-recently-used output configuration. Note that the actual
+  // configuration changes asynchronously.
+  std::vector<OutputSnapshot> cached_outputs_;
+
   ObserverList<Observer> observers_;
 
   // The timer to delay configuring outputs. See also the comments in
   // Dispatch().
   scoped_ptr<base::OneShotTimer<OutputConfigurator> > configure_timer_;
 
   DISALLOW_COPY_AND_ASSIGN(OutputConfigurator);
 };
 
 typedef std::vector<OutputConfigurator::OutputSnapshot> OutputSnapshotList;
 
 }  // namespace chromeos
 
 #endif  // CHROMEOS_DISPLAY_OUTPUT_CONFIGURATOR_H_