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

Unified Diff: tools/chrome_proxy/integration_tests/chrome_proxy_pagesets/block_once.py

Issue 2672803002: [Telemetry refactor] Migrate clients to new JavaScript API (batch 3) (Closed)
Patch Set: add comment on tools/android Created 3 years, 10 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: tools/chrome_proxy/integration_tests/chrome_proxy_pagesets/block_once.py
diff --git a/tools/chrome_proxy/integration_tests/chrome_proxy_pagesets/block_once.py b/tools/chrome_proxy/integration_tests/chrome_proxy_pagesets/block_once.py
index 30def6634575f1f5f8837f5c8f7673fea26a7597..977b08f03ebbb276c8d0b2f37e7082d7ffd91a11 100644
--- a/tools/chrome_proxy/integration_tests/chrome_proxy_pagesets/block_once.py
+++ b/tools/chrome_proxy/integration_tests/chrome_proxy_pagesets/block_once.py
@@ -17,7 +17,7 @@ class BlockOncePage(page_module.Page):
super(BlockOncePage, self).RunNavigateSteps(action_runner)
# Test block-once on a POST request.
# Ensure that a subsequent request uses the data reduction proxy.
- action_runner.ExecuteJavaScript('''
+ action_runner.ExecuteJavaScript2('''
(function() {
window.post_request_completed = false;
var request = new XMLHttpRequest();
@@ -35,8 +35,8 @@ class BlockOncePage(page_module.Page):
request.send();
})();
''')
- action_runner.WaitForJavaScriptCondition(
- "window.post_request_completed == true", timeout_in_seconds=30)
+ action_runner.WaitForJavaScriptCondition2(
+ "window.post_request_completed == true", timeout=30)
class BlockOnceStorySet(story.StorySet):

Powered by Google App Engine
This is Rietveld 408576698