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

Side by Side Diff: components/page_load_metrics/renderer/metrics_render_frame_observer.cc

Issue 1312213010: PageLoadMetrics renderer and browser implementation. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Nasko fixes Created 5 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
(Empty)
1 // Copyright 2015 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file.
4
5 #include "components/page_load_metrics/renderer/metrics_render_frame_observer.h"
6
7 #include "base/time/time.h"
8 #include "base/timer/timer.h"
9 #include "components/page_load_metrics/renderer/page_timing_metrics_sender.h"
10 #include "content/public/renderer/render_frame.h"
11 #include "third_party/WebKit/public/platform/WebURLResponse.h"
12 #include "third_party/WebKit/public/web/WebDataSource.h"
13 #include "third_party/WebKit/public/web/WebDocument.h"
14 #include "third_party/WebKit/public/web/WebLocalFrame.h"
15 #include "third_party/WebKit/public/web/WebPerformance.h"
16 #include "url/gurl.h"
17
18 namespace page_load_metrics {
19
20 MetricsRenderFrameObserver::MetricsRenderFrameObserver(
21 content::RenderFrame* render_frame)
22 : content::RenderFrameObserver(render_frame) {}
23
24 MetricsRenderFrameObserver::~MetricsRenderFrameObserver() {}
25
26 void MetricsRenderFrameObserver::DidChangePerformanceTiming() {
27 SendMetrics();
28 }
29
30 void MetricsRenderFrameObserver::DidCommitProvisionalLoad(
31 bool is_new_navigation,
32 bool is_same_page_navigation) {
33 // Same-page navigations (e.g. an in-document navigation from a fragment
34 // link) aren't full page loads, since they don't go to network to load the
35 // main HTML resource. DidStartProvisionalLoad doesn't get invoked for same
36 // page navigations, so we may still have an active
37 // page_timing_metrics_sender_ at this point.
38 if (is_same_page_navigation)
39 return;
40
41 // Make sure to release the sender for a previous navigation, if we have one.
42 page_timing_metrics_sender_.reset();
43
44 // We only create a PageTimingMetricsSender if the page meets the criteria for
45 // sending and recording metrics. Once page_timing_metrics_sender_ is
46 // non-null, we will send metrics for the current page at some later time, as
47 // those metrics become available.
48 if (ShouldSendMetrics()) {
49 page_timing_metrics_sender_.reset(
50 new PageTimingMetricsSender(this, routing_id(), CreateTimer()));
51 }
52 }
53
54 void MetricsRenderFrameObserver::SendMetrics() {
55 if (!page_timing_metrics_sender_)
56 return;
57
58 PageLoadTiming timing(GetTiming());
59 page_timing_metrics_sender_->Send(timing);
60 }
61
62 bool MetricsRenderFrameObserver::ShouldSendMetrics() const {
63 const blink::WebLocalFrame* frame = render_frame()->GetWebFrame();
64 // We only generate historgrams for main frames.
65 if (frame->parent())
66 return false;
67
68 const blink::WebDocument& document = frame->document();
69 // Ignore non-HTTP schemes (e.g. chrome://).
70 const GURL& url = document.url();
71 if (!url.SchemeIsHTTPOrHTTPS())
72 return false;
73
74 const blink::WebURLResponse& url_response = frame->dataSource()->response();
75 // Ignore multipart responses (e.g. MHTML).
76 if (url_response.isMultipartPayload())
77 return false;
78
79 // Ignore non-HTML documents (e.g. SVG). Note that images are treated by
80 // Blink as HTML documents, so to exclude images, we must perform
81 // additional mime type checking below.
82 if (!document.isHTMLDocument() && !document.isXHTMLDocument())
83 return false;
84
85 // Ignore non-HTML mime types (e.g. images).
86 std::string mime_type = url_response.mimeType().utf8();
87 if (mime_type != "text/html" && mime_type != "application/xhtml+xml")
88 return false;
89
90 return true;
91 }
92
93 PageLoadTiming MetricsRenderFrameObserver::GetTiming() const {
94 blink::WebLocalFrame* frame = render_frame()->GetWebFrame();
95 return PageLoadTiming(frame->performance());
nasko 2015/09/17 20:58:43 nit: return PageLoadTiming(render_frame()->GetWebF
Charlie Harrison 2015/09/17 21:57:27 Done.
96 }
97
98 scoped_ptr<base::Timer> MetricsRenderFrameObserver::CreateTimer() const {
99 return scoped_ptr<base::Timer>(
100 new base::OneShotTimer<PageTimingMetricsSender>());
101 }
102
103 } // namespace page_load_metrics
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698