Implement extension specific events

chrome/renderer/resources/extension_process_bindings.js

 // Copyright (c) 2009 The chrome Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 // -----------------------------------------------------------------------------
 // NOTE: If you change this file you need to touch renderer_resources.grd to
 // have your change take effect.
 // -----------------------------------------------------------------------------
 
 // This script contains privileged chrome extension related javascript APIs.
@@ skipping 12 matching lines @@
   native function GetTab();
   native function GetSelectedTab();
   native function GetAllTabsInWindow();
   native function CreateTab();
   native function UpdateTab();
   native function MoveTab();
   native function RemoveTab();
   native function GetTabLanguage();
   native function EnablePageAction();
   native function DisablePageAction();
+  native function GetCurrentPageActions();
   native function GetBookmarks();
   native function GetBookmarkChildren();
   native function GetBookmarkTree();
   native function SearchBookmarks();
   native function RemoveBookmark();
   native function CreateBookmark();
   native function MoveBookmark();
   native function SetBookmarkTitle();
   native function GetChromeHidden();
   native function GetNextRequestId();
@@ skipping 347 matching lines @@
     {
       type: "object",
       properties: {
         tabId: chrome.types.pInt,
         url: chrome.types.str
       },
       optional: false
     }
   ];
 
-  // Sends ({pageActionId, tabId, tabUrl}).
-  chrome.pageActions.onExecute =
-       new chrome.Event("page-action-executed");
+  // Page action events send (pageActionId, {tabId, tabUrl}).
+  function setupEventListeners(extensionId) {

[Aaron Boodman, 2009/07/14 21:28:52]

Rename this to something like setupPageActionEvents?

+    var pageActions = GetCurrentPageActions();
+    for (var i = 0; i < pageActions.length; ++i) {
+      var eventName = extensionId + "/" + pageActions[i];

[Aaron Boodman, 2009/07/14 21:28:52]

Nit: there's no block scope in JavaScript so this variable is really defined at
the same level as pageActions. Therefore the style is usually to declare it
there.

+      // Setup events for each extension_id/page_action_id string we find.
+      chrome.pageActions[pageActions[i]] = new chrome.Event(eventName);
+    }
+  }
 
   //----------------------------------------------------------------------------
   // Bookmarks
   chrome.bookmarks = {};
 
   chrome.bookmarks.get = function(ids, callback) {
     validate(arguments, arguments.callee.params);
     sendRequest(GetBookmarks, ids, callback);
   };
 
@@ skipping 128 matching lines @@
   //----------------------------------------------------------------------------
 
   // Self.
   chrome.self = chrome.self || {};
 
   chromeHidden.onLoad.addListener(function (extensionId) {
     chrome.extension = new chrome.Extension(extensionId);
     // TODO(mpcomplete): self.onConnect is deprecated.  Remove it at 1.0.
     // http://code.google.com/p/chromium/issues/detail?id=16356
     chrome.self.onConnect = new chrome.Event("channel-connect:" + extensionId);
+
+    setupEventListeners(extensionId);
   });
 
   chrome.self.getViews = function() {
     return GetViews();
   }
 })();