Implement WebRTC in Chrome for TV

media/base/android/media_player_bridge.h

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 c6db77bcda7faf27a4d4a154ca2557800086f79d..05dd014bd7aa10f1befd8448fccef2d447666b44 100644
--- a/media/base/android/media_player_bridge.h
+++ b/media/base/android/media_player_bridge.h
@@ -44,6 +44,12 @@ class MEDIA_EXPORT MediaPlayerBridge {
     MEDIA_ERROR_INVALID_CODE,
   };
 
+  // Flags for creation
+  enum Flags {
+    FLAG_MEDIA_SOURCE = 1 << 0,

[qinmin, 2013/04/29 17:31:49]

add a default flag here for non media source case.

[wonsik, 2013/05/01 14:15:38]

Done.

+    FLAG_LOW_LATENCY = 1 << 1,

[Ami GONE FROM CHROMIUM, 2013/04/29 22:11:09]

Neither of these names is meaningful to me.
Does MEDIA_SOURCE refer to the W3C MediaSource Extensions spec, or to the
GoogleTV MediaSource API?
What does LOW_LATENCY refer to?  (why would anyone want _high_ latency? ;))
Needs fuller commentary (since these aren't described in the android doco linked
from the class comment) and hopefully a more specific name for MEDIA_SOURCE (off
the top of my head FLAG_MEDIA_SOURCE_EXTENSIONS and
FLAG_GOOGLETV_MEDIA_SOURCE_API make more sense to me, but that's not to say
there aren't better choices)

[wonsik, 2013/05/01 14:15:38]

Improved names.

+  };
+
   // Callback when error happens. Args: player ID, error type.
   typedef base::Callback<void(int, int)> MediaErrorCB;
 
@@ -82,7 +88,7 @@ class MEDIA_EXPORT MediaPlayerBridge {
   static MediaPlayerBridge* Create(
       int player_id,
       const GURL& url,
-      bool is_media_source,
+      int flags,
       const GURL& first_party_for_cookies,
       MediaResourceGetter* resource_getter,
       bool hide_url_log,