Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(422)

Unified Diff: content/browser/android/media_player_manager_impl.h

Issue 14314020: Rename MediaPlayerBridgeManager to MediaPlayerManager (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: additional rebase conflict Created 7 years, 8 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: content/browser/android/media_player_manager_impl.h
diff --git a/content/browser/android/media_player_manager_android.h b/content/browser/android/media_player_manager_impl.h
similarity index 84%
rename from content/browser/android/media_player_manager_android.h
rename to content/browser/android/media_player_manager_impl.h
index 7021b69bc38cabba90cddf06afdbca789fb3c9aa..34905df6a4cb7007547edb1db855ebc8bf6f8247 100644
--- a/content/browser/android/media_player_manager_android.h
+++ b/content/browser/android/media_player_manager_impl.h
@@ -2,8 +2,8 @@
// Use of this source code is governed by a BSD-style license that can be
// found in the LICENSE file.
-#ifndef CONTENT_BROWSER_ANDROID_MEDIA_PLAYER_MANAGER_ANDROID_H_
-#define CONTENT_BROWSER_ANDROID_MEDIA_PLAYER_MANAGER_ANDROID_H_
+#ifndef CONTENT_BROWSER_ANDROID_MEDIA_PLAYER_MANAGER_IMPL_H_
+#define CONTENT_BROWSER_ANDROID_MEDIA_PLAYER_MANAGER_IMPL_H_
#include <map>
@@ -18,7 +18,7 @@
#include "media/base/android/demuxer_stream_player_params.h"
#endif
#include "media/base/android/media_player_bridge.h"
-#include "media/base/android/media_player_bridge_manager.h"
+#include "media/base/android/media_player_manager.h"
#include "ui/gfx/rect_f.h"
namespace content {
@@ -30,13 +30,13 @@ class WebContents;
// them to corresponding MediaPlayerBridge object. Callbacks from
// MediaPlayerBridge objects are converted to IPCs and then sent to the
// render process.
-class MediaPlayerManagerAndroid
+class MediaPlayerManagerImpl
: public RenderViewHostObserver,
- public media::MediaPlayerBridgeManager {
+ public media::MediaPlayerManager {
public:
- // Create a MediaPlayerManagerAndroid object for the |render_view_host|.
- explicit MediaPlayerManagerAndroid(RenderViewHost* render_view_host);
- virtual ~MediaPlayerManagerAndroid();
+ // Create a MediaPlayerManagerImpl object for the |render_view_host|.
+ explicit MediaPlayerManagerImpl(RenderViewHost* render_view_host);
+ virtual ~MediaPlayerManagerImpl();
// RenderViewHostObserver overrides.
virtual bool OnMessageReceived(const IPC::Message& message) OVERRIDE;
@@ -71,6 +71,7 @@ class MediaPlayerManagerAndroid
// media::MediaPlayerBridgeManager overrides.
virtual void RequestMediaResources(media::MediaPlayerBridge* player) OVERRIDE;
virtual void ReleaseMediaResources(media::MediaPlayerBridge* player) OVERRIDE;
+ virtual media::MediaResourceGetter* GetMediaResourceGetter() OVERRIDE;
// Release all the players managed by this object.
void DestroyAllMediaPlayers();
@@ -117,9 +118,12 @@ class MediaPlayerManagerAndroid
WebContents* web_contents_;
- DISALLOW_COPY_AND_ASSIGN(MediaPlayerManagerAndroid);
+ // Object for retrieving resources media players.
+ scoped_ptr<media::MediaResourceGetter> media_resource_getter_;
+
+ DISALLOW_COPY_AND_ASSIGN(MediaPlayerManagerImpl);
};
} // namespace content
-#endif // CONTENT_BROWSER_ANDROID_MEDIA_PLAYER_MANAGER_ANDROID_H_
+#endif // CONTENT_BROWSER_ANDROID_MEDIA_PLAYER_MANAGER_IMPL_H_
« no previous file with comments | « content/browser/android/media_player_manager_android.cc ('k') | content/browser/android/media_player_manager_impl.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698