Allow javascript unit tests using webui test_api framework.

chrome/test/base/v8_unit_test.h

 // 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.
 
 #ifndef CHROME_TEST_BASE_V8_UNIT_TEST_H_
 #define CHROME_TEST_BASE_V8_UNIT_TEST_H_
 #pragma once
 
 #include <string>
+#include <vector>
 
+#include "base/file_path.h"
 #include "testing/gtest/include/gtest/gtest.h"
 #include "v8/include/v8.h"
 
 namespace base {
 class StringPiece;
 }
 
 // A superclass for unit tests that involve running JavaScript.  This class
 // sets up V8 context and has methods that make it easy to execute scripts in
 // this context as well as call functions in the context.
 class V8UnitTest : public testing::Test {
  public:
   V8UnitTest();
   virtual ~V8UnitTest();
 
   virtual void SetUp();
 
  protected:
+  void AddLibrary(const FilePath& library_path);

[James Hawkins, 2011/10/29 22:15:48]

Document these methods and parameters.

[Sheridan Rawlins, 2011/10/30 17:04:21]

Done.

+  bool RunJavascriptTestF(const std::string& testFixture,
+                          const std::string& testName);
+
   // Executes the given script source in the context.  The specified script
   // name is used when reporting errors.
   virtual void ExecuteScriptInContext(const base::StringPiece& script_source,
                                       const base::StringPiece& script_name);
 
   // Set a variable to a string value in the global scope.
   virtual void SetGlobalStringVar(const std::string& var_name,
                                   const std::string& value);
 
   // Converts a v8::TryCatch into a human readable string.
   virtual std::string ExceptionToString(const v8::TryCatch& try_catch);
 
   // Calls the specified function that resides in the global scope of the
   // context.  If the function throws an exception, FAIL() is called to
   // indicate a unit test failure.  This is useful for executing unit test
   // functions implemented in JavaScript.
   virtual void TestFunction(const std::string& function_name);
 
-  // This method is bound to a global function "log" in the context.
-  // Scripts running in the context can call this to print out logging
-  // information to the console.
+  // This method is bound to a global function "log" in the context, as well as
+  // to log, warn, and info of the console object. Scripts running in the
+  // context can call this to print out logging information to the console.
   static v8::Handle<v8::Value> Log(const v8::Arguments& args);
 
+  // This method is bound to console.error in the context. Any calls to this
+  // also signal an error condition and |RunJavascriptF| will fail.
+  static v8::Handle<v8::Value> Error(const v8::Arguments& args);
+
+  // This method is bound to a method "chrome.send" in the context.  When
+  // test_api calls testDone to report its results, this will capture and hold
+  // the results for analysis by |RunJavascriptF|.
+  static v8::Handle<v8::Value> ChromeSend(const v8::Arguments& args);
+
+ private:
+  // Executes all added javascript libraries. Returns true if passing so far.

[James Hawkins, 2011/10/29 22:15:48]

What does "if passing so far" mean?

[Sheridan Rawlins, 2011/10/30 17:04:21]

Done.

+  bool ExecuteJavascriptLibraries();
+
+  // Initialize paths and libraries.

[James Hawkins, 2011/10/29 22:15:48]

Initializes

[Sheridan Rawlins, 2011/10/30 17:04:21]

Done.

+  void InitPathsAndLibraries();
+
   // Handle scope that is used throughout the life of this class.
   v8::HandleScope handle_scope_;
 
   // Context for the JavaScript in the test.
   v8::Handle<v8::Context> context_;
+
+  // Location of test data (currently test/data/webui).

[James Hawkins, 2011/10/29 22:15:48]

Don't add implementation details to the interface (even if in private section). 
It's not unlikely to change.  Here and below.

[Sheridan Rawlins, 2011/10/30 17:04:21]

Done.

+  FilePath test_data_directory_;
+
+  // Location of generated test data (<(PROGRAM_DIR)/test_data).
+  FilePath gen_test_data_directory_;
+
+  // User added libraries.
+  std::vector<FilePath> user_libraries_;
+
+  // Whether errors were seen.
+  static bool had_errors_;

[James Hawkins, 2011/10/29 22:15:48]

Are statics particularly necessary?

[Sheridan Rawlins, 2011/10/30 17:04:21]

Done.

+
+  // testDone results.
+  static bool testResult_ok_;
 };
 
 #endif  // CHROME_TEST_BASE_V8_UNIT_TEST_H_