4.8.10.12.1 Text track model http://www.whatwg.org/specs/web-apps/current-work/#text-track-cue-active-flag The active flag … The user agent must synchronously unset this flag whenever the text track cue is removed from its text track's text track list of cues; whenever the text track itself is removed from its media element's list of text tracks or has its text track mode changed to disabled; and whenever the media element's readyState is changed back to HAVE_NOTHING. When the flag is unset in this way for one or more cues in text tracks that were showing or showing by default prior to the relevant incident, the user agent must, after having unset the flag for all the affected cues, apply the rules for updating the text track rendering of those text tracks. For example, for text tracks based on WebVTT, the rules for updating the display of WebVTT text tracks.
<rdar://problem/10464476>
Created attachment 125930 [details] Proposed patch
We already clear a cue's active flag when it is removed from the track, and existing tests check that. We were not clearing active flags when the media element's readyState is set to HAVE_NOTHING, this patch does that.
Comment on attachment 125930 [details] Proposed patch Attachment 125930 [details] did not pass chromium-ews (chromium-xvfb): Output: http://queues.webkit.org/results/11460118 New failing tests: media/track/track-cue-mutable-text.html media/track/track-active-cues.html
Created attachment 126357 [details] Updated patch.
Comment on attachment 126357 [details] Updated patch. Attachment 126357 [details] did not pass chromium-ews (chromium-xvfb): Output: http://queues.webkit.org/results/11480402 New failing tests: media/track/track-cue-mutable-text.html
Comment on attachment 126357 [details] Updated patch. Code change looks good to me, but please check why media/track/track-cue-mutable-text.html is failing on cr-linux. Maybe it just needs to be skipped. Conditional r+ assuming that test failure can be fixed.
http://trac.webkit.org/changeset/107831