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

Issue 6526021: add ec_info to chrome://system screen (Closed)

Created:
9 years, 10 months ago by dhendrix
Modified:
9 years, 5 months ago
Reviewers:
Benson Leung
CC:
chromium-os-reviews_chromium.org
Visibility:
Public.

Description

add ec_info to chrome://system screen Change-Id: Id92b6f8957b5d228775584706cdb32a0a13fee65 BUG=BUG=chrome-os-partner:2092 TEST=Tested on an Alex Committed: http://chrome-svn/viewvc/chromeos?view=rev&revision=9066de6

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+1 line, -0 lines) Patch
M etc/sys_log_utils.lst View 1 chunk +1 line, -0 lines 0 comments Download

Messages

Total messages: 4 (0 generated)
dhendrix
9 years, 10 months ago (2011-02-15 21:38:34 UTC) #1
Benson Leung
LGTM.
9 years, 10 months ago (2011-02-15 22:32:35 UTC) #2
Benson Leung
On 2011/02/15 22:32:35, Benson Leung wrote: > LGTM. Oh, could you also set the BUG=chrome-os-partner:2092 ...
9 years, 10 months ago (2011-02-15 22:34:06 UTC) #3
dhendrix
9 years, 10 months ago (2011-02-15 23:06:04 UTC) #4
On 2011/02/15 22:34:06, Benson Leung wrote:
> On 2011/02/15 22:32:35, Benson Leung wrote:
> > LGTM.
> 
> Oh, could you also set the BUG=chrome-os-partner:2092 ?
> 
> Thanks,

Added the bug #. Thanks for the quick review!

Powered by Google App Engine
This is Rietveld 408576698