Fix incorrect WebMediaPlayerDelegate notifications.

content/renderer/media/android/webmediaplayer_android.cc

 // Copyright 2013 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "content/renderer/media/android/webmediaplayer_android.h"
 
 #include <limits>
 
 #include "base/android/build_info.h"
 #include "base/bind.h"
@@ skipping 1271 matching lines @@
 
 void WebMediaPlayerAndroid::SetNeedsEstablishPeer(bool needs_establish_peer) {
   needs_establish_peer_ = needs_establish_peer;
 }
 
 void WebMediaPlayerAndroid::setPoster(const blink::WebURL& poster) {
   player_manager_->SetPoster(player_id_, 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 only call this on the
transition? Either that or just put a "if (is_playing == is_playing_) return;"
at the top of the method ?

[DaleCurtis, 2014/09/11 19:28:01]

Done.

     delegate_->DidPause(this);
 }
 
 #if defined(VIDEO_HOLE)
 bool WebMediaPlayerAndroid::UpdateBoundaryRectangle() {
   if (!video_weblayer_)
     return false;
 
   // Compute the geometry of video frame layer.
   cc::Layer* layer = video_weblayer_->layer();
@@ skipping 461 matching lines @@
 
 bool WebMediaPlayerAndroid::IsHLSStream() const {
   std::string mime;
   GURL url = redirected_url_.is_empty() ? url_ : redirected_url_;
   if (!net::GetMimeTypeFromFile(base::FilePath(url.path()), &mime))
     return false;
   return !mime.compare("application/x-mpegurl");
 }
 
 }  // namespace content