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

Issue 2289001: A script to print the hardware class (e.g., hwqual ID) of the device. (Closed)

Created:
10 years, 7 months ago by petkov
Modified:
9 years, 7 months ago
Reviewers:
Sameer Nanda, sosa
CC:
chromium-os-reviews_chromium.org, petkov, Luigi Semenzato, sosa
Base URL:
ssh://git@chromiumos-git/chromeos
Visibility:
Public.

Description

A script to print the hardware class (e.g., hwqual ID) of the device. This script will be used be Chrome's UMA service to add the "hardwareclass" field to the XML to be uploaded.

Patch Set 1 #

Patch Set 2 : Add some double quotes to guard against spaces. #

Total comments: 2

Patch Set 3 : Add cpu info. #

Unified diffs Side-by-side diffs Delta from patch set Stats (+56 lines, -0 lines) Patch
A src/platform/metrics/hardware_class View 1 2 1 chunk +56 lines, -0 lines 0 comments Download

Messages

Total messages: 6 (0 generated)
petkov
Open to and looking for suggestions on: - Location of the script (both in the ...
10 years, 7 months ago (2010-05-26 18:26:07 UTC) #1
Sameer Nanda
http://codereview.chromium.org/2289001/diff/3001/4001 File src/platform/metrics/hardware_class (right): http://codereview.chromium.org/2289001/diff/3001/4001#newcode32 src/platform/metrics/hardware_class:32: you will probably want to add cpu family, model ...
10 years, 7 months ago (2010-05-26 19:25:27 UTC) #2
sosa
/usr/bin and platform/metrics look like good locations. http://codereview.chromium.org/2289001/diff/3001/4001 File src/platform/metrics/hardware_class (right): http://codereview.chromium.org/2289001/diff/3001/4001#newcode32 src/platform/metrics/hardware_class:32: You also ...
10 years, 7 months ago (2010-05-26 19:34:58 UTC) #3
petkov
PTAL. I've added CPU info. It seems unlikely we'll see machines with the same CPU/WiFi ...
10 years, 7 months ago (2010-05-26 21:42:56 UTC) #4
Sameer Nanda
LGTM
10 years, 7 months ago (2010-05-26 21:49:56 UTC) #5
sosa
10 years, 7 months ago (2010-05-26 22:31:00 UTC) #6
LGTM

On Wed, May 26, 2010 at 2:49 PM,  <snanda@chromium.org> wrote:
> LGTM
>
> http://codereview.chromium.org/2289001/show
>

Powered by Google App Engine
This is Rietveld 408576698