Fix the loop behavior on android

Fix the loop behavior on android

During loop, player should not be in a paused state
This change fix that behavior

Layout test:https://codereview.chromium.org/585563003/

BUG=415651
Committed: https://crrev.com/5f328dba01c4ed670a0e4a3d222545c52561a5dd
Cr-Commit-Position: refs/heads/master@{#295585}

[qinmin, 2014-09-18 18:05:47]

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

[qinmin, 2014-09-18 18:05:47]

PTAL

[DaleCurtis, 2014-09-18 18:15:33]

https://codereview.chromium.org/580093005/diff/1/content/renderer/media/andro...
File content/renderer/media/android/webmediaplayer_android.cc (right):

https://codereview.chromium.org/580093005/diff/1/content/renderer/media/andro...
content/renderer/media/android/webmediaplayer_android.cc:779:
player_manager_->Start(player_id_);
Isn't the player already started?

[qinmin, 2014-09-18 18:21:21]

https://codereview.chromium.org/580093005/diff/1/content/renderer/media/andro...
File content/renderer/media/android/webmediaplayer_android.cc (right):

https://codereview.chromium.org/580093005/diff/1/content/renderer/media/andro...
content/renderer/media/android/webmediaplayer_android.cc:779:
player_manager_->Start(player_id_);
On 2014/09/18 18:15:33, DaleCurtis wrote:
> Isn't the player already started?

when android mediaplayer reaches the end, it will automatically pause. This is
to kick the player to play again.

[DaleCurtis, 2014-09-18 18:23:33]

https://codereview.chromium.org/580093005/diff/1/content/renderer/media/andro...
File content/renderer/media/android/webmediaplayer_android.cc (right):

https://codereview.chromium.org/580093005/diff/1/content/renderer/media/andro...
content/renderer/media/android/webmediaplayer_android.cc:779:
player_manager_->Start(player_id_);
On 2014/09/18 18:21:21, qinmin wrote:
> On 2014/09/18 18:15:33, DaleCurtis wrote:
> > Isn't the player already started?
> 
> when android mediaplayer reaches the end, it will automatically pause. This is
> to kick the player to play again.

Is it okay to do this before the seek completes?

[qinmin, 2014-09-18 18:25:33]

On 2014/09/18 18:23:33, DaleCurtis wrote:
>
https://codereview.chromium.org/580093005/diff/1/content/renderer/media/andro...
> File content/renderer/media/android/webmediaplayer_android.cc (right):
> 
>
https://codereview.chromium.org/580093005/diff/1/content/renderer/media/andro...
> content/renderer/media/android/webmediaplayer_android.cc:779:
> player_manager_->Start(player_id_);
> On 2014/09/18 18:21:21, qinmin wrote:
> > On 2014/09/18 18:15:33, DaleCurtis wrote:
> > > Isn't the player already started?
> > 
> > when android mediaplayer reaches the end, it will automatically pause. This
is
> > to kick the player to play again.
> 
> Is it okay to do this before the seek completes?

Yes, the play command will be executed once seek complete.

[DaleCurtis, 2014-09-18 18:32:58]

lgtm

[qinmin, 2014-09-18 19:42:36]

The CQ bit was checked by qinmin@chromium.org

[commit-bot: I haz the power, 2014-09-18 19:44:05]

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

[commit-bot: I haz the power, 2014-09-18 19:47:31]

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

[commit-bot: I haz the power, 2014-09-18 19:47:32]

Try jobs failed on following builders:
  mac_gpu on tryserver.chromium.gpu
(http://build.chromium.org/p/tryserver.chromium.gpu/builders/mac_gpu/builds/57832)
  ios_dbg_simulator on tryserver.chromium.mac
(http://build.chromium.org/p/tryserver.chromium.mac/builders/ios_dbg_simulator...)
  ios_rel_device on tryserver.chromium.mac
(http://build.chromium.org/p/tryserver.chromium.mac/builders/ios_rel_device/bu...)
  ios_rel_device_ninja on tryserver.chromium.mac
(http://build.chromium.org/p/tryserver.chromium.mac/builders/ios_rel_device_ni...)
  mac_chromium_compile_dbg on tryserver.chromium.mac
(http://build.chromium.org/p/tryserver.chromium.mac/builders/mac_chromium_comp...)
  mac_chromium_rel_swarming on tryserver.chromium.mac
(http://build.chromium.org/p/tryserver.chromium.mac/builders/mac_chromium_rel_...)
  win8_chromium_rel on tryserver.chromium.win
(http://build.chromium.org/p/tryserver.chromium.win/builders/win8_chromium_rel...)
  win_chromium_x64_rel_swarming on tryserver.chromium.win
(http://build.chromium.org/p/tryserver.chromium.win/builders/win_chromium_x64_...)

[qinmin, 2014-09-18 21:31:14]

The CQ bit was checked by qinmin@chromium.org

[commit-bot: I haz the power, 2014-09-18 21:35:31]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/580093005/20001

[commit-bot: I haz the power, 2014-09-18 22:44:18]

Committed patchset #2 (id:20001) as 2314bc55e0c58e3a83e223fdc4aaeae388b57593

[commit-bot: I haz the power, 2014-09-18 22:44:55]

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