Added some comments and a warning message to pyauto perf tests in perf.py.

Added some comments and a warning message to pyauto perf tests in perf.py.

Since performance key values (descriptions) are truncated to 30 characters
before they are stored into the autotest database, a warning message is now
printed in perf.py in the event that a perf description > length 30 is
outputted.

BUG=None
TEST=None


Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=106457

[truty, 2011-10-20 00:12:01]

lgtm

http://codereview.chromium.org/8342055/diff/1/chrome/test/functional/perf.py
File chrome/test/functional/perf.py (right):

http://codereview.chromium.org/8342055/diff/1/chrome/test/functional/perf.py#...
chrome/test/functional/perf.py:129: '(length 30) when added to the autotest
database.' %
Not a big deal, but I think it's preferred when calling logging.xxxx() to avoid
the % string substitution and simple append arguments i.e. logging.warning('blah
%s more %s.', sub1, sub2)

[dennis_jeffrey, 2011-10-20 01:18:21]

Thanks, submitting now.

http://codereview.chromium.org/8342055/diff/1/chrome/test/functional/perf.py
File chrome/test/functional/perf.py (right):

http://codereview.chromium.org/8342055/diff/1/chrome/test/functional/perf.py#...
chrome/test/functional/perf.py:129: '(length 30) when added to the autotest
database.' %
On 2011/10/20 00:12:01, truty wrote:
> Not a big deal, but I think it's preferred when calling logging.xxxx() to
avoid
> the % string substitution and simple append arguments i.e.
logging.warning('blah
> %s more %s.', sub1, sub2)

Done (also changed in several other places in this file where the same comment
applies).

[commit-bot: I haz the power, 2011-10-20 01:18:30]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/dennisjeffrey@chromium.org/8342055/3001

[commit-bot: I haz the power, 2011-10-20 05:28:08]

Change committed as 106457