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

Side by Side Diff: chrome/browser/metrics/subprocess_metrics_provider.h

Issue 1671933002: Create and pass shared-histogram-allocator from browser to renderer. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@hsm-merge
Patch Set: addressd review comments by Alexei Created 4 years, 8 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 2016 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 #ifndef CHROME_BROWSER_METRICS_SUBPROCESS_METRICS_PROVIDER_H_
6 #define CHROME_BROWSER_METRICS_SUBPROCESS_METRICS_PROVIDER_H_
7
8 #include <set>
9
10 #include "base/gtest_prod_util.h"
11 #include "base/id_map.h"
12 #include "base/memory/scoped_ptr.h"
13 #include "base/scoped_observer.h"
14 #include "base/threading/thread_checker.h"
15 #include "components/metrics/metrics_provider.h"
16 #include "content/public/browser/notification_observer.h"
17 #include "content/public/browser/notification_registrar.h"
18 #include "content/public/browser/render_process_host_observer.h"
19
20 namespace base {
21 class PersistentHistogramAllocator;
22 class SharedPersistentMemoryAllocator;
23 }
24
25 // SubprocessMetricsProvider gathers and logs histograms stored in shared
26 // memory segments between processes.
27 class SubprocessMetricsProvider : public metrics::MetricsProvider,
28 public content::NotificationObserver,
29 public content::RenderProcessHostObserver {
30 public:
31 SubprocessMetricsProvider();
32 ~SubprocessMetricsProvider() override;
33
34 private:
35 friend class SubprocessMetricsProviderTest;
36 FRIEND_TEST_ALL_PREFIXES(SubprocessMetricsProviderTest, SnapshotMetrics);
37
38 // Indicates subprocess to be monitored with unique id for later reference.
39 // Metrics reporting will read histograms from it and upload them to UMA.
40 void RegisterSubprocessAllocator(
41 int id,
42 scoped_ptr<base::PersistentHistogramAllocator> allocator);
43
44 // Indicates that a subprocess has exited and is thus finished with the
45 // allocator it was using.
46 void DeregisterSubprocessAllocator(int id);
47
48 // Report all histograms of a given allocator to the snapshot-manager.
49 void RecordHistogramSnapshotsFromAllocator(
50 base::HistogramSnapshotManager* snapshot_manager,
51 int id,
52 base::PersistentHistogramAllocator* allocator);
53
54 // metrics::MetricsDataProvider:
55 void RecordHistogramSnapshots(
56 base::HistogramSnapshotManager* snapshot_manager) override;
57
58 // content::NotificationObserver:
59 void Observe(int type,
60 const content::NotificationSource& source,
61 const content::NotificationDetails& details) override;
62
63 // content::RenderProcessHostObserver:
64 void RenderProcessReady(content::RenderProcessHost* host) override;
65 void RenderProcessHostDestroyed(content::RenderProcessHost* host) override;
66
67 base::ThreadChecker thread_checker_;
68
69 // Object for registing notification requests.
70 content::NotificationRegistrar registrar_;
71
72 // All of the shared-persistent-allocators for known sub-processes.
73 using AllocatorByIdMap =
74 IDMap<base::PersistentHistogramAllocator, IDMapOwnPointer, int>;
75 AllocatorByIdMap allocators_by_id_;
76
77 // Allocators that are no longer attached to a subprocess, to be released
78 // once the last data contained therein has been reported.
79 std::vector<scoped_ptr<base::PersistentHistogramAllocator>>
80 allocators_to_release_;
81
82 // Track all observed render processes to un-observe them on exit.
83 ScopedObserver<content::RenderProcessHost, SubprocessMetricsProvider>
84 scoped_observer_;
85
86 DISALLOW_COPY_AND_ASSIGN(SubprocessMetricsProvider);
87 };
88
89 #endif // CHROME_BROWSER_METRICS_SUBPROCESS_METRICS_PROVIDER_H_
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698