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

Unified Diff: chrome/test/data/prerender/prerender_page_pending.html

Issue 142013004: Re-enable prerender RemovingLink browser tests. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: New test, and machinery to support said test. Created 6 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: chrome/test/data/prerender/prerender_page_pending.html
diff --git a/chrome/test/data/prerender/prerender_page_pending.html b/chrome/test/data/prerender/prerender_page_pending.html
index 177a8ea82ce6a4f1b5bc5ee44bbfc4def31c618e..e3253990988b2edbd3dc02fcffa38aa8a6f033ad 100644
--- a/chrome/test/data/prerender/prerender_page_pending.html
+++ b/chrome/test/data/prerender/prerender_page_pending.html
@@ -14,7 +14,7 @@ var pageWasPrerendered = false;
function DidPrerenderPass() {
pageWasPrerendered = true;
- return !receivedPrerenderStartEvents[0] && !receivedPrerenderStopEvents[0];
+ return !GetPrerenderEventCount(0, 'webkitprerenderstart');
mmenke 2014/02/13 16:34:33 optional: I think it may make more sense to check
davidben 2014/02/14 01:01:13 Hrm. DidDisplayPass gets called before the pending
}
// Make sure DidPrerenderPass() was called first. Otherwise, the page was

Powered by Google App Engine
This is Rietveld 408576698