[Telemetry] Migrate browser_test_runner to use typ as the test runner

[Telemetry] Migrate browser_test_runner to use typ as the test runner

This is a reland of https://codereview.chromium.org/2671483002/. The
previous work was reverted to test out the change to gpu tests in chromium repo.

Patch 1 is the original work, patch 2 & beyond includes fix for TearDownProcess

BUG=chromium:636153
TBR=kbr@chromium.org

Review-Url: https://codereview.chromium.org/2671383003
Committed: https://chromium.googlesource.com/external/github.com/catapult-project/catapult/+/12bf6ba0611a7c58086e665844f0fbf3321b578a

[nednguyen, 2017-02-07 00:43:39]

https://codereview.chromium.org/2671383003/diff/1/telemetry/telemetry/testing...
File telemetry/telemetry/testing/run_browser_tests.py (right):

https://codereview.chromium.org/2671383003/diff/1/telemetry/telemetry/testing...
telemetry/telemetry/testing/run_browser_tests.py:352:
context.test_class.TearDownProcess()
This is the bug. context were deleted & turn out failure in _TearDownProcess is
silent. This is a bit scary. 

I only found out about this after try run in
https://codereview.chromium.org/2663813003/ fails with the error message that
we're leaking browser process.

+Dirk: I guess this requires a fix in typ to make failure in _TearDownProcess
not silent.

[nednguyen, 2017-02-07 00:43:54]

Description was changed from

==========
[Telemetry] Migrate browser_test_runner to use typ as the test runner

This is a reland of https://codereview.chromium.org/2671483002/. The
previous work was reverted to test out the change to gpu tests in chromium repo.

Patch 1 is the original work, patch 2 & beyond includes fix for TearDownProcess

BUG=chromium:636153
==========

to

==========
[Telemetry] Migrate browser_test_runner to use typ as the test runner

This is a reland of https://codereview.chromium.org/2671483002/. The
previous work was reverted to test out the change to gpu tests in chromium repo.

Patch 1 is the original work, patch 2 & beyond includes fix for TearDownProcess

BUG=chromium:636153
TBR=kbr@chromium.org
==========

[nednguyen, 2017-02-07 00:43:54]

nednguyen@google.com changed reviewers:
+ dpranke@chromium.org

[nednguyen, 2017-02-07 00:43:58]

The CQ bit was checked by nednguyen@google.com

[commit-bot: I haz the power, 2017-02-07 00:44:08]

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

[nednguyen, 2017-02-07 00:44:34]

On 2017/02/07 00:43:39, nednguyen wrote:
>
https://codereview.chromium.org/2671383003/diff/1/telemetry/telemetry/testing...
> File telemetry/telemetry/testing/run_browser_tests.py (right):
> 
>
https://codereview.chromium.org/2671383003/diff/1/telemetry/telemetry/testing...
> telemetry/telemetry/testing/run_browser_tests.py:352:
> context.test_class.TearDownProcess()
> This is the bug. context were deleted & turn out failure in _TearDownProcess
is
> silent. This is a bit scary. 
> 
> I only found out about this after try run in
> https://codereview.chromium.org/2663813003/ fails with the error message that
> we're leaking browser process.
> 
> +Dirk: I guess this requires a fix in typ to make failure in _TearDownProcess
> not silent.

Land this so I can revert it again & run try job in
https://codereview.chromium.org/2663813003/

[commit-bot: I haz the power, 2017-02-07 01:05:16]

CQ is committing da patch.

Bot data: {"patchset_id": 20001, "attempt_start_ts": 1486428238914220,
"parent_rev": "32163602970d410a546747e0a3578419a7f25dc4", "commit_rev":
"12bf6ba0611a7c58086e665844f0fbf3321b578a"}

[commit-bot: I haz the power, 2017-02-07 01:05:19]

Description was changed from

==========
[Telemetry] Migrate browser_test_runner to use typ as the test runner

This is a reland of https://codereview.chromium.org/2671483002/. The
previous work was reverted to test out the change to gpu tests in chromium repo.

Patch 1 is the original work, patch 2 & beyond includes fix for TearDownProcess

BUG=chromium:636153
TBR=kbr@chromium.org
==========

to

==========
[Telemetry] Migrate browser_test_runner to use typ as the test runner

This is a reland of https://codereview.chromium.org/2671483002/. The
previous work was reverted to test out the change to gpu tests in chromium repo.

Patch 1 is the original work, patch 2 & beyond includes fix for TearDownProcess

BUG=chromium:636153
TBR=kbr@chromium.org

Review-Url: https://codereview.chromium.org/2671383003
Committed:
https://chromium.googlesource.com/external/github.com/catapult-project/catapu...
==========

[commit-bot: I haz the power, 2017-02-07 01:05:20]

Committed patchset #2 (id:20001) as
https://chromium.googlesource.com/external/github.com/catapult-project/catapu...

[nednguyen, 2017-02-07 01:08:55]

A revert of this CL (patchset #2 id:20001) has been created in
https://codereview.chromium.org/2681453003/ by nednguyen@google.com.

The reason for reverting is: Revert to try out the change in Chromium CQ.