Add one benchmark to get CPU and memory usage without MR.

Add one benchmark to get CPU and memory usage without MR.

BUG=594225
Committed: https://crrev.com/cf85acfd343c7a02c9be08b6903f5290568fa26e
Cr-Commit-Position: refs/heads/master@{#393408}

[Lei Lei, 2016-05-12 00:19:36]

leilei@chromium.org changed reviewers:
+ apacible@chromium.org, imcheng@chromium.org

[imcheng, 2016-05-12 20:57:08]

https://codereview.chromium.org/1975603002/diff/1/chrome/test/media_router/te...
File chrome/test/media_router/telemetry/benchmarks/media_router_benchmark.py
(right):

https://codereview.chromium.org/1975603002/diff/1/chrome/test/media_router/te...
chrome/test/media_router/telemetry/benchmarks/media_router_benchmark.py:108:
'--load-extension=' +
1) we don't need to load the extension if MR is disabled
2) prefer explicitly setting --media-router=0

[Lei Lei, 2016-05-12 21:07:14]

https://codereview.chromium.org/1975603002/diff/1/chrome/test/media_router/te...
File chrome/test/media_router/telemetry/benchmarks/media_router_benchmark.py
(right):

https://codereview.chromium.org/1975603002/diff/1/chrome/test/media_router/te...
chrome/test/media_router/telemetry/benchmarks/media_router_benchmark.py:108:
'--load-extension=' +
On 2016/05/12 20:57:08, imcheng wrote:
> 1) we don't need to load the extension if MR is disabled
> 2) prefer explicitly setting --media-router=0

1), it doesn't load MR extension, it only loads the test extension which is
needed to collect cpu and memory usage. For MR on case, we need to load 2
extensions, one is MR extension, another is the test extension. With MR off, we
only need to load test extension.

2) Done.

[imcheng, 2016-05-12 21:09:39]

lgtm

Thanks!

[Lei Lei, 2016-05-12 21:40:57]

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

[commit-bot: I haz the power, 2016-05-12 21:41:18]

Dry run: CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1975603002/20001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1975603002/20001

[apacible, 2016-05-12 21:51:53]

lgtm

[commit-bot: I haz the power, 2016-05-12 22:33:40]

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

[commit-bot: I haz the power, 2016-05-12 22:33:41]

Dry run: This issue passed the CQ dry run.

[Lei Lei, 2016-05-13 00:20:14]

The CQ bit was checked by leilei@chromium.org

[commit-bot: I haz the power, 2016-05-13 00:20:36]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1975603002/20001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1975603002/20001

[commit-bot: I haz the power, 2016-05-13 00:24:19]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-05-13 00:25:26]

Description was changed from

==========
Add one benchmark to get CPU and memory usage without MR.

BUG=594225
==========

to

==========
Add one benchmark to get CPU and memory usage without MR.

BUG=594225

Committed: https://crrev.com/cf85acfd343c7a02c9be08b6903f5290568fa26e
Cr-Commit-Position: refs/heads/master@{#393408}
==========

[commit-bot: I haz the power, 2016-05-13 00:25:27]

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