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

Unified Diff: chrome/browser/net/load_timing_observer_browsertest.cc

Issue 11753009: Simplify ExecuteJavaScript* functions. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Update prerender_browsertest.cc. Created 7 years, 12 months 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/net/load_timing_observer_browsertest.cc
diff --git a/chrome/browser/net/load_timing_observer_browsertest.cc b/chrome/browser/net/load_timing_observer_browsertest.cc
index b97a706533c6fb83397a45bddf2ac20bada0323d..3d143ddf9d4b6acc7cb1f3f67474d85824de5c00 100644
--- a/chrome/browser/net/load_timing_observer_browsertest.cc
+++ b/chrome/browser/net/load_timing_observer_browsertest.cc
@@ -26,18 +26,15 @@ IN_PROC_BROWSER_TEST_F(LoadTimingObserverTest, DISABLED_CacheHitAfterRedirect) {
int response_start = 0;
int response_end = 0;
- content::RenderViewHost* render_view_host =
- chrome::GetActiveWebContents(browser())->GetRenderViewHost();
- ASSERT_TRUE(content::ExecuteJavaScriptAndExtractInt(
- render_view_host,
- "",
+ content::WebContents* contents = chrome::GetActiveWebContents(browser());
+ ASSERT_TRUE(content::ExecuteScriptAndExtractInt(
+ contents,
"window.domAutomationController.send("
" window.performance.timing.responseStart - "
" window.performance.timing.navigationStart)",
&response_start));
- ASSERT_TRUE(content::ExecuteJavaScriptAndExtractInt(
- render_view_host,
- "",
+ ASSERT_TRUE(content::ExecuteScriptAndExtractInt(
+ contents,
"window.domAutomationController.send("
" window.performance.timing.responseEnd - "
" window.performance.timing.navigationStart)",
« no previous file with comments | « chrome/browser/loadtimes_extension_bindings_browsertest.cc ('k') | chrome/browser/notifications/notification_browsertest.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698