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

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: 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 81%
rename from content/browser/android/media_player_manager_android.h
rename to content/browser/android/media_player_manager_impl.h
index c21cb6dfcbae0fc80e3f6b015464e0074833cd4c..7a8e3361c5fb9f3f8956f520b85f6f22368fd542 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>
@@ -15,7 +15,7 @@
#include "content/public/browser/render_view_host_observer.h"
#include "googleurl/src/gurl.h"
#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 {
@@ -27,13 +27,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;
@@ -59,9 +59,10 @@ class MediaPlayerManagerAndroid
void OnError(int player_id, int error);
void OnVideoSizeChanged(int player_id, int width, int height);
- // media::MediaPlayerBridgeManager overrides.
+ // media::MediaPlayerManager 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();
@@ -101,9 +102,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_

Powered by Google App Engine
This is Rietveld 408576698