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

Unified Diff: chrome/renderer/chrome_content_renderer_client.cc

Issue 2737563007: Add support for tracking loading behavior of child frames. (Closed)
Patch Set: test fixes Created 3 years, 9 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/renderer/chrome_content_renderer_client.cc
diff --git a/chrome/renderer/chrome_content_renderer_client.cc b/chrome/renderer/chrome_content_renderer_client.cc
index 265c073baec8efcc99bdea89801db67cc32de389..1b96488ad0536b9b1ac4f233a1846360a9d13027 100644
--- a/chrome/renderer/chrome_content_renderer_client.cc
+++ b/chrome/renderer/chrome_content_renderer_client.cc
@@ -519,17 +519,14 @@ void ChromeContentRendererClient::RenderFrameCreated(
new NetErrorHelper(render_frame);
- if (render_frame->IsMainFrame()) {
- // Only attach MetricsRenderFrameObserver to the main frame, since
- // we only want to log page load metrics for the main frame.
- new page_load_metrics::MetricsRenderFrameObserver(render_frame);
- } else {
+ new page_load_metrics::MetricsRenderFrameObserver(render_frame);
+
+ if (!render_frame->IsMainFrame() &&
+ prerender::PrerenderHelper::IsPrerendering(
+ render_frame->GetRenderView()->GetMainRenderFrame())) {
// Avoid any race conditions from having the browser tell subframes that
// they're prerendering.
- if (prerender::PrerenderHelper::IsPrerendering(
- render_frame->GetRenderView()->GetMainRenderFrame())) {
- new prerender::PrerenderHelper(render_frame);
- }
+ new prerender::PrerenderHelper(render_frame);
}
// Set up a mojo service to test if this page is a distiller page.

Powered by Google App Engine
This is Rietveld 408576698