Adding logging of offstore extensions state to user metrics.

components/metrics/proto/system_profile.proto

 // Copyright 2014 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 //
 // Stores information about the user's brower and system configuration.
 // The system configuration fields are recorded once per client session.
 
 syntax = "proto2";
 
 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;
 
   // A version number string for the application.
   // Most commonly this is the browser version number found in a user agent
   // string, and is typically a 4-tuple of numbers separated by periods.  In
   // cases where the user agent version might be ambiguous (example: Linux 64-
   // bit build, rather than 32-bit build, or a Windows version used in some
   // special context, such as ChromeFrame running in IE), then this may include
@@ skipping 475 matching lines @@
   // Each client first picks a value for client_key derived from its UMA
   // client_id:
   //   client_key = client_id % 4096
   // Then, each installed extension is mapped into a hash bucket according to
   //   bucket = CityHash64(StringPrintf("%d:%s",
   //                                    client_key, extension_id)) % 1024
   // The client reports the set of hash buckets occupied by all installed
   // 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;
 }