Make it possible to include another file and port print_preview unit test to unit_tests.

chrome/test/data/webui/test_api.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.
 
 /**
  * @fileoverview Library providing basic test framework functionality.
  */
 
 /**
  * Namespace for |Test|.
  * @type {Object}
  */
 var testing = {};
 (function(window) {

[arv (Not doing code reviews), 2011/11/03 19:04:12]

rename window to exports for consistence with code out there

[Sheridan Rawlins, 2011/11/05 16:51:25]

Done.

   /**
    * Hold the currentTestCase across between preLoad and run.
    * @type {TestCase}
    */
   var currentTestCase = null;
 
   /**
    * The string representation of the currently running test function.
    * @type {?string}
    */
@@ skipping 78 matching lines @@
      */
     isAsync: false,
 
     /**
      * True when the test is expected to fail for testing the test framework.
      * @type {boolean}
      */
     testShouldFail: false,
 
     /**
+     * Add libraries before loading this test file.

[dpapad, 2011/11/03 16:14:34]

Would this be a bit more clear as "Libraries to add before loading this test
file."?

[Sheridan Rawlins, 2011/11/05 16:51:25]

Done.

+     * @type {Array.<string>}
+     */
+    addLibraries: [],

[arv (Not doing code reviews), 2011/11/03 19:04:12]

This should just be called libraries or something like that.

[Sheridan Rawlins, 2011/11/05 16:51:25]

Changed to extraLibraries - does that work for you?

Done.

On 2011/11/03 19:04:12, arv wrote:

+
+    /**
      * Override this method to perform initialization during preload (such as
      * creating mocks and registering handlers).
      * @type {Function}
      */
     preLoad: function() {},
 
     /**
      * Override this method to perform tasks before running your test.
      * @type {Function}
      */
@@ skipping 722 matching lines @@
     currentTestCase = createTestCase(testFixture, testName);
     currentTestCase.preLoad();
   }
 
   /**
    * During generation phase, this outputs; do nothing at runtime.
    */
   function GEN() {}
 
   /**
+   * During generation phase, this outputs; do nothing at runtime.
+   */
+  function GEN_INCLUDE() {}
+
+  /**
    * At runtime, register the testName with a test fixture. Since this method
    * doesn't have a test fixture, create a dummy fixture to hold its |name|
    * and |testCaseBodies|.
    * @param {string} testCaseName The name of the test case.
    * @param {string} testName The name of the test function.
    * @param {Function} testBody The body to execute when running this test.
    */
   function TEST(testCaseName, testName, testBody) {
     var fixtureConstructor = this[testCaseName];
     if (fixtureConstructor === undefined) {
@@ skipping 432 matching lines @@
   window.resetTestState = resetTestState;
   window.runAllActions = runAllActions;
   window.runAllActionsAsync = runAllActionsAsync;
   window.runTest = runTest;
   window.runTestFunction = runTestFunction;
   window.SaveMockArguments = SaveMockArguments;
   window.DUMMY_URL = DUMMY_URL;
   window.TEST = TEST;
   window.TEST_F = TEST_F;
   window.GEN = GEN;
+  window.GEN_INCLUDE = GEN_INCLUDE;
   window.WhenTestDone = WhenTestDone;
 
   // Import the Mock4JS helpers.
   Mock4JS.addMockSupport(window);
 })(('window' in this) ? window : this);

[arv (Not doing code reviews), 2011/11/03 19:04:12]

What does window in this do here? How is this better than just passing this at
all times?

[Sheridan Rawlins, 2011/11/05 16:51:25]

I forget why I needed this at some point - it may have been when experimenting
with node.js...

I'll remove and see how things fly with trybots.

Done.

On 2011/11/03 19:04:12, arv wrote: