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 048994912aa7377092e315f14cfd0e39924acb5b..46619e363b65cb41daaf058e8602d41577f94f80 100644 |
| --- a/content/renderer/media/android/webmediaplayer_android.cc |
| +++ b/content/renderer/media/android/webmediaplayer_android.cc |
| @@ -1289,12 +1289,13 @@ void WebMediaPlayerAndroid::setPoster(const blink::WebURL& poster) { |
| } |
| void WebMediaPlayerAndroid::UpdatePlayingState(bool is_playing) { |
| + const bool was_playing = is_playing_; |
| is_playing_ = is_playing; |
| if (!delegate_) |
| return; |
| - if (is_playing) |
| + if (!was_playing && is_playing) |
| delegate_->DidPlay(this); |
| - else |
| + else if (was_playing) |
|
acolwell GONE FROM CHROMIUM
2014/09/11 19:23:25
nit: Don't you need && !is_playing_ here so you on
DaleCurtis
2014/09/11 19:28:01
Done.
|
| delegate_->DidPause(this); |
| } |