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..5261ad611dabf4019bed379b3e8a7d03b39a6841 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,

[Alexei Svitkine (slow), 2014/08/12 19:17:13]

Nit: "off store" -> "off-store" to make it easier to understand.

[jwd, 2014/08/12 21:24:17]

Done.

+  // then HAS_OFFSTORE is reported.
+  enum ExtensionsState {
+    NO_EXTENSIONS = 0;
+    NO_OFFSTORE_VERIFIED = 1;
+    NO_OFFSTORE_UNVERIFIED = 2;
+    HAS_OFFSTORE = 3;
+  }
+  optional ExtensionsState offstore_extensions_state = 19;
 }