Fix media_router_bindings.js for binary messaging.

Fix media_router_bindings.js for binary messaging.

Mojo JS codegen provides binary data in objects of type Array<number>,
not Uint8Array. This change fixes the assumption, and provides the
required Uint8Array to the provider handler function.

Committed: https://crrev.com/6582d0b551556bb7793daf855d994c2ec864ddb3
Cr-Commit-Position: refs/heads/master@{#438025}

[miu, 2016-12-13 01:27:17]

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

[miu, 2016-12-13 01:27:18]

imcheng: PTAL.

[imcheng, 2016-12-13 01:40:34]

lgtm

https://codereview.chromium.org/2568933003/diff/1/extensions/renderer/resourc...
File extensions/renderer/resources/media_router_bindings.js (right):

https://codereview.chromium.org/2568933003/diff/1/extensions/renderer/resourc...
extensions/renderer/resources/media_router_bindings.js:719: * @param {!Array}
data
Should this be !Array<number> ?

[miu, 2016-12-13 03:11:23]

https://codereview.chromium.org/2568933003/diff/1/extensions/renderer/resourc...
File extensions/renderer/resources/media_router_bindings.js (right):

https://codereview.chromium.org/2568933003/diff/1/extensions/renderer/resourc...
extensions/renderer/resources/media_router_bindings.js:719: * @param {!Array}
data
On 2016/12/13 01:40:34, imcheng wrote:
> Should this be !Array<number> ?

Done.

[miu, 2016-12-13 03:11:26]

The CQ bit was checked by miu@chromium.org

[miu, 2016-12-13 03:11:26]

The patchset sent to the CQ was uploaded after l-g-t-m from imcheng@chromium.org
Link to the patchset: https://codereview.chromium.org/2568933003/#ps20001
(title: "!Array<number>")

[commit-bot: I haz the power, 2016-12-13 03:11:49]

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, 2016-12-13 03:27:14]

CQ is committing da patch.

Bot data: {"patchset_id": 20001, "attempt_start_ts": 1481598686294120,
"parent_rev": "0f4b430a5fda7107267bd389f65f3184e45d23cc", "commit_rev":
"ef2b148d858dcc4545f956dbd629fee8d3bc530e"}

[commit-bot: I haz the power, 2016-12-13 03:27:45]

Description was changed from

==========
Fix media_router_bindings.js for binary messaging.

Mojo JS codegen provides binary data in objects of type Array<number>,
not Uint8Array. This change fixes the assumption, and provides the
required Uint8Array to the provider handler function.
==========

to

==========
Fix media_router_bindings.js for binary messaging.

Mojo JS codegen provides binary data in objects of type Array<number>,
not Uint8Array. This change fixes the assumption, and provides the
required Uint8Array to the provider handler function.

Review-Url: https://codereview.chromium.org/2568933003
==========

[commit-bot: I haz the power, 2016-12-13 03:27:46]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-12-13 03:29:35]

Description was changed from

==========
Fix media_router_bindings.js for binary messaging.

Mojo JS codegen provides binary data in objects of type Array<number>,
not Uint8Array. This change fixes the assumption, and provides the
required Uint8Array to the provider handler function.

Review-Url: https://codereview.chromium.org/2568933003
==========

to

==========
Fix media_router_bindings.js for binary messaging.

Mojo JS codegen provides binary data in objects of type Array<number>,
not Uint8Array. This change fixes the assumption, and provides the
required Uint8Array to the provider handler function.

Committed: https://crrev.com/6582d0b551556bb7793daf855d994c2ec864ddb3
Cr-Commit-Position: refs/heads/master@{#438025}
==========

[commit-bot: I haz the power, 2016-12-13 03:29:36]

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