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

Issue 3569001: Forcibly turn display on and reset X settings at the start of each iteration. (Closed)

Created:
10 years, 2 months ago by Sameer Nanda
Modified:
9 years, 5 months ago
CC:
chromium-os-reviews_chromium.org, sosa+cc_chromium.org, seano+cc_chromium.org, ericli, petkov+cc_chromium.org
Visibility:
Public.

Description

Forcibly turn display on and reset X settings at the start of each iteration. BUG=chromium-os:7154 TEST= ran the power_LoadTest Change-Id: I2e5301074c51de5e22cd5e1c0b5cb81329f164e4 Committed: http://chrome-svn/viewvc/chromeos?view=rev&revision=d075074

Patch Set 1 #

Patch Set 2 : moved stopping of powerd to after login as well #

Patch Set 3 : Added setting of backlight level #

Patch Set 4 : logging backlight level per loop #

Unified diffs Side-by-side diffs Delta from patch set Stats (+39 lines, -18 lines) Patch
M client/site_tests/power_LoadTest/power_LoadTest.py View 1 2 3 4 chunks +39 lines, -18 lines 0 comments Download

Messages

Total messages: 5 (0 generated)
Sameer Nanda
10 years, 2 months ago (2010-09-29 00:03:27 UTC) #1
Benson Leung
Looks good to me.
10 years, 2 months ago (2010-09-29 00:37:58 UTC) #2
Sameer Nanda
PTAL. Added per loop backlight level setting since powerd may modify backlight level each time ...
10 years, 2 months ago (2010-09-29 16:22:51 UTC) #3
tbroch1
LGTM
10 years, 2 months ago (2010-09-29 17:02:45 UTC) #4
tbroch1
10 years, 2 months ago (2010-09-29 17:02:45 UTC) #5
LGTM

Powered by Google App Engine
This is Rietveld 408576698