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

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

Issue 2697323004: Adds UMA metrics for renderer uptime (Closed)
Patch Set: Removed metric provider 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 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_tracker.h"
6
7 #include "base/metrics/histogram_macros.h"
8 #include "content/public/browser/notification_service.h"
9 #include "content/public/browser/notification_types.h"
10 #include "content/public/browser/render_process_host.h"
11 #include "extensions/features/features.h"
12
13 #if BUILDFLAG(ENABLE_EXTENSIONS)
14 #include "extensions/browser/process_map.h"
15 #endif
16
17 namespace {
18
19 RendererUptimeTracker* g_instance = nullptr;
20
21 } // namespace
22
23 // static
24 void RendererUptimeTracker::Initialize() {
25 DCHECK(!g_instance);
26 g_instance = new RendererUptimeTracker;
27 }
28
29 // static
30 bool RendererUptimeTracker::IsInitialized() {
31 return g_instance != nullptr;
32 }
33
34 // static
35 RendererUptimeTracker* RendererUptimeTracker::Get() {
36 DCHECK(g_instance);
37 return g_instance;
38 }
39
40 RendererUptimeTracker::RendererUptimeTracker() {
41 registrar_.Add(this, content::NOTIFICATION_RENDERER_PROCESS_CREATED,
42 content::NotificationService::AllBrowserContextsAndSources());
43 }
44
45 RendererUptimeTracker::~RendererUptimeTracker() {}
46
47 void RendererUptimeTracker::OnRendererStarted(int pid) {
48 info_map_[pid] = RendererInfo{base::TimeTicks::Now(), 0, 0};
49 }
50
51 void RendererUptimeTracker::OnRendererTerminated(int pid) {
52 auto it = info_map_.find(pid);
53 if (it != info_map_.end()) {
54 auto uptime = base::TimeTicks::Now() - it->second.launched_at_;
55 UMA_HISTOGRAM_CUSTOM_TIMES("Memory.Experimental.Renderer.Uptime", uptime,
56 base::TimeDelta::FromHours(1),
57 base::TimeDelta::FromDays(7), 50);
58 UMA_HISTOGRAM_COUNTS_10000(
59 "Memory.Experimental.Renderer.LoadsInMainFrameDuringUptime",
60 it->second.num_loads_in_main_frame_);
61 info_map_.erase(it);
62 }
63 }
64
65 void RendererUptimeTracker::OnLoadInMainFrame(int pid) {
66 auto it = info_map_.find(pid);
67 DCHECK(it != info_map_.end());
68 it->second.num_loads_in_main_frame_++;
69 }
70
71 void RendererUptimeTracker::Observe(
72 int type,
73 const content::NotificationSource& source,
74 const content::NotificationDetails& details) {
75 switch (type) {
76 case content::NOTIFICATION_RENDERER_PROCESS_CREATED: {
77 content::RenderProcessHost* host =
78 content::Source<content::RenderProcessHost>(source).ptr();
79 #if BUILDFLAG(ENABLE_EXTENSIONS)
80 if (extensions::ProcessMap::Get(host->GetBrowserContext())
81 ->Contains(host->GetID())) {
82 break;
83 }
84 #endif
85 OnRendererStarted(host->GetID());
86 break;
87 }
88
89 case content::NOTIFICATION_RENDERER_PROCESS_TERMINATED: {
Alexei Svitkine (slow) 2017/03/01 15:50:18 Are you forgetting to a register for this?
keishi 2017/03/02 09:37:14 Done.
90 content::RenderProcessHost* host =
91 content::Source<content::RenderProcessHost>(source).ptr();
92 #if BUILDFLAG(ENABLE_EXTENSIONS)
93 if (extensions::ProcessMap::Get(host->GetBrowserContext())
94 ->Contains(host->GetID())) {
95 break;
96 }
97 #endif
98 OnRendererTerminated(host->GetID());
99 break;
100 }
101
102 default:
103 NOTREACHED();
104 break;
105 }
106 }
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698