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

Issue 3448023: autotest: Stash away cryptohomed.log to help diagnose login issues (Closed)

Created:
10 years, 3 months ago by kmixter1
Modified:
9 years, 7 months ago
Reviewers:
Chris Masone, fes, truty1
CC:
chromium-os-reviews_chromium.org, sosa+cc_chromium.org, seano+cc_chromium.org, ericli, petkov+cc_chromium.org
Visibility:
Public.

Description

autotest: Stash away cryptohomed.log to help diagnose login issues Change-Id: I8333131a9d88162792bd5865ba2c3ef8123db34f BUG=6883 TEST=Ran desktopui_ChromeFirstRender, verified cryptohomed.log in results Committed: http://chrome-svn/viewvc/chromeos?view=rev&revision=e1fc460

Patch Set 1 #

Total comments: 1
Unified diffs Side-by-side diffs Delta from patch set Stats (+7 lines, -0 lines) Patch
M client/bin/chromeos_constants.py View 1 chunk +2 lines, -0 lines 0 comments Download
M client/bin/site_ui_test.py View 1 chunk +5 lines, -0 lines 1 comment Download

Messages

Total messages: 3 (0 generated)
kmixter1
10 years, 3 months ago (2010-09-23 22:54:01 UTC) #1
truty1
http://codereview.chromium.org/3448023/diff/1/3 File client/bin/site_ui_test.py (right): http://codereview.chromium.org/3448023/diff/1/3#newcode262 client/bin/site_ui_test.py:262: base = os.path.basename(chromeos_constants.CRYPTOHOMED_LOG) should this be in a try ...
10 years, 3 months ago (2010-09-24 00:27:27 UTC) #2
fes
10 years, 3 months ago (2010-09-24 01:44:55 UTC) #3
LGTM other than truty1's comment.

Powered by Google App Engine
This is Rietveld 408576698