Give names to all WebRTC tests, temporarily disable audio codec tests.

Give names to all WebRTC tests, temporarily disable audio codec tests.

I found out I can give names to the test cases, which should make the
graphs a lot easier to read. Also disabling some tests that currently
don't produce useful data.

BUG=468732
Committed: https://crrev.com/0807f9b84b7f315224d2b920546df1969f8c1cd9
Cr-Commit-Position: refs/heads/master@{#321544}

[phoglund_chromium, 2015-03-19 12:59:41]

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

[phoglund_chromium, 2015-03-19 12:59:41]

If you're uncomfortable reviewing this, let me know if there's anyone else I
should add (I think this is pretty straightforward though).

[qyearsley, 2015-03-20 01:04:02]

On 2015/03/19 12:59:41, phoglund wrote:
> If you're uncomfortable reviewing this, let me know if there's anyone else I
> should add (I think this is pretty straightforward though).

LGTM with nits :-)

Be aware that giving names to the series means that the new data will go to new
series (the old data won't be viewable under the new page name unless it's all
explicitly renamed). There will be a two-week period where the old page names
will be listed in the chart will still be there, then it will be hidden.

[phoglund_chromium, 2015-03-20 09:26:15]

On 2015/03/20 01:04:02, qyearsley wrote:
> On 2015/03/19 12:59:41, phoglund wrote:
> > If you're uncomfortable reviewing this, let me know if there's anyone else I
> > should add (I think this is pretty straightforward though).
> 
> LGTM with nits :-)
> 
> Be aware that giving names to the series means that the new data will go to
new
> series (the old data won't be viewable under the new page name unless it's all
> explicitly renamed). There will be a two-week period where the old page names
> will be listed in the chart will still be there, then it will be hidden.

Thanks! Yeah, once this lands I guess I need to run a migration.

[phoglund_chromium, 2015-03-20 09:26:19]

The CQ bit was checked by phoglund@chromium.org

[commit-bot: I haz the power, 2015-03-20 09:26:48]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1014573006/1

[commit-bot: I haz the power, 2015-03-20 10:33:48]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2015-03-20 10:34:34]

Patchset 1 (id:??) landed as
https://crrev.com/0807f9b84b7f315224d2b920546df1969f8c1cd9
Cr-Commit-Position: refs/heads/master@{#321544}

[qyearsley, 2015-03-20 16:37:28]

Oops, I forgot to actually publish my inline comments before making the comment
before. (Well, they were just nits...)

https://codereview.chromium.org/1014573006/diff/1/tools/perf/page_sets/webrtc...
File tools/perf/page_sets/webrtc_cases.py (right):

https://codereview.chromium.org/1014573006/diff/1/tools/perf/page_sets/webrtc...
tools/perf/page_sets/webrtc_cases.py:31: name="vga_local_stream_10s",
Optional: Use single quotes for more consistency with other strings in this
file.

I actually believed that the style guide explicitly said to use single quotes,
but Now that I look at PEP8 and google style guide, neither of these forbid
double quotes for strings, but they both suggest keeping it consistent.

https://codereview.chromium.org/1014573006/diff/1/tools/perf/page_sets/webrtc...
tools/perf/page_sets/webrtc_cases.py:172: #self.AddUserStory(Page7(self))
You could also I think that deleting these lines and add a TODO rather than
commenting them out.

[phoglund_chromium, 2015-03-23 07:22:15]

Ah. Yeah in retrospect it looked weird that you mentioned nits, but there
weren't any. I've fixed those now and they'll show up in my next patch.