Major cleanup of cros_au_test_harness code.

Major cleanup of cros_au_test_harness code.

This cleanup code consolidates parsing of options, removes
globals, cleans up private vs. public functions, removes the backup full update option, and cleans up
docstrings.

Change-Id: I3eba26b3634fd87f4a36f195ab42ec7bd11e963f

BUG=chromium-os:8901
TEST=Ran with prefix SimpleTest and ran whole suite for a vm using my latest built image.  Also ran normal update test using latest image against latest dev channel image for mario.  Re-tested with new changes after build break.

Committed: http://chrome-svn/viewvc/chromeos?view=rev&revision=2b2d7a1

Committed: http://chrome-svn/viewvc/chromeos?view=rev&revision=8617623

[sosa, 2011-01-05 00:10:34]

Just a cleanup CL.  Don's out so I think you're the next best reviewer.

[petkov, 2011-01-05 00:57:48]

LGTM with a couple of nits.

http://codereview.chromium.org/6015013/diff/2001/bin/cros_au_test_harness.py
File bin/cros_au_test_harness.py (right):

http://codereview.chromium.org/6015013/diff/2001/bin/cros_au_test_harness.py#...
bin/cros_au_test_harness.py:3: # Copyright (c) 2010-2011 The Chromium OS
Authors. All rights reserved.
do we do that, or do we just change to 2011?

http://codereview.chromium.org/6015013/diff/2001/bin/cros_au_test_harness.py#...
bin/cros_au_test_harness.py:88: unittest.assertTrue(percent_passed >=
fits on one line?

http://codereview.chromium.org/6015013/diff/2001/bin/cros_au_test_harness.py#...
bin/cros_au_test_harness.py:159: def ParseOptions(cls, parser, options):
You're not really parsing options here. Rename -- even something generic like
"ProcessOptions" might be better.

[sosa, 2011-01-10 21:16:48]

PTAL (third diff).  I rebased this off TOT and fixed the bug you caught (flip of
target / base for updates.

[petkov, 2011-01-10 21:26:40]

LGTM