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

This reverts commit f2d9a44f3886aeab1378c8dbc1ce445202bc0ca5.

Review URL: http://codereview.chromium.org/115335

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@16030 0039d316-1c4b-4281-b951-d872f2087c98

3 files changed, 2 insertions, 104 deletions

diff --git a/chrome/renderer/extensions/extension_api_client_unittest.cc b/chrome/renderer/extensions/extension_api_client_unittest.cc
index 9689d95..6104bab 100644
--- a/chrome/renderer/extensions/extension_api_client_unittest.cc
+++ b/chrome/renderer/extensions/extension_api_client_unittest.cc
@@ -172,89 +172,6 @@ TEST_F(ExtensionAPIClientTest, GetAllWindows) {
                "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) {
diff --git a/chrome/renderer/renderer_resources.grd b/chrome/renderer/renderer_resources.grd
index cda253b..c59e98c 100644
--- a/chrome/renderer/renderer_resources.grd
+++ b/chrome/renderer/renderer_resources.grd
@@ -1,6 +1,6 @@
 <?xml version="1.0" encoding="UTF-8"?>
 <!-- This comment is only here because changes to resources are not picked up
-without changes to the corresponding grd file. -->
+without changes to the corresponding grd file.  -->
 <grit latest_public_release="0" current_release="1">
   <outputs>
     <output filename="grit/renderer_resources.h" type="rc_header">
diff --git a/chrome/renderer/resources/extension_process_bindings.js b/chrome/renderer/resources/extension_process_bindings.js
index 3828ee2..8b17e63 100644
--- a/chrome/renderer/resources/extension_process_bindings.js
+++ b/chrome/renderer/resources/extension_process_bindings.js
@@ -14,7 +14,6 @@ var chrome;
   native function GetCurrentWindow();
   native function GetLastFocusedWindow();
   native function CreateWindow();
-  native function UpdateWindow();
   native function RemoveWindow();
   native function GetAllWindows();
   native function GetTab();
@@ -166,25 +165,7 @@ var chrome;
     },
     chrome.types.optFun
   ];
-
-  chrome.windows.update = function(windowId, updateData, callback) {
-    validate(arguments, arguments.callee.params);
-    sendRequest(UpdateWindow, [windowId, updateData], callback);
-  };
-  chrome.windows.update.params = [
-    chrome.types.pInt,
-    {
-      type: "object",
-      properties: {
-        left: chrome.types.optInt,
-        top: chrome.types.optInt,
-        width: chrome.types.optPInt,
-        height: chrome.types.optPInt
-      },
-    },
-    chrome.types.optFun
-  ];
-
+  
   chrome.windows.remove = function(windowId, callback) {
     validate(arguments, arguments.callee.params);
     sendRequest(RemoveWindow, windowId, callback);