Implement a module system for the extension bindings JS.

chrome/renderer/resources/extensions/apitest.js

 // Copyright (c) 2011 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.
 
 // extension_apitest.js
 // mini-framework for ExtensionApiTest browser tests
 
-var chrome = chrome || {};
 (function() {

[Matt Perry, 2012/03/02 02:15:49]

This sort of extra wrapping can go away in all the files, right?

[koz (OOO until 15th September), 2012/03/02 02:23:06]

Yes, that's right. I've removed it from all the files, but I've left the
indenting the same for those that were indented, for blame's sake.

[not at google - send to devlin, 2012/03/02 02:24:00]

As in, who to blame for unnecessary indentation?

   chrome.test = chrome.test || {};
 
   chrome.test.tests = chrome.test.tests || [];
 
   var completed = false;
   var currentTest = null;
   var lastTest = null;
 
   function complete() {
     completed = true;
@@ skipping 249 matching lines @@
 
   chrome.test.callbackFail = function(expectedError, func) {
     return chrome.test.callback(func, expectedError);
   };
 
   chrome.test.runTests = function(tests) {
     chrome.test.tests = tests;
     chrome.test.runNextTest();
   };
 })();