[RemotePlayback] Implement 'connecting' state

[RemotePlayback] Implement 'connecting' state

BUG=660408
TEST=manual

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

[whywhat, 2016-11-02 22:51:19]

avayvod@chromium.org changed reviewers:
+ hubbe@chromium.org, mlamouri@chromium.org

[whywhat, 2016-11-02 22:51:20]

PTaL

hubbe: media/blink
mlamouri: remote/ WebKit/

[whywhat, 2016-11-02 22:51:26]

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

[commit-bot: I haz the power, 2016-11-02 22:52:13]

Dry run: 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-11-02 23:03:16]

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

[commit-bot: I haz the power, 2016-11-02 23:03:17]

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

[whywhat, 2016-11-03 00:06:04]

Fixed mocks

[whywhat, 2016-11-03 00:07:11]

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

[commit-bot: I haz the power, 2016-11-03 00:07:57]

Dry run: 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-11-03 02:09:44]

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

[commit-bot: I haz the power, 2016-11-03 02:09:44]

Dry run: This issue passed the CQ dry run.

[mlamouri (slow - plz ping), 2016-11-03 16:17:48]

lgtm but as for https://codereview.chromium.org/2468303002, we need tests for
this

https://codereview.chromium.org/2464163004/diff/20001/content/renderer/media/...
File content/renderer/media/android/webmediaplayer_android.cc (right):

https://codereview.chromium.org/2464163004/diff/20001/content/renderer/media/...
content/renderer/media/android/webmediaplayer_android.cc:882: }
Nice of you to think about our old friend WMPA :)

[whywhat, 2016-11-04 15:32:06]

avayvod@chromium.org changed reviewers:
+ dcheng@chromium.org

[whywhat, 2016-11-04 15:32:07]

hubbe, please review *media*
+dcheng for media_player_messages_android.h

[whywhat, 2016-11-04 16:45:41]

Rebased

[dcheng, 2016-11-04 17:14:17]

ipc lgtm

[whywhat, 2016-11-04 23:17:47]

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

[commit-bot: I haz the power, 2016-11-04 23:18:45]

Dry run: 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-11-05 02:46:33]

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

[commit-bot: I haz the power, 2016-11-05 02:46:34]

Dry run: This issue passed the CQ dry run.

[hubbe, 2016-11-05 03:33:14]

media lgtm

[whywhat, 2016-11-06 03:16:20]

The CQ bit was checked by avayvod@chromium.org

[whywhat, 2016-11-06 03:16:21]

The patchset sent to the CQ was uploaded after l-g-t-m from
mlamouri@chromium.org
Link to the patchset: https://codereview.chromium.org/2464163004/#ps40001
(title: "Rebased")

[commit-bot: I haz the power, 2016-11-06 03:16:29]

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-11-06 04:32:27]

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

[commit-bot: I haz the power, 2016-11-06 04:32:28]

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

[whywhat, 2016-11-07 21:26:14]

The CQ bit was checked by avayvod@chromium.org

[commit-bot: I haz the power, 2016-11-07 21:26: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-11-07 22:08:31]

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

[commit-bot: I haz the power, 2016-11-07 22:08:32]

Try jobs failed on following builders:
  ios-device on master.tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/ios-device/builds...)

[whywhat, 2016-11-07 23:52:18]

The CQ bit was checked by avayvod@chromium.org

[commit-bot: I haz the power, 2016-11-07 23:52:55]

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-11-08 00:00:42]

Committed patchset #3 (id:40001)

[commit-bot: I haz the power, 2016-11-08 00:10:14]

Description was changed from

==========
[RemotePlayback] Implement 'connecting' state

BUG=660408
TEST=manual
==========

to

==========
[RemotePlayback] Implement 'connecting' state

BUG=660408
TEST=manual

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

[commit-bot: I haz the power, 2016-11-08 00:10:16]

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