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

Side by Side Diff: chrome/browser/metrics/renderer_uptime_web_contents_observer.cc

Issue 2697323004: Adds UMA metrics for renderer uptime (Closed)
Patch Set: fix Created 3 years, 10 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 2017 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 "chrome/browser/metrics/renderer_uptime_web_contents_observer.h"
6
7 #include "chrome/browser/metrics/renderer_uptime_metrics_provider.h"
8 #include "chrome/browser/metrics/renderer_uptime_tracker.h"
9 #include "content/public/browser/render_process_host.h"
10
11 DEFINE_WEB_CONTENTS_USER_DATA_KEY(metrics::RendererUptimeWebContentsObserver);
12
13 namespace metrics {
14
15 RendererUptimeWebContentsObserver::RendererUptimeWebContentsObserver(
16 content::WebContents* web_contents)
17 : content::WebContentsObserver(web_contents) {}
18
19 // static
20 RendererUptimeWebContentsObserver*
21 RendererUptimeWebContentsObserver::CreateForWebContents(
22 content::WebContents* web_contents) {
23 DCHECK(web_contents);
24
25 if (!::RendererUptimeTracker::IsInitialized())
Alexei Svitkine (slow) 2017/02/28 16:13:12 Nit: No :: Remove from later in the file as well.
keishi 2017/03/01 08:09:54 Done.
26 return nullptr;
27
28 RendererUptimeWebContentsObserver* observer = FromWebContents(web_contents);
29 if (!observer) {
30 observer = new RendererUptimeWebContentsObserver(web_contents);
31 web_contents->SetUserData(UserDataKey(), observer);
32 }
33 return observer;
34 }
35
36 void RendererUptimeWebContentsObserver::DocumentAvailableInMainFrame() {
37 if (!::RendererUptimeTracker::IsInitialized())
38 return;
39 ::RendererUptimeTracker::Get()->OnLoadInMainFrame(
40 web_contents()->GetRenderProcessHost()->GetID());
41 }
42
43 } // namespace metrics
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698