 Chromium Code Reviews
 Chromium Code Reviews Issue 14949012:
  Add instrumentation to see how often a local predictor prerender could be  (Closed) 
  Base URL: svn://svn.chromium.org/chrome/trunk/src/
    
  
    Issue 14949012:
  Add instrumentation to see how often a local predictor prerender could be  (Closed) 
  Base URL: svn://svn.chromium.org/chrome/trunk/src/| Index: chrome/browser/prerender/prerender_local_predictor.h | 
| =================================================================== | 
| --- chrome/browser/prerender/prerender_local_predictor.h (revision 199567) | 
| +++ chrome/browser/prerender/prerender_local_predictor.h (working copy) | 
| @@ -18,6 +18,7 @@ | 
| namespace content { | 
| class SessionStorageNamespace; | 
| +class WebContents; | 
| } | 
| namespace gfx { | 
| @@ -84,6 +85,9 @@ | 
| EVENT_ISSUING_PRERENDER = 43, | 
| EVENT_NO_PRERENDER_CANDIDATES = 44, | 
| EVENT_GOT_HISTORY_ISSUING_LOOKUP = 45, | 
| + EVENT_TAB_HELPER_URL_SEEN = 46, | 
| + EVENT_TAB_HELPER_URL_SEEN_MATCH = 47, | 
| + EVENT_TAB_HELPER_URL_SEEN_NAMESPACE_MATCH = 48, | 
| EVENT_MAX_VALUE | 
| }; | 
| @@ -103,6 +107,8 @@ | 
| void OnPLTEventForURL(const GURL& url, base::TimeDelta page_load_time); | 
| + void OnTabHelperURLSeen(const GURL& url, content::WebContents* web_contents); | 
| + | 
| private: | 
| struct PrerenderProperties; | 
| HistoryService* GetHistoryIfExists() const; | 
| @@ -155,6 +161,8 @@ | 
| base::WeakPtrFactory<PrerenderLocalPredictor> weak_factory_; | 
| + bool current_prerender_would_have_matched_; | 
| 
Shishir
2013/05/14 18:19:34
Comment on var - and the fact that it is only used
 
tburkard
2013/05/14 19:28:06
Done.
 | 
| + | 
| DISALLOW_COPY_AND_ASSIGN(PrerenderLocalPredictor); | 
| }; |