[Android] Stop all media on undoable close.

content/browser/media/media_web_contents_observer.h

 // Copyright 2014 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.
 
 #ifndef CONTENT_BROWSER_MEDIA_MEDIA_WEB_CONTENTS_OBSERVER_H_
 #define CONTENT_BROWSER_MEDIA_MEDIA_WEB_CONTENTS_OBSERVER_H_
 
 #include "base/compiler_specific.h"
 #include "base/containers/scoped_ptr_hash_map.h"
 #include "content/common/content_export.h"
@@ skipping 40 matching lines @@
 
   // Gets the media player manager associated with |render_frame_host|. Creates
   // a new one if it doesn't exist. The caller doesn't own the returned pointer.
   BrowserMediaPlayerManager* GetMediaPlayerManager(
       RenderFrameHost* render_frame_host);
 
   void OnSetCdm(RenderFrameHost* render_frame_host, int player_id, int cdm_id);
 
   // Pauses all media player.
   void PauseVideo();
+  void ReleaseAllMediaPlayers(RenderFrameHost* render_frame_host);

[qinmin, 2014/08/22 21:49:32]

I think you don't need this call. you can just call
MediaWebContentsObserver::GetMediaPlayerManager() in WebContentsAndroid and then
call BrowserMediaPlayerManager::ReleaseAllMediaPlayers();

[David Trainor- moved to gerrit, 2014/08/25 18:11:43]

Done.

 
 #if defined(VIDEO_HOLE)
   void OnFrameInfoUpdated();
 #endif  // defined(VIDEO_HOLE)
 
 #endif  // defined(OS_ANDROID)
 
  private:
   // Map from RenderFrameHost* to BrowserCdmManager.
   typedef base::ScopedPtrHashMap<uintptr_t, BrowserCdmManager> CdmManagerMap;
   CdmManagerMap cdm_managers_;
 
 #if defined(OS_ANDROID)
   // Map from RenderFrameHost* to BrowserMediaPlayerManager.
   typedef base::ScopedPtrHashMap<uintptr_t, BrowserMediaPlayerManager>
       MediaPlayerManagerMap;
   MediaPlayerManagerMap media_player_managers_;
 #endif  // defined(OS_ANDROID)
 
   DISALLOW_COPY_AND_ASSIGN(MediaWebContentsObserver);
 };
 
 }  // namespace content
 
 #endif  // CONTENT_BROWSER_MEDIA_MEDIA_WEB_CONTENTS_OBSERVER_H_