[Media Router] Adds a pre-invocation hook to Media Router bindings.

[Media Router] Adds a pre-invocation hook to Media Router bindings.

This hook allows the Media Router component to properly record the cause of
event page wakeups that are caused by invoking the Media Route Provider API.

BUG=

Committed: https://crrev.com/8e100e892b381a7088ad6c48e6c6fd11acf9a246
Cr-Commit-Position: refs/heads/master@{#427097}

[mark a. foltz, 2016-10-20 17:47:58]

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

[mark a. foltz, 2016-10-20 17:47:59]

Will wait to commit until a component that supports onBeforeInvokeHandler is
available in canary, then I revert the check at L548.

[imcheng, 2016-10-20 17:56:35]

lgtm

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

https://codereview.chromium.org/2422773004/diff/20001/extensions/renderer/res...
extensions/renderer/resources/media_router_bindings.js:867: sinkId, sourceUrn,
searchCriteria) {
nit: revert this line?

[mark a. foltz, 2016-10-24 17:32:03]

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

https://codereview.chromium.org/2422773004/diff/20001/extensions/renderer/res...
extensions/renderer/resources/media_router_bindings.js:867: sinkId, sourceUrn,
searchCriteria) {
On 2016/10/20 at 17:56:35, imcheng wrote:
> nit: revert this line?

Done.

[mark a. foltz, 2016-10-24 17:32:11]

The CQ bit was checked by mfoltz@chromium.org

[mark a. foltz, 2016-10-24 17:32:12]

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/2422773004/#ps40001
(title: "Update comment & respond to imcheng@")

[commit-bot: I haz the power, 2016-10-24 17:32:42]

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-10-24 18:07:02]

Committed patchset #3 (id:40001)

[commit-bot: I haz the power, 2016-10-24 18:22:40]

Description was changed from

==========
[Media Router] Adds a pre-invocation hook to Media Router bindings.

This hook allows the Media Router component to properly record the cause of
event page wakeups that are caused by invoking the Media Route Provider API.

BUG=
==========

to

==========
[Media Router] Adds a pre-invocation hook to Media Router bindings.

This hook allows the Media Router component to properly record the cause of
event page wakeups that are caused by invoking the Media Route Provider API.

BUG=

Committed: https://crrev.com/8e100e892b381a7088ad6c48e6c6fd11acf9a246
Cr-Commit-Position: refs/heads/master@{#427097}
==========

[commit-bot: I haz the power, 2016-10-24 18:22:41]

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