Allow extensions to pin tabs:...

chrome/renderer/extensions/extension_api_client_unittest.cc

Index: chrome/renderer/extensions/extension_api_client_unittest.cc
===================================================================
--- chrome/renderer/extensions/extension_api_client_unittest.cc	(revision 28937)
+++ chrome/renderer/extensions/extension_api_client_unittest.cc	(working copy)
@@ -383,18 +383,22 @@
   ExpectJsFail("chrome.tabs.update(42, {selected: 'foo'});",
                "Uncaught Error: Invalid value for argument 1. Property "
                "'selected': Expected 'boolean' but got 'string'.");
+  ExpectJsFail("chrome.tabs.update(42, {pinned: 'foo'});",

[Aaron Boodman, 2010/10/14 18:04:17]

We don't really use this class anymore. We should actually remove it... But we
should add to extension_tabs_apitest.cc to exercise the api.

+               "Uncaught Error: Invalid value for argument 1. Property "
+               "'pinned': Expected 'boolean' but got 'string'.");
   ExpectJsFail("chrome.tabs.update(42, {url: 42});",
                "Uncaught Error: Invalid value for argument 1. Property "
                "'url': Expected 'string' but got 'integer'.");
 
   ExpectJsPass("chrome.tabs.update(42, {"
                "  url:'http://www.google.com/',"
-               "  selected:true"
+               "  selected:true,"
+               "  pinned:false"
                "})",
                "tabs.update",
                "[42,"
                "{\"url\":\"http://www.google.com/\","
-               "\"selected\":true}]");
+               "\"selected\":true,\"pinned\":false}]");
 }
 
 TEST_F(ExtensionAPIClientTest, MoveTab) {