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

Side by Side Diff: chrome/browser/prerender/prerender_tab_helper.cc

Issue 580133002: Update entry page type to include error pages when appropriate. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Compile fix for unit_tests Created 6 years, 3 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 unified diff | Download patch
OLDNEW
1 // Copyright (c) 2012 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #include "chrome/browser/prerender/prerender_tab_helper.h" 5 #include "chrome/browser/prerender/prerender_tab_helper.h"
6 6
7 #include "base/bind.h" 7 #include "base/bind.h"
8 #include "base/metrics/histogram.h" 8 #include "base/metrics/histogram.h"
9 #include "base/time/time.h" 9 #include "base/time/time.h"
10 #include "chrome/browser/prerender/prerender_histograms.h" 10 #include "chrome/browser/prerender/prerender_histograms.h"
(...skipping 81 matching lines...) Expand 10 before | Expand all | Expand 10 after
92 return; 92 return;
93 if (prerender_manager->IsWebContentsPrerendering(web_contents(), NULL)) 93 if (prerender_manager->IsWebContentsPrerendering(web_contents(), NULL))
94 return; 94 return;
95 ReportTabHelperURLSeenToLocalPredictor(prerender_manager, url, 95 ReportTabHelperURLSeenToLocalPredictor(prerender_manager, url,
96 web_contents()); 96 web_contents());
97 } 97 }
98 98
99 void PrerenderTabHelper::DidCommitProvisionalLoadForFrame( 99 void PrerenderTabHelper::DidCommitProvisionalLoadForFrame(
100 content::RenderFrameHost* render_frame_host, 100 content::RenderFrameHost* render_frame_host,
101 const GURL& validated_url, 101 const GURL& validated_url,
102 bool url_is_unreachable,
102 ui::PageTransition transition_type) { 103 ui::PageTransition transition_type) {
103 if (render_frame_host->GetParent()) 104 if (render_frame_host->GetParent())
104 return; 105 return;
105 RecordEvent(EVENT_MAINFRAME_COMMIT); 106 RecordEvent(EVENT_MAINFRAME_COMMIT);
106 RecordEventIfLoggedInURL(EVENT_MAINFRAME_COMMIT_DOMAIN_LOGGED_IN, 107 RecordEventIfLoggedInURL(EVENT_MAINFRAME_COMMIT_DOMAIN_LOGGED_IN,
107 validated_url); 108 validated_url);
108 url_ = validated_url; 109 url_ = validated_url;
109 PrerenderManager* prerender_manager = MaybeGetPrerenderManager(); 110 PrerenderManager* prerender_manager = MaybeGetPrerenderManager();
110 if (!prerender_manager) 111 if (!prerender_manager)
111 return; 112 return;
(...skipping 155 matching lines...) Expand 10 before | Expand all | Expand 10 after
267 prerender_manager->RecordPerceivedPageLoadTime( 268 prerender_manager->RecordPerceivedPageLoadTime(
268 origin_, navigation_type_, perceived_page_load_time, 269 origin_, navigation_type_, perceived_page_load_time,
269 fraction_plt_elapsed_at_swap_in, url_); 270 fraction_plt_elapsed_at_swap_in, url_);
270 271
271 // Reset state for the next navigation. 272 // Reset state for the next navigation.
272 navigation_type_ = NAVIGATION_TYPE_NORMAL; 273 navigation_type_ = NAVIGATION_TYPE_NORMAL;
273 origin_ = ORIGIN_NONE; 274 origin_ = ORIGIN_NONE;
274 } 275 }
275 276
276 } // namespace prerender 277 } // namespace prerender
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698