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

Unified Diff: chrome/browser/prerender/prerender_manager.h

Issue 9270018: Make a separate histogram for MatchComplete Final Status'es and (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src/
Patch Set: Created 8 years, 11 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/prerender/prerender_manager.h
===================================================================
--- chrome/browser/prerender/prerender_manager.h (revision 118385)
+++ chrome/browser/prerender/prerender_manager.h (working copy)
@@ -205,6 +205,15 @@
uint8 experiment_id,
FinalStatus final_status) const;
+ // Record a final status of a prerendered page in a histogram.
+ // This variation allows specifying whether prerendering had been started
+ // (necessary to flag MatchComplete dummies).
+ void RecordFinalStatusWithMatchCompleteStatus(
+ Origin origin,
+ uint8 experiment_id,
+ PrerenderContents::MatchCompleteStatus mc_status,
+ FinalStatus final_status) const;
+
const Config& config() const { return config_; }
Config& mutable_config() { return config_; }
@@ -243,6 +252,7 @@
FRIEND_TEST_ALL_PREFIXES(PrerenderManagerTest, FragmentMatchesPageTest);
FRIEND_TEST_ALL_PREFIXES(PrerenderManagerTest, PageMatchesFragmentTest);
FRIEND_TEST_ALL_PREFIXES(PrerenderManagerTest, PendingPrerenderTest);
+ FRIEND_TEST_ALL_PREFIXES(PrerenderManagerTest, ControlGroupVisited);
FRIEND_TEST_ALL_PREFIXES(PrerenderManagerTest, RateLimitInWindowTest);
FRIEND_TEST_ALL_PREFIXES(PrerenderManagerTest, RateLimitOutsideWindowTest);
FRIEND_TEST_ALL_PREFIXES(PrerenderManagerTest, SourceRenderViewClosed);

Powered by Google App Engine
This is Rietveld 408576698