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

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

Issue 11442056: Add external surface rendering mode (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 8 years 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_android.h
diff --git a/content/browser/android/media_player_manager_android.h b/content/browser/android/media_player_manager_android.h
index 9e7dc4bc8e858902486a05bece92be711e986fc3..fe38164bd23d09f726c29881be51213de9a3493c 100644
--- a/content/browser/android/media_player_manager_android.h
+++ b/content/browser/android/media_player_manager_android.h
@@ -76,14 +76,18 @@ class MediaPlayerManagerAndroid
void OnReleaseResources(int player_id);
void OnDestroyPlayer(int player_id);
+ // Helper methods
+ void ExitFromVideoView(int player_id);
+
// An array of managed players.
ScopedVector<media::MediaPlayerBridge> players_;
// The fullscreen video view object.
ContentVideoView video_view_;
- // Player ID of the fullscreen media player.
- int fullscreen_player_id_;
+ // Player ID of the media player that uses video view.
+ int video_view_player_id_;
+ int video_view_personality_;
ycheo (away) 2012/12/13 05:55:00 IMO, video_view_ has a personality, so this seems
wonsik2 2012/12/17 04:36:05 If we don't track personality of a video view, we
ycheo (away) 2012/12/17 04:43:54 You can get this from ContentVideoView.personality
wonsik2 2012/12/17 05:24:55 Done.
DISALLOW_COPY_AND_ASSIGN(MediaPlayerManagerAndroid);
};

Powered by Google App Engine
This is Rietveld 408576698