Rename process_log_utils and related things.

Rename process_log_utils and related things.

I've wanted to do this for a looooooong time.

 - process_log_utils.py -> performance_log_processor.py (Because the main purpose of that module is that it contains PerformanceLogProcessor and subclasses).
 - runtest_annotator_test.py -> performance_log_processor_test.py (because it's a test of the above module).

And, in runtest.py, instances of these classes have been called:
  log parser
  log processor
  results tracker
  etc.
This CL changes runtest.py to use the term "log processor" for consistency.

All the changed modules pass pylint.

BUG=

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

[qyearsley, 2014-09-09 04:39:01]

qyearsley@chromium.org changed reviewers:
+ stip@chromium.org

[ghost stip (do not use), 2014-09-09 17:47:57]

quinten, can you coordinate with sullivan@ w.r.t.
https://chromiumcodereview.appspot.com/545803002/? I think you'll have minimal
conflicts, but I have a feeling someone will have to rebase on top of the other

[sullivan, 2014-09-09 17:53:30]

On 2014/09/09 17:47:57, stip wrote:
> quinten, can you coordinate with sullivan@ w.r.t.
> https://chromiumcodereview.appspot.com/545803002/ I think you'll have minimal
> conflicts, but I have a feeling someone will have to rebase on top of the
other

It would probably be better to submit this CL first and rebase mine on top of
it, since mine's more likely to get reverted and re-landed if we hit unexpected
issues.

Quinten, I'd love to do more cleanups in this code (get rid of process_log_utils
altogether, even!) but I am pretty swamped at least until end of quarter. If you
have more time to spend on improving the buildbot source, I think there is a lot
more room for cleanup after https://chromiumcodereview.appspot.com/545803002/
has landed. And probably room for making the code cleaner as a part of that CL
as well.

[qyearsley, 2014-09-09 18:30:02]

On 2014/09/09 17:53:30, sullivan wrote:
> On 2014/09/09 17:47:57, stip wrote:
> > quinten, can you coordinate with sullivan@ w.r.t.
> > https://chromiumcodereview.appspot.com/545803002/ I think you'll have
minimal
> > conflicts, but I have a feeling someone will have to rebase on top of the
> other
> 
> It would probably be better to submit this CL first and rebase mine on top of
> it, since mine's more likely to get reverted and re-landed if we hit
unexpected
> issues.
> 
> Quinten, I'd love to do more cleanups in this code (get rid of
process_log_utils
> altogether, even!) but I am pretty swamped at least until end of quarter. If
you
> have more time to spend on improving the buildbot source, I think there is a
lot
> more room for cleanup after https://chromiumcodereview.appspot.com/545803002/
> has landed. And probably room for making the code cleaner as a part of that CL
> as well.

Yep, sounds good! I'm also fine with holding out and rebasing on that CL as
well.
I actually listed as one of my OKRs to refactor/simplify the code here,
hopefully replacing the performance log processor classes, but I'm also very
unlikely to do that until next quarter.

[ghost stip (do not use), 2014-09-09 21:45:22]

lgtm

https://chromiumcodereview.appspot.com/548773004/diff/1/scripts/slave/unittes...
File scripts/slave/unittests/runtest_annotator_test.py (left):

https://chromiumcodereview.appspot.com/548773004/diff/1/scripts/slave/unittes...
scripts/slave/unittests/runtest_annotator_test.py:1: #!/usr/bin/env python
we finally renamed this file :)

[qyearsley, 2014-09-09 22:23:33]

The CQ bit was checked by qyearsley@chromium.org

[commit-bot: I haz the power, 2014-09-09 22:23:56]

CQ is trying da patch. Follow status at
 https://chromium-status.appspot.com/cq/qyearsley@chromium.org/548773004/20001

[commit-bot: I haz the power, 2014-09-09 22:25:33]

Committed patchset #2 (id:20001) as 291894