Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(89)

Unified Diff: chrome/browser/ui/webui/web_ui_browsertest.h

Issue 8586009: Allow WebUI Tests to use preLoad in HtmlDialogUI. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Revert to patch set 4. Created 9 years, 1 month ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: chrome/browser/ui/webui/web_ui_browsertest.h
diff --git a/chrome/browser/ui/webui/web_ui_browsertest.h b/chrome/browser/ui/webui/web_ui_browsertest.h
index 99f086abca1b80d16eeaf583e4df7ebc672af5d6..f121e5bc06df743aad165e23e1e1044197c06a03 100644
--- a/chrome/browser/ui/webui/web_ui_browsertest.h
+++ b/chrome/browser/ui/webui/web_ui_browsertest.h
@@ -12,7 +12,8 @@
#include "base/string16.h"
#include "chrome/browser/ui/webui/web_ui_test_handler.h"
#include "chrome/test/base/in_process_browser_test.h"
-#include "chrome/test/test_navigation_observer.h"
+#include "chrome/test/base/js_injection_ready_observer.h"
+#include "chrome/test/base/test_navigation_observer.h"
class RenderViewHost;
class WebUIMessageHandler;
@@ -37,7 +38,7 @@ class Value;
// and the lone test within this class.
class WebUIBrowserTest
: public InProcessBrowserTest,
- public TestNavigationObserver::JsInjectionReadyObserver {
+ public JsInjectionReadyObserver {
public:
typedef std::vector<const base::Value*> ConstValueVector;
virtual ~WebUIBrowserTest();
@@ -101,17 +102,13 @@ class WebUIBrowserTest
// the javascript for the given |preload_test_fixture| and
// |preload_test_name|. chrome.send will be overridden to allow javascript
// handler mocking.
- void BrowsePreload(const GURL& browse_to,
- const std::string& preload_test_fixture,
- const std::string& preload_test_name);
+ void BrowsePreload(const GURL& browse_to);
// Called by javascript-generated test bodies to browse to a page and preload
// the javascript for the given |preload_test_fixture| and
// |preload_test_name|. chrome.send will be overridden to allow javascript
// handler mocking.
- void BrowsePrintPreload(const GURL& browse_to,
- const std::string& preload_test_fixture,
- const std::string& preload_test_name);
+ void BrowsePrintPreload(const GURL& browse_to);
protected:
// URL to dummy WebUI page for testing framework.
@@ -119,6 +116,14 @@ class WebUIBrowserTest
WebUIBrowserTest();
+ // Accessors for preload test fixture and name.
+ void set_preload_test_fixture(const std::string& preload_test_fixture) {
+ preload_test_fixture_ = preload_test_fixture;
James Hawkins 2011/11/22 23:03:29 Move the implementation the .cc file.
Sheridan Rawlins 2011/11/23 20:10:07 Done.
+ }
+ void set_preload_test_name(const std::string& preload_test_name) {
+ preload_test_name_ = preload_test_name;
+ }
+
// Set up & tear down console error catching.
virtual void SetUpOnMainThread() OVERRIDE;
virtual void CleanUpOnMainThread() OVERRIDE;
@@ -140,7 +145,7 @@ class WebUIBrowserTest
static GURL WebUITestDataPathToURL(const FilePath::StringType& path);
private:
- // TestNavigationObserver::JsInjectionReadyObserver implementation.
+ // JsInjectionReadyObserver implementation.
virtual void OnJsInjectionReady(RenderViewHost* render_view_host) OVERRIDE;
// Builds a string containing all added javascript libraries.

Powered by Google App Engine
This is Rietveld 408576698