Disable power-save/screensaver in X for factory

Disable power-save/screensaver in X for factory

BUG=chrome-os-partner id:385

TEST=on DUT w/ B824

[Nick Sanders, 2010-07-13 04:49:38]

lgtm w/ comment

http://codereview.chromium.org/2975005/diff/1/2
File client/site_tests/suite_Factory/control (right):

http://codereview.chromium.org/2975005/diff/1/2#newcode217
client/site_tests/suite_Factory/control:217: factory.log("cfg_factory_x: cmd =
%s" % cmd)
can you print the whole cmd, including xset here, for better readability?

[Tammo Spalink, 2010-07-13 18:03:22]

Have you verified that this works?  I am asking because I initially wanted to
move these x commands into the control file, but it would break access to X in
the forked off tests, and I could never figure out why.  If this works, it looks
great.

[Tammo Spalink, 2010-07-13 18:05:43]

http://codereview.chromium.org/2975005/diff/4001/5001
File client/site_tests/suite_Factory/control (right):

http://codereview.chromium.org/2975005/diff/4001/5001#newcode212
client/site_tests/suite_Factory/control:212: os.environ.setdefault('DISPLAY',
':0')
this should already be set, by the factory startx script

http://codereview.chromium.org/2975005/diff/4001/5001#newcode213
client/site_tests/suite_Factory/control:213: os.environ.setdefault('XAUTHORITY',
xauth)
this too...

[Sameer Nanda, 2010-07-13 18:07:16]

http://codereview.chromium.org/2975005/diff/1/2
File client/site_tests/suite_Factory/control (right):

http://codereview.chromium.org/2975005/diff/1/2#newcode217
client/site_tests/suite_Factory/control:217: factory.log("cfg_factory_x: cmd =
%s" % cmd)
drive by comments: you will likely want to stop powerd since idle detection and
screen-lock invocation is now done by powerd.

[Tammo Spalink, 2010-07-16 01:14:28]

LGTM -- provided you go with the one line solution, which is verified to also
work for me.

[Tammo Spalink, 2010-07-16 01:15:04]

LGTM