Add frame color option to packaged app windows.

chrome/test/data/extensions/platform_apps/window_api/test.js

 // Copyright 2013 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.
 
 var callbackPass = chrome.test.callbackPass;
+var callbackFail = chrome.test.callbackFail;
 var defaultFuzzFactor = 1;
 
 function assertFuzzyEq(expected, actual, fuzzFactor, message) {
   if (!message) {
     message = "Expected: " + expected + "; actual: " + actual + "; "
                + "fuzzyFactor: " + fuzzFactor;
   }
 
   chrome.test.assertTrue(actual - fuzzFactor <= expected
                          && actual + fuzzFactor >= expected, message);
@@ skipping 536 matching lines @@
         win.clearBadge();
         win.setBadgeIcon('non_square.png');
         win.clearBadge();
         chrome.test.sendMessage(
             'WaitForRoundTrip', callbackPass(function(reply) {}));
       }));
     },
   ]);
 }
 
+function testFrameColors() {
+  chrome.test.runTests([
+    function testWithNoColor() {
+      chrome.app.window.create('test.html', callbackPass(function(win) {
+        chrome.test.assertEq(false, win.hasFrameColor);
+        win.close();
+      }));
+    },
+
+    function testWithFrameNone() {
+      chrome.app.window.create('test.html', {
+        frame: 'none'
+      },
+      callbackPass(function(win) {
+        chrome.test.assertEq(false, win.hasFrameColor);
+        win.close();
+      }));
+    },
+
+    function testWithBlack() {

[Matt Giuca, 2014/02/18 06:03:18]

I would prefer if one of these tests (black or white) was changed to a colour
with different channel values (like #abcd12) to ensure that the parsing
algorithm is correct.

[benwells, 2014/02/18 07:47:25]

As discussed, color parsing is unit tested.

+      chrome.app.window.create('test.html', {
+        frameOptions: {
+          type: 'chrome',
+          color: '#000000'
+        }
+      },
+      callbackPass(function(win) {
+        chrome.test.assertEq(true, win.hasFrameColor);
+        chrome.test.assertEq(-16777216, win.frameColor);
+        win.close();
+      }));
+    },
+
+    function testWithWhite() {
+      chrome.app.window.create('test.html', {
+        frameOptions: {
+          color: '#FFFFFF'
+        }
+      },
+      callbackPass(function(win) {
+        chrome.test.assertEq(true, win.hasFrameColor);
+        chrome.test.assertEq(-1, win.frameColor);
+        win.close();
+      }));
+    },
+
+    function testWithWhiteShorthand() {
+      chrome.app.window.create('test.html', {
+        frameOptions: {
+          color: '#FFF'
+        }
+      },
+      callbackPass(function(win) {
+        chrome.test.assertEq(true, win.hasFrameColor);
+        chrome.test.assertEq(-1, win.frameColor);
+        win.close();
+      }));
+    },
+
+    function testWithFrameAndFrameOptions() {
+      chrome.app.window.create('test.html', {
+        frame: 'chrome',
+        frameOptions: {
+          color: '#FFF'
+        }
+      },
+      callbackFail('Only one of frame and frameOptions can be supplied.'));
+    },
+
+    function testWithFrameNoneAndColor() {
+      chrome.app.window.create('test.html', {
+        frameOptions: {
+          type: 'none',
+          color: '#FFF'
+        }
+      },
+      callbackFail('Windows with no frame cannot have a color.'));
+    },
+
+    function testWithInvalidColor() {

[Matt Giuca, 2014/02/18 06:03:18]

Can you also test with an invalid string starting with # and length 7, such as
"#bcdefg" (which will test some other code paths).

Actually, preferably to exhaustively testing the color parsing code here would
be to have a separate unit test (not involving JS or the windowing API) to test
all the nutty cases.

+      chrome.app.window.create('test.html', {
+        frameOptions: {
+          color: 'DontWorryBeHappy'
+        }
+      },
+      callbackFail('The color specification could not be parsed.'));
+    }
+  ]);
+}
+
+function testFrameColorsInStable() {
+  chrome.test.runTests([
+    function testWithNoColor() {
+      chrome.app.window.create('test.html', callbackPass(function(win) {
+        chrome.test.assertEq(true, win.hasFrameColor);
+        chrome.test.assertEq(-1, win.frameColor);
+        win.close();
+      }));
+    },
+
+    function testWithOptionsGivesError() {
+      chrome.app.window.create('test.html', {
+        frameOptions: {
+          color: '#FFF'
+        }
+      },
+      callbackFail('frameOptions is only available in dev channel.'));
+    }
+  ]);
+}
+
 chrome.app.runtime.onLaunched.addListener(function() {
   chrome.test.sendMessage('Launched', function(reply) {
     window[reply]();
   });
 });