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

Issue 6633013: autotest/factoryVerify: fix variable binding issue (Closed)

Created:
9 years, 9 months ago by Hung-Te
Modified:
9 years, 7 months ago
Reviewers:
Nick Sanders, ericli
CC:
chromium-os-reviews_chromium.org, truty+cc_chromium.org, sosa+cc_chromium.org, seano+cc_chromium.org, ericli
Visibility:
Public.

Description

autotest/factoryVerify: fix variable binding issue In conde refine ( http://codereview.chromium.org/6596018 ), the module name was changed and caused conflicts with variable name. BUG=chrome-os-partner:2611 TEST=verified pylint and grep -Ril 'gpio = gpio', grep -Ril 'vblock = vblock'. Change-Id: I6d530435ccc48869edd5fbfc39ece367507c12ca Committed: http://chrome-svn/viewvc/chromeos?view=rev&revision=c1a954c

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+3 lines, -3 lines) Patch
M client/site_tests/factory_Verify/factory_Verify.py View 1 chunk +3 lines, -3 lines 0 comments Download

Messages

Total messages: 2 (0 generated)
Hung-Te
9 years, 9 months ago (2011-03-05 03:11:28 UTC) #1
Nick Sanders
9 years, 9 months ago (2011-03-05 05:22:48 UTC) #2
lgtm

Powered by Google App Engine
This is Rietveld 408576698