Add a few more unit tests for page actions

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/event_bindings.h"
@@ skipping 462 matching lines @@
                "[42,{\"parentId\":1,\"index\":0}]");
 }
 
 TEST_F(ExtensionAPIClientTest, SetBookmarkTitle) {
   ExpectJsPass("chrome.bookmarks.update(42,{title:'x'});",
                "SetBookmarkTitle",
                "[42,{\"title\":\"x\"}]");
 }
 
 TEST_F(ExtensionAPIClientTest, EnablePageAction) {
+  // Basic old-school enablePageAction call.
   ExpectJsPass("chrome.pageActions.enableForTab("
                "\"dummy\", {tabId: 0, url: \"http://foo/\"});",
                "EnablePageAction",
                "[\"dummy\",{\"tabId\":0,\"url\":\"http://foo/\"}]");
+  // Try both optional parameters (title and iconId).
+  ExpectJsPass("chrome.pageActions.enableForTab("
+               "\"dummy\", {tabId: 0, url: \"http://foo/\","
+                           "title: \"a\", iconId: 0});",
+               "EnablePageAction",
+               "[\"dummy\",{\"tabId\":0,\"url\":\"http://foo/\","
+                           "\"title\":\"a\",\"iconId\":0}]");
 
+  // Now try disablePageAction.
   ExpectJsPass("chrome.pageActions.disableForTab("
                "\"dummy\", {tabId: 0, url: \"http://foo/\"});",
                "DisablePageAction",
                "[\"dummy\",{\"tabId\":0,\"url\":\"http://foo/\"}]");
 }