Throttle media decoding after excessive Android media server crashes

content/browser/media/android/browser_media_player_manager.h

 // 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.
 
 #ifndef CONTENT_BROWSER_MEDIA_ANDROID_BROWSER_MEDIA_PLAYER_MANAGER_H_
 #define CONTENT_BROWSER_MEDIA_ANDROID_BROWSER_MEDIA_PLAYER_MANAGER_H_
 
+#include <map>
+
 #include "base/basictypes.h"
 #include "base/callback.h"
 #include "base/memory/scoped_ptr.h"
 #include "base/memory/scoped_vector.h"
 #include "base/time/time.h"
 #include "content/browser/android/content_video_view.h"
 #include "content/browser/media/android/media_session_observer.h"
 #include "content/common/content_export.h"
 #include "content/common/media/media_player_messages_enums_android.h"
 #include "content/public/browser/android/content_view_core.h"
@@ skipping 135 matching lines @@
   // Helper function to send messages to RenderFrameObserver.
   bool Send(IPC::Message* msg);
 
  private:
   // Constructs a MediaPlayerAndroid object.
   media::MediaPlayerAndroid* CreateMediaPlayer(
       const MediaPlayerHostMsg_Initialize_Params& media_player_params,
       bool hide_url_log,
       BrowserDemuxerAndroid* demuxer);
 
-  // MediaPlayerAndroid must call this before it is going to decode
-  // media streams. This helps the manager object maintain an array
-  // of active MediaPlayerAndroid objects and release the resources
-  // when needed. Currently we only count video resources as they are
-  // constrained by hardware and memory limits.
-  virtual void OnMediaResourcesRequested(int player_id);
+  // Called to request decoder resources. Returns true if the request is
+  // permitted, or false otherwise. The manager object maintains an array
+  // of active MediaPlayerAndroid objects and releases the inactive resources
+  // when needed. If |temporary| is true, the request is short lived
+  // and it will not be cleaned up when handling other requests.
+  // On the contrary, requests with false |temporary| value are subject to
+  // clean up if their players are idle.
+  virtual bool RequestDecoderResources(int player_id, bool temporary);

[ctserng, 2015/09/30 17:41:11]

Can we make RequestDecoderResources protected so that it'll be possible to
implement a derived class that uses the same base resource manager.

[qinmin, 2015/09/30 19:27:42]

Done.

 
-  // Called when a player releases all decoding resources.
-  void ReleaseMediaResources(int player_id);
+  // MediaPlayerAndroid must call this to inform the manager that it has
+  // released the decoder resources. This can be triggered by the
+  // ReleasePlayer() call below, or when meta data is extracted, or when player
+  // is stuck in an error.
+  virtual void OnDecorderResourcesReleased(int player_id);

[ctserng, 2015/09/30 17:41:10]

/s/Decorder/Decoder

Also can we make OnDecoderResourcesReleased() protected so that it'll be
possible to implement a derived class that uses the same base resource manager.

[qinmin, 2015/09/30 19:27:42]

Done.

 
-  // Releases the player. However, don't remove it from |players_|.
+  // Instructs |player| to release its java player. This will not remove the
+  // player from |players_|.
   void ReleasePlayer(media::MediaPlayerAndroid* player);
 
+  // Called when user approves media playback after being throttled.
+  void OnPlaybackPermissionGranted(int player_id, bool granted);
+
+  // Helper method to start playback.
+  void StartInternal(int player_id);
+
 #if defined(VIDEO_HOLE)
   void ReleasePlayerOfExternalVideoSurfaceIfNeeded(int future_player);
   void OnRequestExternalSurface(int player_id, const gfx::RectF& rect);
+  void ReleaseExternalSurface(int player_id);
 #endif  // defined(VIDEO_HOLE)
 
   RenderFrameHost* const render_frame_host_;
 
   // An array of managed players.
   ScopedVector<media::MediaPlayerAndroid> players_;
 
+  typedef std::map<int, bool> ActivePlayerMap;
+  // Players that have requested decoding resources. Even though resource is
+  // requested, a player may be in a paused or error state and the manager
+  // will release its resources later.
+  ActivePlayerMap active_players_;
+
   // The fullscreen video view object or NULL if video is not played in
   // fullscreen.
   scoped_ptr<ContentVideoView> video_view_;
 
 #if defined(VIDEO_HOLE)
   scoped_ptr<ExternalVideoSurfaceContainer> external_video_surface_container_;
 #endif
 
   // Player ID of the fullscreen media player.
   int fullscreen_player_id_;
 
   // Whether the fullscreen player has been Release()-d.
   bool fullscreen_player_is_released_;
 
   WebContents* const web_contents_;
 
   // Object for retrieving resources media players.
   scoped_ptr<media::MediaResourceGetter> media_resource_getter_;
 
   // NOTE: Weak pointers must be invalidated before all other member variables.
   base::WeakPtrFactory<BrowserMediaPlayerManager> weak_ptr_factory_;
 
   DISALLOW_COPY_AND_ASSIGN(BrowserMediaPlayerManager);
 };
 
 }  // namespace content
 
 #endif  // CONTENT_BROWSER_MEDIA_ANDROID_BROWSER_MEDIA_PLAYER_MANAGER_H_