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

Unified Diff: media/base/android/media_player_bridge.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: media/base/android/media_player_bridge.h
diff --git a/media/base/android/media_player_bridge.h b/media/base/android/media_player_bridge.h
index 1287de4a232e068cea564c4006fa78d2b43d4b35..755fd76977c9e293b16c9840a042b28c448995a5 100644
--- a/media/base/android/media_player_bridge.h
+++ b/media/base/android/media_player_bridge.h
@@ -21,8 +21,7 @@
namespace media {
-class MediaResourceGetter;
-class MediaPlayerBridgeManager;
+class MediaPlayerManager;
// This class serves as a bridge for native code to call java functions inside
// android mediaplayer class. For more information on android mediaplayer, check
@@ -78,9 +77,8 @@ class MEDIA_EXPORT MediaPlayerBridge {
MediaPlayerBridge(int player_id,
const GURL& url,
const GURL& first_party_for_cookies,
- MediaResourceGetter* resource_getter,
bool hide_url_log,
- MediaPlayerBridgeManager* manager,
+ MediaPlayerManager* manager,
const MediaErrorCB& media_error_cb,
const VideoSizeChangedCB& video_size_changed_cb,
const BufferingUpdateCB& buffering_update_cb,
@@ -218,10 +216,7 @@ class MEDIA_EXPORT MediaPlayerBridge {
std::string cookies_;
// Resource manager for all the media players.
- MediaPlayerBridgeManager* manager_;
-
- // Object for retrieving resources for this media player.
- scoped_ptr<MediaResourceGetter> resource_getter_;
+ MediaPlayerManager* manager_;
// Java MediaPlayer instance.
base::android::ScopedJavaGlobalRef<jobject> j_media_player_;

Powered by Google App Engine
This is Rietveld 408576698