[Cast, Android, Presentation API] Implement ReceiverAction messages.

[Cast, Android, Presentation API] Implement ReceiverAction messages.

BUG=561470
TEST=manual with the YT experiment

1. Allow a route with a special presentation id to join a non-existing
session to pass the CAST message back through.
2. Send the CAST message back via the special route before the route is being created with the
selected device info.
3. Send the STOP message via the normal route when the session is about to
close via the CloseRoute() or the notification controls.

Committed: https://crrev.com/e96721f1f8484454c52a23b214a830a2ffa8f4eb
Cr-Commit-Position: refs/heads/master@{#365900}

[mlamouri (slow - plz ping), 2015-12-17 15:45:12]

mlamouri@chromium.org changed reviewers:
+ mlamouri@chromium.org

[mlamouri (slow - plz ping), 2015-12-17 15:45:13]

lgtm

https://codereview.chromium.org/1534783002/diff/1/chrome/android/java/src/org...
File
chrome/android/java/src/org/chromium/chrome/browser/media/router/cast/CastMediaRouteProvider.java
(right):

https://codereview.chromium.org/1534783002/diff/1/chrome/android/java/src/org...
chrome/android/java/src/org/chromium/chrome/browser/media/router/cast/CastMediaRouteProvider.java:452:
if (mSession != null) mSession.clientIds.add(clientId);
nit: could be added after the check that `clientId != null`.

[whywhat, 2015-12-17 16:02:22]

https://codereview.chromium.org/1534783002/diff/1/chrome/android/java/src/org...
File
chrome/android/java/src/org/chromium/chrome/browser/media/router/cast/CastMediaRouteProvider.java
(right):

https://codereview.chromium.org/1534783002/diff/1/chrome/android/java/src/org...
chrome/android/java/src/org/chromium/chrome/browser/media/router/cast/CastMediaRouteProvider.java:452:
if (mSession != null) mSession.clientIds.add(clientId);
On 2015/12/17 at 15:45:13, Mounir Lamouri (slow til Jan) wrote:
> nit: could be added after the check that `clientId != null`.

Both line 445 and 452 add a client id to a set, both depend on the clientId not
being null so I think it's in the right place.
We might want eliminate one of the sets later on though.

[whywhat, 2015-12-17 16:02:26]

The CQ bit was checked by avayvod@chromium.org

[commit-bot: I haz the power, 2015-12-17 16:04:30]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1534783002/1
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1534783002/1

[commit-bot: I haz the power, 2015-12-17 17:44:22]

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

[commit-bot: I haz the power, 2015-12-17 17:44:23]

Try jobs failed on following builders:
  linux_android_rel_ng on tryserver.chromium.android (JOB_FAILED,
https://build.chromium.org/p/tryserver.chromium.android/builders/linux_androi...)

[whywhat, 2015-12-17 17:49:54]

The CQ bit was checked by avayvod@chromium.org

[commit-bot: I haz the power, 2015-12-17 17:50:20]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1534783002/1
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1534783002/1

[commit-bot: I haz the power, 2015-12-17 19:46:08]

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

[commit-bot: I haz the power, 2015-12-17 19:46:09]

Try jobs failed on following builders:
  linux_android_rel_ng on tryserver.chromium.android (JOB_FAILED,
https://build.chromium.org/p/tryserver.chromium.android/builders/linux_androi...)

[whywhat, 2015-12-17 19:59:09]

The CQ bit was checked by avayvod@chromium.org

[commit-bot: I haz the power, 2015-12-17 20:04:03]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1534783002/1
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1534783002/1

[commit-bot: I haz the power, 2015-12-17 21:56:13]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2015-12-17 21:58:15]

Description was changed from

==========
[Cast, Android, Presentation API] Implement ReceiverAction messages.

BUG=561470
TEST=manual with the YT experiment

1. Allow a route with a special presentation id to join a non-existing
session to pass the CAST message back through.
2. Send the CAST message back via the special route before the route is being
created with the
selected device info.
3. Send the STOP message via the normal route when the session is about to
close via the CloseRoute() or the notification controls.
==========

to

==========
[Cast, Android, Presentation API] Implement ReceiverAction messages.

BUG=561470
TEST=manual with the YT experiment

1. Allow a route with a special presentation id to join a non-existing
session to pass the CAST message back through.
2. Send the CAST message back via the special route before the route is being
created with the
selected device info.
3. Send the STOP message via the normal route when the session is about to
close via the CloseRoute() or the notification controls.

Committed: https://crrev.com/e96721f1f8484454c52a23b214a830a2ffa8f4eb
Cr-Commit-Position: refs/heads/master@{#365900}
==========

[commit-bot: I haz the power, 2015-12-17 21:58:17]

Patchset 1 (id:??) landed as
https://crrev.com/e96721f1f8484454c52a23b214a830a2ffa8f4eb
Cr-Commit-Position: refs/heads/master@{#365900}