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

Side by Side Diff: chrome/common/metrics/proto/system_profile.proto

Issue 134773004: Include external touchscreen vid/pid in UMA hardware profile (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Rebase onto refactor. Created 6 years, 10 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 unified diff | Download patch | Annotate | Revision Log
OLDNEW
1 // Copyright (c) 2012 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 // 4 //
5 // Stores information about the user's brower and system configuration. 5 // Stores information about the user's brower and system configuration.
6 // The system configuration fields are recorded once per client session. 6 // The system configuration fields are recorded once per client session.
7 7
8 syntax = "proto2"; 8 syntax = "proto2";
9 9
10 option optimize_for = LITE_RUNTIME; 10 option optimize_for = LITE_RUNTIME;
(...skipping 65 matching lines...) Expand 10 before | Expand all | Expand 10 after
76 optional string name = 1; 76 optional string name = 1;
77 77
78 // The version of the OS. The meaning of this field is OS-dependent. 78 // The version of the OS. The meaning of this field is OS-dependent.
79 optional string version = 2; 79 optional string version = 2;
80 80
81 // The fingerprint of the build. This field is used only on Android. 81 // The fingerprint of the build. This field is used only on Android.
82 optional string fingerprint = 3; 82 optional string fingerprint = 3;
83 } 83 }
84 optional OS os = 5; 84 optional OS os = 5;
85 85
86 // Next tag for Hardware: 14 86 // Next tag for Hardware: 16
87 // Information on the user's hardware. 87 // Information on the user's hardware.
88 message Hardware { 88 message Hardware {
89 // The CPU architecture (x86, PowerPC, x86_64, ...) 89 // The CPU architecture (x86, PowerPC, x86_64, ...)
90 optional string cpu_architecture = 1; 90 optional string cpu_architecture = 1;
91 91
92 // The amount of RAM present on the system, in megabytes. 92 // The amount of RAM present on the system, in megabytes.
93 optional int64 system_ram_mb = 2; 93 optional int64 system_ram_mb = 2;
94 94
95 // The base memory address that chrome.dll was loaded at. 95 // The base memory address that chrome.dll was loaded at.
96 // (Logged only on Windows.) 96 // (Logged only on Windows.)
(...skipping 115 matching lines...) Expand 10 before | Expand all | Expand 10 after
212 // Product ID of the device, where available. 212 // Product ID of the device, where available.
213 optional uint32 product_id = 6; 213 optional uint32 product_id = 6;
214 214
215 // Device ID of the device, generally the release or version number in 215 // Device ID of the device, generally the release or version number in
216 // BCD format, where available. 216 // BCD format, where available.
217 optional uint32 device_id = 7; 217 optional uint32 device_id = 7;
218 } 218 }
219 repeated PairedDevice paired_device = 3; 219 repeated PairedDevice paired_device = 3;
220 } 220 }
221 optional Bluetooth bluetooth = 11; 221 optional Bluetooth bluetooth = 11;
222
223 // Whether the internal display produces touch events.
Alexei Svitkine (slow) 2014/02/04 21:13:35 Expand comment to mention that if this is omitted
tdresser 2014/02/04 22:01:49 Done. I'll land the google3 side of this change on
224 optional bool internal_display_supports_touch = 14;
225
226 // Vendor ids and product ids of external touchscreens.
227 message TouchScreen {
228 // Touch screen vendor id.
229 optional uint32 vendor_id = 1;
230 // Touch screen product id.
231 optional uint32 product_id = 2;
232 }
233 // Lists vendor and product ids of external touchscreens.
234 repeated TouchScreen external_touchscreen = 15;
222 } 235 }
223 optional Hardware hardware = 6; 236 optional Hardware hardware = 6;
224 237
225 // Information about the network connection. 238 // Information about the network connection.
226 message Network { 239 message Network {
227 // Set to true if connection_type changed during the lifetime of the log. 240 // Set to true if connection_type changed during the lifetime of the log.
228 optional bool connection_type_is_ambiguous = 1; 241 optional bool connection_type_is_ambiguous = 1;
229 242
230 // See net::NetworkChangeNotifier::ConnectionType. 243 // See net::NetworkChangeNotifier::ConnectionType.
231 enum ConnectionType { 244 enum ConnectionType {
(...skipping 204 matching lines...) Expand 10 before | Expand all | Expand 10 after
436 // Currently, the identifier is a hash of the group's name. 449 // Currently, the identifier is a hash of the group's name.
437 optional fixed32 group_id = 2; 450 optional fixed32 group_id = 2;
438 } 451 }
439 repeated FieldTrial field_trial = 9; 452 repeated FieldTrial field_trial = 9;
440 453
441 // Number of users currently signed into a multiprofile session. 454 // Number of users currently signed into a multiprofile session.
442 // A zero value indicates that the user count changed while the log is open. 455 // A zero value indicates that the user count changed while the log is open.
443 // Logged only on ChromeOS. 456 // Logged only on ChromeOS.
444 optional uint32 multi_profile_user_count = 17; 457 optional uint32 multi_profile_user_count = 17;
445 } 458 }
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698