Simplify HTMLMediaElement::textTrackModeChanged

Source/core/html/HTMLMediaElement.cpp

Index: Source/core/html/HTMLMediaElement.cpp
diff --git a/Source/core/html/HTMLMediaElement.cpp b/Source/core/html/HTMLMediaElement.cpp
index 23ba1efbfe7eab84dee7b489679d6b4c1a681d4e..7435a5095c4ae7576f0d677ea5e4d9d6d29b8c8e 100644
--- a/Source/core/html/HTMLMediaElement.cpp
+++ b/Source/core/html/HTMLMediaElement.cpp
@@ -1151,32 +1151,9 @@ void HTMLMediaElement::textTrackReadyStateChanged(TextTrack* track)
 
 void HTMLMediaElement::textTrackModeChanged(TextTrack* track)
 {
-    if (track->trackType() == TextTrack::TrackElement) {
-        // 4.8.10.12.3 Sourcing out-of-band text tracks
-        // ... when a text track corresponding to a track element is created with text track
-        // mode set to disabled and subsequently changes its text track mode to hidden, showing,
-        // or showing by default for the first time, the user agent must immediately and synchronously
-        // run the following algorithm ...
-
-        for (HTMLTrackElement* trackElement = Traversal<HTMLTrackElement>::firstChild(*this); trackElement; trackElement = Traversal<HTMLTrackElement>::nextSibling(*trackElement)) {
-            if (trackElement->track() != track)
-                continue;
-
-            // Mark this track as "configured" so configureTextTracks won't change the mode again.
-            track->setHasBeenConfigured(true);
-            if (track->mode() != TextTrack::disabledKeyword()) {
-                if (trackElement->readyState() == HTMLTrackElement::LOADED)
-                    cueTimeline().addCues(track, track->cues());
-
-                // If this is the first added track, create the list of text tracks.

[philipj_slow, 2015/02/25 15:52:14]

Was this just dead code?

[fs, 2015/02/25 16:26:11]

Yes. (This method was only called from within TextTrack::setMode, and only if
mediaElement(), and the latter is "m_trackList ? m_trackList->owner() : 0;" -
i.e. it's hard to send this notification if there was no list in the first
place... Probably some old iteration left behind.)

-                if (!m_textTracks)
-                    m_textTracks = TextTrackList::create(this);
-            }
-            break;
-        }
-    } else if (track->trackType() == TextTrack::AddTrack && track->mode() != TextTrack::disabledKeyword()) {
-        cueTimeline().addCues(track, track->cues());
-    }
+    // Mark this track as "configured" so configureTextTracks won't change the mode again.
+    if (track->trackType() == TextTrack::TrackElement)
+        track->setHasBeenConfigured(true);
 
     configureTextTrackDisplay(AssumeVisibleChange);