[Media Router] Export Mojom definitions in media_router_bindings.js.

[Media Router] Export Mojom definitions in media_router_bindings.js.

In media_router_bindings, instead of just exporting the mediaRouter
instance to the component, also export definitions of generated Mojom
classes that will be needed by the component. This lets the component
work with Mojo objects directly rather than relying on bindings as an
intermediate layer.

BUG=687365
Review-Url: https://codereview.chromium.org/2779203003
Cr-Commit-Position: refs/heads/master@{#461194}
Committed: https://chromium.googlesource.com/chromium/src/+/532ac77836647714a0ed93c871b4a0b75159fcaa

[imcheng, 2017-03-29 22:04:45]

Description was changed from

==========
[Media Router] Export Mojom definitions in media_router_bindings.js.

In media_router_bindings, instead of just exporting the mediaRouter
instance to the component, also export definitions of generated Mojom
classes that will be needed by the component. This lets the component
work with Mojo objects directly rather than relying on bindings as an
intermediate layer.

BUG=687365
==========

to

==========
[Media Router] Export Mojom definitions in media_router_bindings.js.

In media_router_bindings, instead of just exporting the mediaRouter
instance to the component, also export definitions of generated Mojom
classes that will be needed by the component. This lets the component
work with Mojo objects directly rather than relying on bindings as an
intermediate layer.

BUG=687365
==========

[imcheng, 2017-03-29 23:01:45]

imcheng@chromium.org changed reviewers:
+ mfoltz@chromium.org

[imcheng, 2017-03-29 23:01:46]

PTAL

[imcheng, 2017-03-29 23:37:33]

https://codereview.chromium.org/2779203003/diff/20001/extensions/renderer/res...
File extensions/renderer/resources/media_router_bindings.js (right):

https://codereview.chromium.org/2779203003/diff/20001/extensions/renderer/res...
extensions/renderer/resources/media_router_bindings.js:9:
'chrome/browser/media/router/mojo/media_controller.mojom',
Note: I will change this if Takumi's patch to move mojom files lands first.

[mark a. foltz, 2017-03-30 23:21:05]

Thanks, this looks cleaner.   Looking forward to the day when we can eliminate
the trampoline "hanlder" methods.

https://codereview.chromium.org/2779203003/diff/40001/extensions/renderer/res...
File extensions/renderer/resources/media_router_bindings.js (right):

https://codereview.chromium.org/2779203003/diff/40001/extensions/renderer/res...
extensions/renderer/resources/media_router_bindings.js:597: // TODO(mfoltz):
Remove when component that supports this method is
We should do a pass through these bindings and remove obsolete backwards
compatibility code.
I can take this on :)

[mark a. foltz, 2017-03-30 23:21:07]

lgtm

[imcheng, 2017-03-31 19:32:44]

The CQ bit was checked by imcheng@chromium.org

[commit-bot: I haz the power, 2017-03-31 19:33:40]

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-03-31 20:00:48]

CQ is committing da patch.

Bot data: {"patchset_id": 40001, "attempt_start_ts": 1490988764159630,
"parent_rev": "e23c1b6375d994f06ed745805ba3847e8218cdba", "commit_rev":
"532ac77836647714a0ed93c871b4a0b75159fcaa"}

[commit-bot: I haz the power, 2017-03-31 20:01:43]

Description was changed from

==========
[Media Router] Export Mojom definitions in media_router_bindings.js.

In media_router_bindings, instead of just exporting the mediaRouter
instance to the component, also export definitions of generated Mojom
classes that will be needed by the component. This lets the component
work with Mojo objects directly rather than relying on bindings as an
intermediate layer.

BUG=687365
==========

to

==========
[Media Router] Export Mojom definitions in media_router_bindings.js.

In media_router_bindings, instead of just exporting the mediaRouter
instance to the component, also export definitions of generated Mojom
classes that will be needed by the component. This lets the component
work with Mojo objects directly rather than relying on bindings as an
intermediate layer.

BUG=687365

Review-Url: https://codereview.chromium.org/2779203003
Cr-Commit-Position: refs/heads/master@{#461194}
Committed:
https://chromium.googlesource.com/chromium/src/+/532ac77836647714a0ed93c871b4...
==========

[commit-bot: I haz the power, 2017-03-31 20:01:44]

Committed patchset #3 (id:40001) as
https://chromium.googlesource.com/chromium/src/+/532ac77836647714a0ed93c871b4...