TBR Revert "Revert "implemented extensions api windows.update().""

chrome/renderer/extensions/extension_api_client_unittest.cc

 // Copyright (c) 2009 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.
 
 #include "base/file_util.h"
 #include "base/path_service.h"
 #include "base/string_util.h"
 #include "chrome/common/chrome_paths.h"
 #include "chrome/common/render_messages.h"
 #include "chrome/renderer/extensions/extension_process_bindings.h"
@@ skipping 154 matching lines @@
   ExpectJsPass("chrome.windows.getAll(true, function(){})",
                "GetAllWindows", "true");
 
   ExpectJsPass("chrome.windows.getAll(null, function(){})",
                "GetAllWindows", "null");
 
   ExpectJsPass("chrome.windows.getAll(undefined, function(){})",
                "GetAllWindows", "null");
 }
 
+TEST_F(ExtensionAPIClientTest, CreateWindow) {
+  ExpectJsFail("chrome.windows.create({url: 1}, function(){});",
+               "Uncaught Error: Invalid value for argument 0. Property "
+               "'url': Expected 'string' but got 'integer'.");
+  ExpectJsFail("chrome.windows.create({left: 'foo'}, function(){});",
+               "Uncaught Error: Invalid value for argument 0. Property "
+               "'left': Expected 'integer' but got 'string'.");
+  ExpectJsFail("chrome.windows.create({top: 'foo'}, function(){});",
+               "Uncaught Error: Invalid value for argument 0. Property "
+               "'top': Expected 'integer' but got 'string'.");
+  ExpectJsFail("chrome.windows.create({width: 'foo'}, function(){});",
+               "Uncaught Error: Invalid value for argument 0. Property "
+               "'width': Expected 'integer' but got 'string'.");
+  ExpectJsFail("chrome.windows.create({height: 'foo'}, function(){});",
+               "Uncaught Error: Invalid value for argument 0. Property "
+               "'height': Expected 'integer' but got 'string'.");
+  ExpectJsFail("chrome.windows.create({foo: 42}, function(){});",
+               "Uncaught Error: Invalid value for argument 0. Property "
+               "'foo': Unexpected property.");
+
+  ExpectJsPass("chrome.windows.create({"
+               "  url:'http://www.google.com/',"
+               "  left:0,"
+               "  top: 10,"
+               "  width:100,"
+               "  height:200"
+               "})",
+               "CreateWindow",
+               "{\"url\":\"http://www.google.com/\","
+               "\"left\":0,"
+               "\"top\":10,"
+               "\"width\":100,"
+               "\"height\":200}");
+}
+
+TEST_F(ExtensionAPIClientTest, UpdateWindow) {
+  ExpectJsFail("chrome.windows.update(null);",
+               "Uncaught Error: Parameter 0 is required.");
+  ExpectJsFail("chrome.windows.update(42, {left: 'foo'});",
+               "Uncaught Error: Invalid value for argument 1. Property "
+               "'left': Expected 'integer' but got 'string'.");
+  ExpectJsFail("chrome.windows.update(42, {top: 'foo'});",
+               "Uncaught Error: Invalid value for argument 1. Property "
+               "'top': Expected 'integer' but got 'string'.");
+  ExpectJsFail("chrome.windows.update(42, {height: false});",
+               "Uncaught Error: Invalid value for argument 1. Property "
+               "'height': Expected 'integer' but got 'boolean'.");
+  ExpectJsFail("chrome.windows.update(42, {width: false});",
+               "Uncaught Error: Invalid value for argument 1. Property "
+               "'width': Expected 'integer' but got 'boolean'.");
+  ExpectJsFail("chrome.windows.update(42, {foo: false});",
+               "Uncaught Error: Invalid value for argument 1. Property "
+               "'foo': Unexpected property.");
+
+  ExpectJsPass("chrome.windows.update(42, {"
+               "  width:100,"
+               "  height:200"
+               "})",
+               "UpdateWindow",
+               "[42,"
+               "{\"width\":100,"
+               "\"height\":200}]");
+}
+
+TEST_F(ExtensionAPIClientTest, RemoveWindow) {
+  ExpectJsFail("chrome.windows.remove(32, function(){}, 20);",
+               "Uncaught Error: Too many arguments.");
+
+  ExpectJsFail("chrome.windows.remove('abc', function(){});",
+               "Uncaught Error: Invalid value for argument 0. "
+               "Expected 'integer' but got 'string'.");
+
+  ExpectJsFail("chrome.windows.remove(1, 1);",
+               "Uncaught Error: Invalid value for argument 1. "
+               "Expected 'function' but got 'integer'.");
+
+  ExpectJsPass("chrome.windows.remove(2, function(){})",
+               "RemoveWindow", "2");
+
+  ExpectJsPass("chrome.windows.remove(2)",
+               "RemoveWindow", "2");
+}
+
 // Tab API tests
 
 TEST_F(ExtensionAPIClientTest, GetTab) {
   ExpectJsFail("chrome.tabs.get(32, function(){}, 20);",
                "Uncaught Error: Too many arguments.");
 
   ExpectJsFail("chrome.tabs.get(32);",
                "Uncaught Error: Parameter 1 is required.");
 
   ExpectJsFail("chrome.tabs.get('abc', function(){});",
@@ skipping 203 matching lines @@
                "SetBookmarkTitle",
                "{\"id\":42,\"title\":\"x\"}");
 }
 
 TEST_F(ExtensionAPIClientTest, EnablePageAction) {
   ExpectJsPass("chrome.pageActions.enableForTab("
                "\"dummy\", {tabId: 0, url: \"http://foo/\"});",
                "EnablePageAction",
                "[\"dummy\",{\"tabId\":0,\"url\":\"http://foo/\"}]");
 }