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

Side by Side Diff: components/sync_driver/device_count_metrics_provider.h

Issue 2203673002: [Sync] Move //components/sync_driver to //components/sync/driver. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@sd-a
Patch Set: Full change rebased on static lib. Created 4 years, 4 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 COMPONENTS_SYNC_DRIVER_DEVICE_COUNT_METRICS_PROVIDER_H_
6 #define COMPONENTS_SYNC_DRIVER_DEVICE_COUNT_METRICS_PROVIDER_H_
7
8 #include <memory>
9 #include <vector>
10
11 #include "base/callback.h"
12 #include "base/macros.h"
13 #include "components/metrics/metrics_provider.h"
14
15 namespace sync_driver {
16
17 class DeviceInfoTracker;
18
19 // A registerable metrics provider that will emit the number of active syncing
20 // devices a user has upon UMA upload. When there are multiple active profiles
21 // that are aware of syncable devices, the largest count is used. This approach
22 // is useful for several reasons. We're significantly interested in "power"
23 // users that have multiple devices and profiles, and if we skipped emitting any
24 // metrics in multi profile cases, then we'd be unaware of this entire segment
25 // of the population. If we emitted multiple metrics, we would have much more
26 // noise and discerning which metrics actually belonged to non-syncing users
27 // would be much trickier.
28 class DeviceCountMetricsProvider : public metrics::MetricsProvider {
29 public:
30 typedef base::Callback<void(
31 std::vector<const sync_driver::DeviceInfoTracker*>* trackers)>
32 ProvideTrackersCallback;
33
34 explicit DeviceCountMetricsProvider(
35 const ProvideTrackersCallback& provide_trackers);
36 ~DeviceCountMetricsProvider() override;
37
38 // MetricsProvider:
39 void ProvideGeneralMetrics(
40 metrics::ChromeUserMetricsExtension* uma_proto) override;
41
42 private:
43 // Returns the max number of active devices across all accounts.
44 int MaxActiveDeviceCount() const;
45
46 const ProvideTrackersCallback provide_trackers_;
47
48 DISALLOW_COPY_AND_ASSIGN(DeviceCountMetricsProvider);
49 };
50
51 } // namespace sync_driver
52
53 #endif // COMPONENTS_SYNC_DRIVER_DEVICE_COUNT_METRICS_PROVIDER_H_
OLDNEW
« no previous file with comments | « components/sync_driver/data_type_status_table.cc ('k') | components/sync_driver/device_count_metrics_provider.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698