Implement a module system for the extension bindings JS.

chrome/renderer/resources/extensions/browser_action_custom_bindings.js

 // Copyright (c) 2012 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.
 
 // Custom bindings for the browserAction API.
 
-(function() {
-
-native function GetChromeHidden();
+var GetChromeHidden = natives.GetChromeHidden;

[Aaron Boodman, 2012/02/16 01:54:42]

I'm still not crazy about the magical names. Did you try my suggestion of
wrapping in a function?

[koz (OOO until 15th September), 2012/02/27 01:06:14]

No - wrapping it in a function would definitely work, but I think it's just
extra boilerplate and an extra source of potential bugs. I think it's okay to
have these variables just appear in scope here - almost all JS environments
execute JS in a context where extra variables are available (eg: JS on the web,
JS in extensions, node) and they wouldn't be improved by requiring developers to
enumerate those variables as parameters in an anonymous function.

[Aaron Boodman, 2012/02/28 02:42:59]

OK, I see your point of view.

 
 GetChromeHidden().registerCustomHook('browserAction', function(bindingsAPI) {
   var apiFunctions = bindingsAPI.apiFunctions;
   var setIcon = bindingsAPI.setIcon;
 
   apiFunctions.setHandleRequest('browserAction.setIcon', function(details) {
     setIcon(details, this.name, this.definition.parameters, 'browser action');
   });
 });
-
-})();