Adding logging of offstore extensions state to user metrics.

components/metrics/proto/system_profile.proto

Index: components/metrics/proto/system_profile.proto
diff --git a/components/metrics/proto/system_profile.proto b/components/metrics/proto/system_profile.proto
index 11453333049aaf7285be8f9bf017cc11220f32f8..1d90254f3519534b2a17e18da6c437e02373d620 100644
--- a/components/metrics/proto/system_profile.proto
+++ b/components/metrics/proto/system_profile.proto
@@ -11,7 +11,7 @@ option optimize_for = LITE_RUNTIME;
 
 package metrics;
 
-// Next tag: 19
+// Next tag: 20
 message SystemProfileProto {
   // The time when the client was compiled/linked, in seconds since the epoch.
   optional int64 build_timestamp = 1;
@@ -507,4 +507,17 @@ message SystemProfileProto {
   // extensions.  If multiple extensions map to the same bucket, that bucket is
   // still only reported once.
   repeated int32 occupied_extension_bucket = 18;
+
+  // The state of loaded extensions for this system. The system can have either
+  // no applicable extensions, extensions only from the webstore and verified by
+  // the webstore, extensions only from the webstore but not verified, or
+  // extensions not from the store. If there is a single off-store extension,
+  // then HAS_OFFSTORE is reported.
+  enum ExtensionsState {

[Devlin, 2014/08/13 18:18:30]

Make a note to keep this in sync with the other enum?

[jwd, 2014/08/13 19:34:37]

Done.

+    NO_EXTENSIONS = 0;
+    NO_OFFSTORE_VERIFIED = 1;
+    NO_OFFSTORE_UNVERIFIED = 2;
+    HAS_OFFSTORE = 3;
+  }
+  optional ExtensionsState offstore_extensions_state = 19;
 }