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

Unified Diff: tools/chrome_proxy/webdriver/simple_smoke.py

Issue 2554383002: Add timeouts to LoadPage and ExecJavascipt. (Closed)
Patch Set: Fix nit. Created 4 years 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
« no previous file with comments | « tools/chrome_proxy/webdriver/common.py ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: tools/chrome_proxy/webdriver/simple_smoke.py
diff --git a/tools/chrome_proxy/webdriver/simple_smoke.py b/tools/chrome_proxy/webdriver/simple_smoke.py
index f1e537e2760ebffad365c35965733b4b688d3fb3..f70cbbb8a15c6a50f51705616cbb71d3a10154ef 100644
--- a/tools/chrome_proxy/webdriver/simple_smoke.py
+++ b/tools/chrome_proxy/webdriver/simple_smoke.py
@@ -16,7 +16,8 @@ class SimpleSmoke(IntegrationTest):
t.AddChromeArgs(['--enable-spdy-proxy-auth'])
t.SetURL('http://check.googlezip.net/test.html')
t.LoadPage()
- print 'Document Title: ', t.ExecuteJavascript('document.title')
+ print 'Document Title: ', t.ExecuteJavascriptStatement('document.title',
+ timeout=1)
time.sleep(5)
responses = t.GetHTTPResponses()
for response in responses:
« no previous file with comments | « tools/chrome_proxy/webdriver/common.py ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698