Support text track selection in video controls

Source/core/html/HTMLMediaElement.h

 /*
  * Copyright (C) 2007, 2008, 2009, 2010, 2011, 2012, 2013 Apple 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:
  * 1. Redistributions of source code must retain the above copyright
  *    notice, this list of conditions and the following disclaimer.
  * 2. Redistributions in binary form must reproduce the above copyright
  *    notice, this list of conditions and the following disclaimer in the
@@ skipping 192 matching lines @@
     enum VisibilityChangeAssumption {
         AssumeNoVisibleChange,
         AssumeVisibleChange
     };
     void configureTextTrackDisplay(VisibilityChangeAssumption);
     void updateTextTrackDisplay();
     double lastSeekTime() const { return m_lastSeekTime; }
     void textTrackReadyStateChanged(TextTrack*);
 
     void textTrackModeChanged(TextTrack*);
+    void disableAutomaticTextTrackSelection();
 
     // EventTarget function.
     // Both Node (via HTMLElement) and ActiveDOMObject define this method, which
     // causes an ambiguity error at compile time. This class's constructor
     // ensures that both implementations return document, so return the result
     // of one of them here.
     using HTMLElement::executionContext;
 
     bool hasSingleSecurityOrigin() const { return !m_player || (webMediaPlayer() && webMediaPlayer()->hasSingleSecurityOrigin()); }
 
@@ skipping 315 matching lines @@
 
     // time has not changed since sending an "ended" event
     bool m_sentEndEvent : 1;
 
     bool m_closedCaptionsVisible : 1;
 
     bool m_completelyLoaded : 1;
     bool m_havePreparedToPlay : 1;
     bool m_delayingLoadForPreloadNone : 1;
 
+    bool m_didPerformAutomaticTrackSelection : 1;

[philipj_slow, 2015/05/05 14:36:56]

This name is a bit inaccurate since it can be set to one without actually
performing automatic text track selection. Maybe shouldPerform... would be
better.

However, as I say in another comment, I'm not sure this code is needed at all.

[fs, 2015/05/05 15:32:44]

I believe I suggested this name, with an outset of implementing the flag from
the spec with the same name (only dash-separated of course...) I agree with your
other comment though.

[philipj_slow, 2015/05/06 09:28:43]

Oh, I didn't know about
https://html.spec.whatwg.org/#did-perform-automatic-track-selection or at least
forgot. We should implement that, but in the precise places where the spec says.
With some luck, it might not be needed in this CL at all.

     bool m_tracksAreReady : 1;
     bool m_haveVisibleTextTrack : 1;
     bool m_processingPreferenceChange : 1;
     bool m_remoteRoutesAvailable : 1;
     bool m_playingRemotely : 1;
 #if ENABLE(OILPAN)
     bool m_isFinalizing : 1;
     bool m_closeMediaSourceWhenFinalizing : 1;
 #endif
     bool m_initialPlayWithoutUserGestures : 1;
@@ skipping 25 matching lines @@
 inline bool isHTMLMediaElement(const HTMLElement& element)
 {
     return isHTMLAudioElement(element) || isHTMLVideoElement(element);
 }
 
 DEFINE_HTMLELEMENT_TYPE_CASTS_WITH_FUNCTION(HTMLMediaElement);
 
 } // namespace blink
 
 #endif // HTMLMediaElement_h