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

Unified Diff: chrome/browser/metrics/extensions_metrics_provider.h

Issue 464463003: Adding logging of offstore extensions state to user metrics. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 6 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 side-by-side diff with in-line comments
Download patch
Index: chrome/browser/metrics/extensions_metrics_provider.h
diff --git a/chrome/browser/metrics/extensions_metrics_provider.h b/chrome/browser/metrics/extensions_metrics_provider.h
index 6cca94dc30a5da87b50815a8fe7f554538573297..6da4ff53ea891ef4187b5ac1d1b6838b3a097fa0 100644
--- a/chrome/browser/metrics/extensions_metrics_provider.h
+++ b/chrome/browser/metrics/extensions_metrics_provider.h
@@ -33,10 +33,6 @@ class ExtensionsMetricsProvider : public metrics::MetricsProvider {
metrics::MetricsStateManager* metrics_state_manager);
virtual ~ExtensionsMetricsProvider();
- // metrics::MetricsProvider:
-
- // Writes the hashed list of installed extensions into the specified
- // SystemProfileProto object.
virtual void ProvideSystemProfileMetrics(
metrics::SystemProfileProto* system_profile) OVERRIDE;
@@ -47,7 +43,8 @@ class ExtensionsMetricsProvider : public metrics::MetricsProvider {
// TODO(mvrable): If metrics are ever converted to being per-profile, then
// this should be updated to return extensions installed in a specified
// profile.
- virtual scoped_ptr<extensions::ExtensionSet> GetInstalledExtensions();
+ virtual scoped_ptr<extensions::ExtensionSet> GetInstalledExtensions(
+ Profile* profile);
// Retrieves the client ID.
virtual uint64 GetClientID();
@@ -63,12 +60,19 @@ class ExtensionsMetricsProvider : public metrics::MetricsProvider {
// same value so that reported extensions are consistent.
Profile* GetMetricsProfile();
+ // Writes whether any loaded profiles have extensions not from the webstore.
+ void ProvideOffStoreMetric(metrics::SystemProfileProto* system_profile);
+
+ // Writes the hashed list of installed extensions into the specified
+ // SystemProfileProto object.
+ void ProvideOccupiedBucketMetric(metrics::SystemProfileProto* system_profile);
+
// The MetricsStateManager from which the client ID is obtained.
metrics::MetricsStateManager* metrics_state_manager_;
- // The profile for which extensions are gathered. Once a profile is found
- // its value is cached here so that GetMetricsProfile() can return a
- // consistent value.
+ // The profile for which extensions are gathered for the occupied bucket
+ // metric. Once a profile is found its value is cached here so that
+ // GetMetricsProfile() can return a consistent value.
Profile* cached_profile_;
DISALLOW_COPY_AND_ASSIGN(ExtensionsMetricsProvider);

Powered by Google App Engine
This is Rietveld 408576698