Index: chrome/test/ui_test_utils.cc |
diff --git a/chrome/test/ui_test_utils.cc b/chrome/test/ui_test_utils.cc |
index ffe7a2ec51413410d89cc7b750ef99dbb478c4ea..ac553804eacce38dd9a038c8730ff359cbf13e85 100644 |
--- a/chrome/test/ui_test_utils.cc |
+++ b/chrome/test/ui_test_utils.cc |
@@ -75,9 +75,9 @@ class NavigationNotificationObserver : public NotificationObserver { |
class DOMOperationObserver : public NotificationObserver { |
public: |
- explicit DOMOperationObserver(TabContents* tab_contents) { |
+ explicit DOMOperationObserver(RenderViewHost* render_view_host) { |
registrar_.Add(this, NotificationType::DOM_OPERATION_RESPONSE, |
- Source<TabContents>(tab_contents)); |
+ Source<RenderViewHost>(render_view_host)); |
RunMessageLoop(); |
} |
@@ -230,16 +230,15 @@ void NavigateToURLBlockUntilNavigationsComplete(Browser* browser, |
WaitForNavigations(controller, number_of_navigations); |
} |
-Value* ExecuteJavaScript(TabContents* tab_contents, |
+Value* ExecuteJavaScript(RenderViewHost* render_view_host, |
const std::wstring& frame_xpath, |
const std::wstring& original_script) { |
// TODO(jcampan): we should make the domAutomationController not require an |
// automation id. |
std::wstring script = L"window.domAutomationController.setAutomationId(0);" + |
original_script; |
- tab_contents->render_view_host()->ExecuteJavascriptInWebFrame(frame_xpath, |
- script); |
- DOMOperationObserver dom_op_observer(tab_contents); |
+ render_view_host->ExecuteJavascriptInWebFrame(frame_xpath, script); |
+ DOMOperationObserver dom_op_observer(render_view_host); |
std::string json = dom_op_observer.response(); |
// Wrap |json| in an array before deserializing because valid JSON has an |
// array or an object as the root. |
@@ -259,36 +258,39 @@ Value* ExecuteJavaScript(TabContents* tab_contents, |
return result; |
} |
-bool ExecuteJavaScriptAndExtractInt(TabContents* tab_contents, |
+bool ExecuteJavaScriptAndExtractInt(RenderViewHost* render_view_host, |
const std::wstring& frame_xpath, |
const std::wstring& script, |
int* result) { |
DCHECK(result); |
- scoped_ptr<Value> value(ExecuteJavaScript(tab_contents, frame_xpath, script)); |
+ scoped_ptr<Value> value(ExecuteJavaScript(render_view_host, frame_xpath, |
+ script)); |
if (!value.get()) |
return false; |
return value->GetAsInteger(result); |
} |
-bool ExecuteJavaScriptAndExtractBool(TabContents* tab_contents, |
+bool ExecuteJavaScriptAndExtractBool(RenderViewHost* render_view_host, |
const std::wstring& frame_xpath, |
const std::wstring& script, |
bool* result) { |
DCHECK(result); |
- scoped_ptr<Value> value(ExecuteJavaScript(tab_contents, frame_xpath, script)); |
+ scoped_ptr<Value> value(ExecuteJavaScript(render_view_host, frame_xpath, |
+ script)); |
if (!value.get()) |
return false; |
return value->GetAsBoolean(result); |
} |
-bool ExecuteJavaScriptAndExtractString(TabContents* tab_contents, |
+bool ExecuteJavaScriptAndExtractString(RenderViewHost* render_view_host, |
const std::wstring& frame_xpath, |
const std::wstring& script, |
std::string* result) { |
DCHECK(result); |
- scoped_ptr<Value> value(ExecuteJavaScript(tab_contents, frame_xpath, script)); |
+ scoped_ptr<Value> value(ExecuteJavaScript(render_view_host, frame_xpath, |
+ script)); |
if (!value.get()) |
return false; |