Updating endure parser to read new csv output format

Updating endure parser to read new csv output format

TEST='scripts/tools/runit.py scripts/slave/unittests/runtest_annotator_test.py GraphingEndureLogProcessorTest'

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

[edmundyan, 2013-09-05 18:59:58]

Michael, how's this look?  Our bots are actually already broken because the
header line for csv is already changed to "page_name," instead of "url,"

[Michael Achenbach, 2013-09-06 08:34:21]

lgtm with nit

Actually I think we can keep it working with both url and page_name - also after
https://codereview.chromium.org/23963003/ lands. Doesn't hurt.

https://codereview.chromium.org/23740006/diff/1/scripts/slave/process_log_uti...
File scripts/slave/process_log_utils.py (right):

https://codereview.chromium.org/23740006/diff/1/scripts/slave/process_log_uti...
scripts/slave/process_log_utils.py:903: # Assume url is the first column.
nit: update comment, e.g. url -> test name

https://codereview.chromium.org/23740006/diff/1/scripts/slave/process_log_uti...
scripts/slave/process_log_utils.py:925: # Just replace all special characters in
the url with underscore.
nit: update comment: url -> str

[edmundyan, 2013-09-06 16:41:51]

+xusydoc

Hey Mike, could you take a look please?  We needed to update our parser for a
new csv format (https://codereview.chromium.org/23545023/).  Tonyg's concerns
shouldn't affect the csv output.  And this change is still backward-compatible
with the old format.

[edmundyan, 2013-09-09 19:11:14]

On 2013/09/06 16:41:51, edmundyan wrote:
> +xusydoc
> 
> Hey Mike, could you take a look please?  We needed to update our parser for a
> new csv format (https://codereview.chromium.org/23545023/).  Tonyg's concerns
> shouldn't affect the csv output.  And this change is still backward-compatible
> with the old format.

Friendly bump. Our bots are actually broken, so this change is needed now.

[Mike Stip (use stip instead), 2013-09-10 00:05:27]

lgtm

[commit-bot: I haz the power, 2013-09-10 00:07:06]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/edmundyan@chromium.org/23740006/6001

[commit-bot: I haz the power, 2013-09-10 00:08:11]

Change committed as 222139