Chromium Code Reviews| Index: content/renderer/media/android/webmediaplayer_android.cc |
| diff --git a/content/renderer/media/android/webmediaplayer_android.cc b/content/renderer/media/android/webmediaplayer_android.cc |
| index 22abbade2bcdcd31ceb5b601496923660c435c85..99e31747b12a8945e38d5d74332d52087cd51310 100644 |
| --- a/content/renderer/media/android/webmediaplayer_android.cc |
| +++ b/content/renderer/media/android/webmediaplayer_android.cc |
| @@ -296,6 +296,11 @@ void WebMediaPlayerAndroid::DidLoadMediaInfo( |
| UpdateReadyState(WebMediaPlayer::ReadyStateHaveMetadata); |
| UpdateReadyState(WebMediaPlayer::ReadyStateHaveEnoughData); |
| } |
| + // Android doesn't start fetching resources until an implementation-defined |
| + // event (e.g. playback request) occurs. Sets to the network state to IDLE |
| + // if play is not requested yet. |
| + if (paused()) |
| + UpdateNetworkState(WebMediaPlayer::NetworkStateIdle); |
|
scherkus (not reviewing)
2013/09/25 00:23:54
this still isn't technically correct
for example,
Jinsuk Kim
2013/09/25 01:42:11
That's true. I think I need a new flag rather than
|
| } |
| void WebMediaPlayerAndroid::play() { |
| @@ -314,6 +319,7 @@ void WebMediaPlayerAndroid::play() { |
| if (paused()) |
| proxy_->Start(player_id_); |
| UpdatePlayingState(true); |
| + UpdateNetworkState(WebMediaPlayer::NetworkStateLoading); |
| } |
| void WebMediaPlayerAndroid::pause() { |