Change web_ui testing code to avoid using JS router/proxyClass directly.

Change web_ui testing code to avoid using JS router/proxyClass directly.

The previous code didn't validate incoming messages at all.

BUG=579646
Review-Url: https://codereview.chromium.org/2613813002
Cr-Commit-Position: refs/heads/master@{#443020}
Committed: https://chromium.googlesource.com/chromium/src/+/b59d006f624c34344e9835ed99d8279d4aa5701e

[yzshen1, 2017-01-09 22:57:02]

yzshen@chromium.org changed reviewers:
+ rockot@chromium.org

[yzshen1, 2017-01-09 22:57:02]

Hi, Ken.

Would you please take a look at this small CL? It used to cause flakiness. But I
couldn't repo it anymore (see the try results on #3), maybe it has been fixed by
my recent simplification of the bindings.

I am planning to land this CL after the support of console.log()
https://codereview.chromium.org/2618693004/

So that if the flakiness is caused by some invalid message we will know.

[yzshen1, 2017-01-09 22:57:26]

Description was changed from

==========
Change web_ui testing code to avoid using JS router/proxyClass directly.

It doesn't validate incoming messages at all.

BUG=579646

Mojo JS bindings: remove the connection module.

BUG=579646

Mojo JS bindings: Remove all bindings-internal usage of the connection module.

BUG=579646

.


Mojo JS bindings: switch mock-vr-service to use the new bindings API.

BUG=675325,579646
==========

to

==========
Change web_ui testing code to avoid using JS router/proxyClass directly.

It doesn't validate incoming messages at all.

BUG=579646
==========

[yzshen1, 2017-01-09 22:58:08]

Description was changed from

==========
Change web_ui testing code to avoid using JS router/proxyClass directly.

It doesn't validate incoming messages at all.

BUG=579646
==========

to

==========
Change web_ui testing code to avoid using JS router/proxyClass directly.

The previous code didn't validate incoming messages at all.

BUG=579646
==========

[yzshen1, 2017-01-09 23:21:32]

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

[commit-bot: I haz the power, 2017-01-09 23:22:28]

Dry run: 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, 2017-01-10 01:21:46]

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

[commit-bot: I haz the power, 2017-01-10 01:21:47]

Dry run: This issue passed the CQ dry run.

[Ken Rockot(use gerrit already), 2017-01-10 17:07:01]

lgtm

[yzshen1, 2017-01-11 19:14:27]

The CQ bit was checked by yzshen@chromium.org

[commit-bot: I haz the power, 2017-01-11 19:14: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, 2017-01-11 22:29:38]

CQ is committing da patch.

Bot data: {"patchset_id": 60001, "attempt_start_ts": 1484162067372160,
"parent_rev": "38d5cb2144655aad592209bb2a59d21be02df69d", "commit_rev":
"b59d006f624c34344e9835ed99d8279d4aa5701e"}

[commit-bot: I haz the power, 2017-01-11 22:30:08]

Description was changed from

==========
Change web_ui testing code to avoid using JS router/proxyClass directly.

The previous code didn't validate incoming messages at all.

BUG=579646
==========

to

==========
Change web_ui testing code to avoid using JS router/proxyClass directly.

The previous code didn't validate incoming messages at all.

BUG=579646

Review-Url: https://codereview.chromium.org/2613813002
Cr-Commit-Position: refs/heads/master@{#443020}
Committed:
https://chromium.googlesource.com/chromium/src/+/b59d006f624c34344e9835ed99d8...
==========

[commit-bot: I haz the power, 2017-01-11 22:30:09]

Committed patchset #4 (id:60001) as
https://chromium.googlesource.com/chromium/src/+/b59d006f624c34344e9835ed99d8...