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

Unified Diff: Source/web/WebMediaPlayerClientImpl.cpp

Issue 284513003: Implement AudioTrack, AudioTrackList, VideoTrack, and VideoTrackList (Closed) Base URL: svn://svn.chromium.org/blink/trunk
Patch Set: platform handle per track Created 6 years, 7 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: Source/web/WebMediaPlayerClientImpl.cpp
diff --git a/Source/web/WebMediaPlayerClientImpl.cpp b/Source/web/WebMediaPlayerClientImpl.cpp
index 09635ef423ac7da4eb5cd1becf68b75abdb9aa93..a259e9d7fda904010d8c28a3bc7c8f2f3d150364 100644
--- a/Source/web/WebMediaPlayerClientImpl.cpp
+++ b/Source/web/WebMediaPlayerClientImpl.cpp
@@ -146,6 +146,26 @@ void WebMediaPlayerClientImpl::setWebLayer(blink::WebLayer* layer)
m_client->mediaPlayerSetWebLayer(layer);
}
+void WebMediaPlayerClientImpl::addAudioTrack(uintptr_t handle, const WebString& id, AudioTrackKind kind, const WebString& label, const WebString& language, bool enabled)
+{
+ mediaElement().addAudioTrack(id, kind, label, language, enabled);
acolwell GONE FROM CHROMIUM 2014/05/12 18:23:11 I'm assuming you'll add handle here.
philipj_slow 2014/05/15 14:55:53 Yes, I didn't want to go all the way before the ou
+}
+
+void WebMediaPlayerClientImpl::removeAudioTrack(uintptr_t handle)
+{
+ mediaElement().removeAudioTrack(id);
acolwell GONE FROM CHROMIUM 2014/05/12 18:23:11 s/id/handle/?
+}
+
+void WebMediaPlayerClientImpl::addVideoTrack(uintptr_t handle, const WebString& id, VideoTrackKind kind, const WebString& label, const WebString& language, bool selected)
+{
+ mediaElement().addVideoTrack(id, kind, label, language, selected);
+}
+
+void WebMediaPlayerClientImpl::removeVideoTrack(uintptr_t handle)
+{
+ mediaElement().removeVideoTrack(id);
+}
+
void WebMediaPlayerClientImpl::addTextTrack(WebInbandTextTrack* textTrack)
{
m_client->mediaPlayerDidAddTextTrack(textTrack);

Powered by Google App Engine
This is Rietveld 408576698