[Android,RemotePlayback] Resolve HTMLMediaElement.remote.connect() with false when device is not se…

[Android,RemotePlayback] Resolve HTMLMediaElement.remote.connect() with false when device is not selected.

BUG=578833
TEST=manual

Committed: https://crrev.com/94711d0ac1303d5799a8ff04789e20423e73a05c
Cr-Commit-Position: refs/heads/master@{#384309}

[whywhat, 2016-03-23 21:38:34]

Fixed some spaces

[whywhat, 2016-03-23 21:41:30]

avayvod@chromium.org changed reviewers:
+ aberent@chromium.org, dalecurtis@chromium.org, nasko@chromium.org,
philipj@opera.com

[whywhat, 2016-03-23 21:41:31]

PTaL, mostly straightforward plumbing from RemoteMediaPlayerBridge to
HTMLMediaElement

philipj: HTMLMediaElement.*, WebMediaPlayerClient.h, */remoteplayback/*
dalecurtis: media/blink/*, content/renderer/media/android/*
aberent: */remote/*
nasko: media_player_messages_android.h

[whywhat, 2016-03-23 21:41:38]

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

[commit-bot: I haz the power, 2016-03-23 21:42:13]

Dry run: CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1829923003/20001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1829923003/20001

[DaleCurtis, 2016-03-23 21:49:17]

dalecurtis@chromium.org changed reviewers:
+ hubbe@chromium.org

[DaleCurtis, 2016-03-23 21:49:18]

me => hubbe

[commit-bot: I haz the power, 2016-03-23 21:56:23]

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

[commit-bot: I haz the power, 2016-03-23 21:56:24]

Dry run: Try jobs failed on following builders:
  mac_chromium_compile_dbg_ng on tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/mac_chromium_comp...)
  mac_chromium_rel_ng on tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/mac_chromium_rel_...)

[whywhat, 2016-03-23 22:35:19]

Fixed desktop unittests compile

[whywhat, 2016-03-23 22:38:08]

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

[commit-bot: I haz the power, 2016-03-23 22:38:39]

Dry run: CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1829923003/40001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1829923003/40001

[commit-bot: I haz the power, 2016-03-23 23:08:29]

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

[commit-bot: I haz the power, 2016-03-23 23:08:30]

Dry run: Try jobs failed on following builders:
  linux_blink_oilpan_rel on tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_blink_oil...)

[philipj_slow, 2016-03-24 07:27:15]

third_party/WebKit/ LGTM

[whywhat, 2016-03-24 16:10:58]

Rebased

[whywhat, 2016-03-24 16:11:34]

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

[commit-bot: I haz the power, 2016-03-24 16:11:46]

Dry run: CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1829923003/60001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1829923003/60001

[commit-bot: I haz the power, 2016-03-24 16:25:47]

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

[commit-bot: I haz the power, 2016-03-24 16:25:48]

Dry run: Try jobs failed on following builders:
  mac_chromium_compile_dbg_ng on tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/mac_chromium_comp...)

[whywhat, 2016-03-24 16:48:46]

Another rebase

[nasko, 2016-03-24 21:27:58]

media_player_messages_android.h LGTM

[hubbe, 2016-03-24 21:42:56]

lgtm

[aberent, 2016-03-29 14:51:30]

lgtm

[whywhat, 2016-03-29 18:38:09]

avayvod@chromium.org changed reviewers:
- dalecurtis@chromium.org

[whywhat, 2016-03-29 18:38:09]

Thanks all for the reviews!

[whywhat, 2016-03-31 10:58:39]

Rebased

[whywhat, 2016-03-31 15:51:53]

The CQ bit was checked by avayvod@chromium.org

[whywhat, 2016-03-31 15:51:53]

The patchset sent to the CQ was uploaded after l-g-t-m from philipj@opera.com,
aberent@chromium.org, hubbe@chromium.org, nasko@chromium.org
Link to the patchset: https://codereview.chromium.org/1829923003/#ps100001
(title: "Rebased")

[commit-bot: I haz the power, 2016-03-31 15:52:06]

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

[commit-bot: I haz the power, 2016-03-31 16:57:17]

Committed patchset #6 (id:100001)

[commit-bot: I haz the power, 2016-03-31 17:02:00]

Description was changed from

==========
[Android,RemotePlayback] Resolve HTMLMediaElement.remote.connect() with false
when device is not selected.

BUG=578833
TEST=manual
==========

to

==========
[Android,RemotePlayback] Resolve HTMLMediaElement.remote.connect() with false
when device is not selected.

BUG=578833
TEST=manual

Committed: https://crrev.com/94711d0ac1303d5799a8ff04789e20423e73a05c
Cr-Commit-Position: refs/heads/master@{#384309}
==========

[commit-bot: I haz the power, 2016-03-31 17:02:01]

Patchset 6 (id:??) landed as
https://crrev.com/94711d0ac1303d5799a8ff04789e20423e73a05c
Cr-Commit-Position: refs/heads/master@{#384309}