Include CPU information in UMA system profile.

chrome/common/metrics/proto/system_profile.proto

 // Copyright (c) 2012 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;
@@ skipping 60 matching lines @@
     optional string name = 1;
 
     // The version of the OS.  The meaning of this field is OS-dependent.
     optional string version = 2;
 
     // The fingerprint of the build.  This field is used only on Android.
     optional string fingerprint = 3;
   }
   optional OS os = 5;
 
-  // Next tag for Hardware: 13
+  // Next tag for Hardware: 14
   // Information on the user's hardware.
   message Hardware {
     // The CPU architecture (x86, PowerPC, x86_64, ...)
     optional string cpu_architecture = 1;
 
     // The amount of RAM present on the system, in megabytes.
     optional int64 system_ram_mb = 2;
 
     // The base memory address that chrome.dll was loaded at.
     // (Logged only on Windows.)
@@ skipping 17 matching lines @@
     optional int32 primary_screen_width = 6;
     optional int32 primary_screen_height = 7;
 
     // The device scale factor of the primary screen.
     optional float primary_screen_scale_factor = 12;
 
     // Max DPI for any attached screen. (Windows only)
     optional float max_dpi_x = 9;
     optional float max_dpi_y = 10;
 
+    // Information on the CPU obtained by CPUID.
+    message CPU {
+      // A 12 character string naming the vendor, e.g. "GeniuneIntel".

[jar (doing other things), 2013/06/26 19:29:34]

Have you gotten this approved for upload in UMA?  We usually shy away from
uploading strings... although there are cases where we do so.

[James Simonsen, 2013/07/09 23:53:26]

Yes, it's been approved by the privacy team. It's effectively the same as the
GPU info, which also uploads vendor strings.

[jar (doing other things), 2013/07/17 00:43:50]

OK... if you've gotten approval, SGTM ;-).

+      optional string vendor_name = 1;
+
+      // Identifies the version within the model.
+      optional uint32 stepping = 2;
+
+      // Identifies the model within the family.
+      optional uint32 model = 3;
+
+      // Identifies processors with similar characteristics.
+      optional uint32 family = 4;
+
+      // Identifies the processor type.
+      optional uint32 type = 5;
+
+      // Additional model bits beyond the basic model. They've overflowed the
+      // original field.
+      optional uint32 extended_model = 6;
+
+      // Additional family bits beyond the basic family. They've overflowed the
+      // original field.
+      optional uint32 extended_family = 7;
+    }
+    optional CPU cpu = 13;
+
     // Information on the GPU
     message Graphics {
       // The GPU manufacturer's vendor id.
       optional uint32 vendor_id = 1;
 
       // The GPU manufacturer's device id for the chip set.
       optional uint32 device_id = 2;
 
       // The driver version on the GPU.
       optional string driver_version = 3;
@@ skipping 284 matching lines @@
     // The name of the field trial, as a 32-bit identifier.
     // Currently, the identifier is a hash of the field trial's name.
     optional fixed32 name_id = 1;
 
     // The user's group within the field trial, as a 32-bit identifier.
     // Currently, the identifier is a hash of the group's name.
     optional fixed32 group_id = 2;
   }
   repeated FieldTrial field_trial = 9;
 }