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

Issue 6627029: Refactor site_power_status, power_strip modules into client/cros. (Closed)

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

Description

Refactor site_power_status, power_strip modules into client/cros. Change-Id: Id9ab9ae36bde45d00992179ee66ac4888a5e2323 BUG=chromium-os:12633 TEST=grep the entire autotest repo and find out all source code which has site_power_status and power_strip and changed them. Committed: http://chrome-svn/viewvc/chromeos?view=rev&revision=7043cc1

Patch Set 1 #

Patch Set 2 : patch #

Patch Set 3 : patch #

Unified diffs Side-by-side diffs Delta from patch set Stats (+47 lines, -45 lines) Patch
A + client/cros/power_status.py View 1 2 chunks +3 lines, -3 lines 0 comments Download
A + client/cros/power_strip.py View 0 chunks +-1 lines, --1 lines 0 comments Download
M client/site_tests/power_ARMSettings/power_ARMSettings.py View 1 2 2 chunks +5 lines, -4 lines 0 comments Download
M client/site_tests/power_Backlight/power_Backlight.py View 2 chunks +3 lines, -3 lines 0 comments Download
M client/site_tests/power_BatteryCharge/control View 1 chunk +2 lines, -2 lines 0 comments Download
M client/site_tests/power_BatteryCharge/power_BatteryCharge.py View 1 chunk +3 lines, -2 lines 0 comments Download
M client/site_tests/power_Draw/control View 1 chunk +1 line, -4 lines 0 comments Download
M client/site_tests/power_Draw/power_Draw.py View 2 chunks +2 lines, -2 lines 0 comments Download
M client/site_tests/power_Idle/power_Idle.py View 2 chunks +7 lines, -6 lines 0 comments Download
M client/site_tests/power_LoadTest/power_LoadTest.py View 1 2 3 chunks +6 lines, -7 lines 0 comments Download
M client/site_tests/power_StatsCPUFreq/power_StatsCPUFreq.py View 2 chunks +2 lines, -2 lines 0 comments Download
M client/site_tests/power_StatsCPUIdle/power_StatsCPUIdle.py View 2 chunks +2 lines, -2 lines 0 comments Download
M client/site_tests/power_StatsUSB/power_StatsUSB.py View 2 chunks +2 lines, -2 lines 0 comments Download
M client/site_tests/power_Status/power_Status.py View 2 chunks +2 lines, -2 lines 0 comments Download
M client/site_tests/power_x86Settings/power_x86Settings.py View 1 2 2 chunks +6 lines, -4 lines 0 comments Download
M server/site_tests/platform_CryptohomeSyncStressServer/platform_CryptohomeSyncStressServer.py View 1 chunk +2 lines, -1 line 0 comments Download

Messages

Total messages: 6 (0 generated)
ericli
9 years, 9 months ago (2011-03-04 23:27:44 UTC) #1
Sameer Nanda
LGTM
9 years, 9 months ago (2011-03-04 23:32:43 UTC) #2
dtu
LGTM
9 years, 9 months ago (2011-03-05 00:01:25 UTC) #3
ericli
Benson, I had update this CL which should fix x86Setting test failure. Please take a ...
9 years, 9 months ago (2011-03-07 17:58:31 UTC) #4
Benson Leung
Hi Eric, Can we close this one now that 6710010 is pushed?
9 years, 9 months ago (2011-03-18 22:38:29 UTC) #5
ericli
9 years, 9 months ago (2011-03-18 23:02:53 UTC) #6
done.

On Fri, Mar 18, 2011 at 3:38 PM, <bleung@chromium.org> wrote:

> Hi Eric,
>
> Can we close this one now that 6710010 is pushed?
>
> http://codereview.chromium.org/6627029/
>



-- 
Eric Li
李咏竹
Google Kirkland

Powered by Google App Engine
This is Rietveld 408576698