Stop the existing session if it exists before start a new session and add mirroring scenario for cp…

Stop the existing session if it exists before start a new session and add mirroring scenario for cpu and memory usage performance tests.

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

[Lei Lei, 2016-04-28 01:17:45]

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

[imcheng, 2016-04-28 18:28:36]

lgtm

https://codereview.chromium.org/1931553004/diff/20001/chrome/test/media_route...
File chrome/test/media_router/telemetry/benchmarks/pagesets/media_router_page.py
(right):

https://codereview.chromium.org/1931553004/diff/20001/chrome/test/media_route...
chrome/test/media_router/telemetry/benchmarks/pagesets/media_router_page.py:81:
logging.info('Is there existing router? ' + str(route))
s/router/route

[apacible, 2016-04-28 20:39:56]

lgtm

https://codereview.chromium.org/1931553004/diff/20001/chrome/test/media_route...
File chrome/test/media_router/telemetry/benchmarks/pagesets/media_router_page.py
(right):

https://codereview.chromium.org/1931553004/diff/20001/chrome/test/media_route...
chrome/test/media_router/telemetry/benchmarks/pagesets/media_router_page.py:69:
"    console.info('sink id: ' + sinks[i].id); "
Do we need to print out all of the sink ids for the test?

https://codereview.chromium.org/1931553004/diff/20001/chrome/test/media_route...
chrome/test/media_router/telemetry/benchmarks/pagesets/media_router_page.py:70:
"    sink_id = sinks[i].id;"
Break early in this loop and below loop if the sink/route id was found, since we
expect unique ids.

[Lei Lei, 2016-04-28 23:11:06]

https://codereview.chromium.org/1931553004/diff/20001/chrome/test/media_route...
File chrome/test/media_router/telemetry/benchmarks/pagesets/media_router_page.py
(right):

https://codereview.chromium.org/1931553004/diff/20001/chrome/test/media_route...
chrome/test/media_router/telemetry/benchmarks/pagesets/media_router_page.py:69:
"    console.info('sink id: ' + sinks[i].id); "
On 2016/04/28 20:39:55, apacible wrote:
> Do we need to print out all of the sink ids for the test?

No, this is just for debug purpose. I would like to have some logs to make sure
it can find the sink with specific name. 

We don't need all the sink ids for the test.

https://codereview.chromium.org/1931553004/diff/20001/chrome/test/media_route...
chrome/test/media_router/telemetry/benchmarks/pagesets/media_router_page.py:70:
"    sink_id = sinks[i].id;"
On 2016/04/28 20:39:55, apacible wrote:
> Break early in this loop and below loop if the sink/route id was found, since
we
> expect unique ids.

Done.

https://codereview.chromium.org/1931553004/diff/20001/chrome/test/media_route...
chrome/test/media_router/telemetry/benchmarks/pagesets/media_router_page.py:81:
logging.info('Is there existing router? ' + str(route))
On 2016/04/28 18:28:36, imcheng wrote:
> s/router/route

Done.

[Lei Lei, 2016-04-28 23:11:15]

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

[commit-bot: I haz the power, 2016-04-28 23:11:29]

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

[commit-bot: I haz the power, 2016-04-29 00:25:54]

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

[commit-bot: I haz the power, 2016-04-29 00:25:55]

Dry run: This issue passed the CQ dry run.

[Lei Lei, 2016-04-29 17:34:01]

The CQ bit was checked by leilei@chromium.org

[Lei Lei, 2016-04-29 17:34:01]

The patchset sent to the CQ was uploaded after l-g-t-m from
imcheng@chromium.org, apacible@chromium.org
Link to the patchset: https://codereview.chromium.org/1931553004/#ps40001
(title: "Addressed Derek's and Jennifer's comments")

[commit-bot: I haz the power, 2016-04-29 17:34:11]

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

[commit-bot: I haz the power, 2016-04-29 17:39:08]

Committed patchset #3 (id:40001)

[commit-bot: I haz the power, 2016-04-30 17:26:52]

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

[commit-bot: I haz the power, 2016-04-30 18:01:17]

Description was changed from

==========
Stop the existing session if it exists before start a new session and add
mirroring scenario for cpu and memory usage performance tests.

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

to

==========
Stop the existing session if it exists before start a new session and add
mirroring scenario for cpu and memory usage performance tests.

BUG=594225

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