Add ability to change display settings to chrome.systemInfo.display

chrome/common/extensions/api/system_info_display.idl

Index: chrome/common/extensions/api/system_info_display.idl
diff --git a/chrome/common/extensions/api/system_info_display.idl b/chrome/common/extensions/api/system_info_display.idl
index 5438e359813b97a45ab8640aabea536d5dccb973..44283586f347635eae7821220b0b2f6bd77621c5 100644
--- a/chrome/common/extensions/api/system_info_display.idl
+++ b/chrome/common/extensions/api/system_info_display.idl
@@ -78,12 +78,67 @@ namespace systemInfo.display {
     // and launcher.
     Bounds workArea;
   };
+
+  dictionary SetDisplayUnitInfoParams {
+    // If set and not empty, starts mirroring between this and the display with
+    // the provided id (the system will determine which of the displays is
+    // actually mirrored).
+    // If set and not empty, stops mirroring between this and the display with
+    // the specified id (if mirroring is in progress).
+    // If set, no other parameter may be set.
+    DOMString? mirroringSourceId;
+  
+    // If set to true, makes the display primary. No-op if set to false.
+    boolean? isPrimary;
+ 
+    // If set, sets the display's overscan insets to the provided values. Note
+    // that overscan values may not be negative or larger than a half of the
+    // screen's size. Overscan cannot be changed on the internal monitor.
+    // It's applied after <code>isPrimary</code> parameter.
+    Insets? overscan;
+ 
+    // If set, updates the display's rotation.
+    // Legal values are [0, 90, 180, 270]. The rotation is set clockwise,
+    // relative to the display's vertical position.
+    // It's applied after <code>overscan</code> paramter.
+    long? rotation;
+ 
+    // If set, updates the display's logical bounds origin along x-axis. Applied
+    // together with <code>boundsOriginY</code>, if <code>boundsOriginY</code>
+    // is set. Note that, when updating the display origin, some constraints
+    // will be applied, so the final bounds origin may be different than the one
+    // set. The final bounds can be retrieved using $ref:getDisplayInfo.
+    // The bounds origin is applied after <code>rotation</code>.
+    // The bounds origin cannot be changed on the primary display. Note that is
+    // also invalid to set bounds origin values if <code>isPrimary</code> is
+    // also set (as <code>isPrimary</code> parameter is applied first).
+    long? boundsOriginX;
+ 
+    // If set, updates the display's logical bounds origin along y-axis.
+    // See documentation for <code>boundsOriginX</code> parameter.
+    long? boundsOriginY;
+  };
   
   callback DisplayInfoCallback = void (DisplayUnitInfo[] displayInfo);
+  callback SetDisplayUnitInfoCallback = void();
 
   interface Functions {
     // Get the information of all attached display devices.
     static void getDisplayInfo(DisplayInfoCallback callback);
+
+    // Updates the properties for the monitor specified by provided identifier,

[asargent_no_longer_on_chrome, 2013/06/14 00:18:59]

optional suggestion: s/"specified by provided identifier"/"specified by |id|"/

[tbarzic, 2013/06/14 01:08:18]

Done.

+    // according to the provided parameters. On failure,
+    // $ref:runtime.lastError will be set.
+    // |id|: The display's unique identifier.
+    // |info|: The information about display parameters that should be changed,
+    //     including their new values.
+    // |callback|: Empty function called when the function finishes. To find out
+    //     whether the function succeeded, $ref:runtime.lastError should be
+    //     queried. 
+    static void setDisplayUnitInfo(

[asargent_no_longer_on_chrome, 2013/06/14 00:18:59]

Would it be better to name this "setDisplayProperties" or "updateMonitor" or
something? The term "unit" seems sort of weird here (I know you use it to
connote a single display in the results you get back from getDisplayInfo, but
developers don't actually need to see it there). 

[tbarzic, 2013/06/14 01:08:18]

changed to setDisplayProperties

+        DOMString id,
+        SetDisplayUnitInfoParams info,
+        optional SetDisplayUnitInfoCallback callback);
   };
   
   interface Events {