Implement AudioTrack, AudioTrackList, VideoTrack, and VideoTrackList

public/platform/WebMediaPlayer.h

 /*
  * Copyright (C) 2009 Google Inc. All rights reserved.
  *
  * Redistribution and use in source and binary forms, with or without
  * modification, are permitted provided that the following conditions are
  * met:
  *
  *     * Redistributions of source code must retain the above copyright
  * notice, this list of conditions and the following disclaimer.
  *     * Redistributions in binary form must reproduce the above
@@ skipping 30 matching lines @@
 class WebAudioSourceProvider;
 class WebAudioSourceProviderClient;
 class WebContentDecryptionModule;
 class WebMediaPlayerClient;
 class WebString;
 class WebURL;
 struct WebRect;
 struct WebSize;
 class WebGraphicsContext3D;
 
+typedef unsigned WebTrackId;

[acolwell GONE FROM CHROMIUM, 2014/05/16 19:47:56]

nit: I'm a little nervous about putting this in the global Blink namespace. How
about moving this to WebMediaPlayer::TrackId?

+
 class WebMediaPlayer {
 public:
     enum NetworkState {
         NetworkStateEmpty,
         NetworkStateIdle,
         NetworkStateLoading,
         NetworkStateLoaded,
         NetworkStateFormatError,
         NetworkStateNetworkError,
         NetworkStateDecodeError,
@@ skipping 94 matching lines @@
     virtual void setContentDecryptionModule(WebContentDecryptionModule* cdm) { }
 
     // Sets the poster image URL.
     virtual void setPoster(const WebURL& poster) { }
 
     // Instruct WebMediaPlayer to enter/exit fullscreen.
     virtual void enterFullscreen() { }
     virtual void exitFullscreen() { }
     // Returns true if the player can enter fullscreen.
     virtual bool canEnterFullscreen() const { return false; }
+
+    virtual void enabledAudioTracksChanged(const WebVector<WebTrackId>& enabledTrackIds) { }
+    virtual void selectedVideoTrackChanged(WebTrackId* selectedTrackId) { }

[acolwell GONE FROM CHROMIUM, 2014/05/16 19:47:56]

docs.

[landell, 2014/05/19 13:50:28]

Yes, I am especially interested to know why this is a pointer. For instance,
could this be NULL and what would that mean?

[philipj_slow, 2014/05/19 14:06:45]

I had to find a way to say that no video track at all is selected, which the DOM
API makes possible. For the audio tracks that's an empty list, but for video the
backend needs to be given 0 or 1 tracks, and this is one way of doing it.

Other than the lack of documentation, does this seem OK?

 };
 
 } // namespace blink
 
 #endif