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

Unified Diff: chrome/browser/prerender/prerender_contents.cc

Issue 8503040: Prerendering: Add MatchComplete PPLT (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src/
Patch Set: '' Created 9 years, 1 month 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/prerender/prerender_contents.cc
===================================================================
--- chrome/browser/prerender/prerender_contents.cc (revision 109293)
+++ chrome/browser/prerender/prerender_contents.cc (working copy)
@@ -349,11 +349,13 @@
void PrerenderContents::set_final_status(FinalStatus final_status) {
DCHECK(final_status >= FINAL_STATUS_USED && final_status < FINAL_STATUS_MAX);
DCHECK(final_status_ == FINAL_STATUS_MAX ||
- final_status_ == FINAL_STATUS_CONTROL_GROUP);
+ final_status_ == FINAL_STATUS_CONTROL_GROUP ||
+ final_status_ == FINAL_STATUS_MATCH_COMPLETE_DUMMY);
// Don't override final_status_ if it's FINAL_STATUS_CONTROL_GROUP,
cbentzel 2011/11/10 13:16:25 Update comment.
tburkard 2011/11/10 18:36:55 Done.
// otherwise data will be collected in the Prerender.FinalStatus histogram.
- if (final_status_ == FINAL_STATUS_CONTROL_GROUP)
+ if (final_status_ == FINAL_STATUS_CONTROL_GROUP ||
+ final_status_ == FINAL_STATUS_MATCH_COMPLETE_DUMMY)
return;
final_status_ = final_status;
@@ -579,7 +581,7 @@
prerendering_has_been_cancelled_ = true;
// This has to be done after setting the final status, as it adds the
// prerender to the history.
- prerender_manager_->MoveEntryToPendingDelete(this);
+ prerender_manager_->MoveEntryToPendingDelete(this, final_status);
// We may destroy the PrerenderContents before we have initialized the
// RenderViewHost. Otherwise set the Observer's PrerenderContents to NULL to

Powered by Google App Engine
This is Rietveld 408576698