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

Unified Diff: media/base/android/media_player_android.h

Issue 302453012: Support casting for embedded YT videos (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Merged parameters into struct to avoid 2 IPC calls, updated nits Created 6 years, 6 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_android.h
diff --git a/media/base/android/media_player_android.h b/media/base/android/media_player_android.h
index cf67bd03a73bba35d91014056f187f9b77bfaa30..bc657b7becb3a1b148441d4eba9c8954392ae5b8 100644
--- a/media/base/android/media_player_android.h
+++ b/media/base/android/media_player_android.h
@@ -60,6 +60,11 @@ class MEDIA_EXPORT MediaPlayerAndroid {
// Set the player volume.
virtual void SetVolume(double volume) = 0;
+ // Sets the url of the frame that contains this player.
+ void set_frame_url(const GURL& frame_url) {
+ frame_url_ = frame_url;
acolwell GONE FROM CHROMIUM 2014/06/09 20:02:09 nit: ISTM this value should be passed in via the c
May 2014/06/10 18:22:20 Done.
+ }
+
// Get the media information from the player.
virtual int GetVideoWidth() = 0;
virtual int GetVideoHeight() = 0;
@@ -85,6 +90,8 @@ class MEDIA_EXPORT MediaPlayerAndroid {
int player_id() { return player_id_; }
+ GURL frame_url() { return frame_url_; }
+
protected:
MediaPlayerAndroid(int player_id,
MediaPlayerManager* manager,
@@ -104,6 +111,9 @@ class MEDIA_EXPORT MediaPlayerAndroid {
// Resource manager for all the media players.
MediaPlayerManager* manager_;
+ // Url for the frame that contains this player.
+ GURL frame_url_;
+
DISALLOW_COPY_AND_ASSIGN(MediaPlayerAndroid);
};

Powered by Google App Engine
This is Rietveld 408576698