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

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

Issue 7835004: Moved the following IPCs out of chrome into content where they are handled by (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src/
Patch Set: '' Created 9 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 side-by-side diff with in-line comments
Download patch
« no previous file with comments | « chrome/browser/prerender/prerender_tab_helper.h ('k') | chrome/browser/ui/browser.h » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/browser/prerender/prerender_tab_helper.cc
===================================================================
--- chrome/browser/prerender/prerender_tab_helper.cc (revision 99765)
+++ chrome/browser/prerender/prerender_tab_helper.cc (working copy)
@@ -192,36 +192,7 @@
MaybeUsePrerenderedPage(url, has_opener_set);
}
-bool PrerenderTabHelper::OnMessageReceived(const IPC::Message& message) {
- IPC_BEGIN_MESSAGE_MAP(PrerenderTabHelper, message)
- IPC_MESSAGE_HANDLER(ViewHostMsg_DidStartProvisionalLoadForFrame,
- OnDidStartProvisionalLoadForFrame)
- IPC_MESSAGE_HANDLER(ViewHostMsg_UpdateTargetURL, OnMsgUpdateTargetURL)
- IPC_END_MESSAGE_MAP()
- return false;
-}
-
-void PrerenderTabHelper::OnDidStartProvisionalLoadForFrame(int64 frame_id,
- bool is_main_frame,
- bool has_opener_set,
- const GURL& url) {
- if (is_main_frame) {
- RecordPageviewEvent(PAGEVIEW_EVENT_LOAD_START);
- if (IsTopSite(url))
- RecordPageviewEvent(PAGEVIEW_EVENT_TOP_SITE_LOAD_START);
-
- // Record the beginning of a new PPLT navigation.
- pplt_load_start_ = base::TimeTicks::Now();
-
- // Update hover stats.
- for (int i = 0; i < kNumHoverThresholds; i++)
- last_hovers_[i].RecordNavigation(url);
-
- MaybeLogCurrentHover(current_hover_url_ == url);
- }
-}
-
-void PrerenderTabHelper::OnMsgUpdateTargetURL(int32 page_id, const GURL& url) {
+void PrerenderTabHelper::UpdateTargetURL(int32 page_id, const GURL& url) {
for (int i = 0; i < kNumHoverThresholds; i++)
last_hovers_[i].RecordHover(url);
@@ -246,6 +217,28 @@
pplt_load_start_ = base::TimeTicks();
}
+void PrerenderTabHelper::DidStartProvisionalLoadForFrame(
+ int64 frame_id,
+ bool is_main_frame,
+ const GURL& validated_url,
+ bool is_error_page,
+ RenderViewHost* render_view_host) {
+ if (is_main_frame) {
+ RecordPageviewEvent(PAGEVIEW_EVENT_LOAD_START);
+ if (IsTopSite(validated_url))
+ RecordPageviewEvent(PAGEVIEW_EVENT_TOP_SITE_LOAD_START);
+
+ // Record the beginning of a new PPLT navigation.
+ pplt_load_start_ = base::TimeTicks::Now();
+
+ // Update hover stats.
+ for (int i = 0; i < kNumHoverThresholds; i++)
+ last_hovers_[i].RecordNavigation(validated_url);
+
+ MaybeLogCurrentHover(current_hover_url_ == validated_url);
+ }
+}
+
PrerenderManager* PrerenderTabHelper::MaybeGetPrerenderManager() const {
Profile* profile =
Profile::FromBrowserContext(tab_contents()->browser_context());
« no previous file with comments | « chrome/browser/prerender/prerender_tab_helper.h ('k') | chrome/browser/ui/browser.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698