Add some more categories for V8.TodoMVC to see what keeps us non-idle

Add some more categories for V8.TodoMVC to see what keeps us non-idle

BUG=636405
R=skyostil@chromium.org
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.perf:android_s5_perf_cq;master.tryserver.chromium.perf:linux_perf_cq;master.tryserver.chromium.perf:mac_retina_perf_cq;master.tryserver.chromium.perf:winx64_10_perf_cq

Committed: https://crrev.com/2c3cb38333c08f67b50e482aad3674a7bbfed6be
Cr-Commit-Position: refs/heads/master@{#414187}

[jochen (gone - plz use gerrit), 2016-08-24 16:07:30]

Description was changed from

==========
Add some more categories for V8.TodoMVC to see what keeps us non-idle

BUG=636405
R=skyostil@chromium.org
==========

to

==========
Add some more categories for V8.TodoMVC to see what keeps us non-idle

BUG=636405
R=skyostil@chromium.org
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.perf:android_s5_perf_cq;master.tryserver.chromium.perf:linux_perf_cq;master.tryserver.chromium.perf:mac_retina_perf_cq;master.tryserver.chromium.perf:winx64_10_perf_cq
==========

[jochen (gone - plz use gerrit), 2016-08-24 16:07:32]

The CQ bit was checked by jochen@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-08-24 16:08:00]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[Sami, 2016-08-24 16:40:17]

lgtm.

https://codereview.chromium.org/2274933002/diff/1/tools/perf/benchmarks/v8.py
File tools/perf/benchmarks/v8.py (right):

https://codereview.chromium.org/2274933002/diff/1/tools/perf/benchmarks/v8.py...
tools/perf/benchmarks/v8.py:26: category_filter.AddIncludedCategory('blink')
Did you mean to add toplevel too? We could also try '*' if we're desperate, but
that's likely to move some metrics too.

[commit-bot: I haz the power, 2016-08-24 17:14:53]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2016-08-24 17:14:53]

Dry run: Try jobs failed on following builders:
  win_chromium_rel_ng on master.tryserver.chromium.win (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.win/builders/win_chromium_rel_...)

[jochen (gone - plz use gerrit), 2016-08-24 20:37:50]

The CQ bit was checked by jochen@chromium.org

[jochen (gone - plz use gerrit), 2016-08-24 20:38:36]

On 2016/08/24 at 16:40:17, skyostil wrote:
> lgtm.
> 
> https://codereview.chromium.org/2274933002/diff/1/tools/perf/benchmarks/v8.py
> File tools/perf/benchmarks/v8.py (right):
> 
>
https://codereview.chromium.org/2274933002/diff/1/tools/perf/benchmarks/v8.py...
> tools/perf/benchmarks/v8.py:26: category_filter.AddIncludedCategory('blink')
> Did you mean to add toplevel too? We could also try '*' if we're desperate,
but that's likely to move some metrics too.

locally, this didn't make a difference (in fact, there weren't blink events
either) I'm mostly hoping for the additional debug output from the todomvc.py

[commit-bot: I haz the power, 2016-08-24 20:38:59]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-08-24 23:04:48]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-08-24 23:06:16]

Description was changed from

==========
Add some more categories for V8.TodoMVC to see what keeps us non-idle

BUG=636405
R=skyostil@chromium.org
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.perf:android_s5_perf_cq;master.tryserver.chromium.perf:linux_perf_cq;master.tryserver.chromium.perf:mac_retina_perf_cq;master.tryserver.chromium.perf:winx64_10_perf_cq
==========

to

==========
Add some more categories for V8.TodoMVC to see what keeps us non-idle

BUG=636405
R=skyostil@chromium.org
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.perf:android_s5_perf_cq;master.tryserver.chromium.perf:linux_perf_cq;master.tryserver.chromium.perf:mac_retina_perf_cq;master.tryserver.chromium.perf:winx64_10_perf_cq

Committed: https://crrev.com/2c3cb38333c08f67b50e482aad3674a7bbfed6be
Cr-Commit-Position: refs/heads/master@{#414187}
==========

[commit-bot: I haz the power, 2016-08-24 23:06:17]

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

[jochen (gone - plz use gerrit), 2016-09-02 13:25:51]

A revert of this CL (patchset #1 id:1) has been created in
https://codereview.chromium.org/2306073002/ by jochen@chromium.org.

The reason for reverting is: no longer needed.