1048120 - Cannot hide caption of video

Status: RESOLVED WORKSFORME

(Websites Graveyard :: mozilla.jp, defect)

Description

[Alice0775 White]

[Tracking Requested - why for this release]: Regression since Firefox31

Steps To Reproduce:
1. Open http://www.mozilla.jp/about/mozilla/
2. Play video
3. Click あ/A icon and Select なし to hide caption

Actual Results:
caption displays

Expected Results:
caption should disappear

Comment 1

[Alice0775 White]

media.webvtt.enabled  = false helps

Regression window
Good:
http://hg.mozilla.org/integration/mozilla-inbound/rev/07e357311cf3
Mozilla/5.0 (Windows NT 6.1; WOW64; rv:28.0) Gecko/20100101 Firefox/28.0 ID:20131202095303
Bad:
http://hg.mozilla.org/integration/mozilla-inbound/rev/76e799f77069
Mozilla/5.0 (Windows NT 6.1; WOW64; rv:28.0) Gecko/20100101 Firefox/28.0 ID:20131202101149
Pushlog:
http://hg.mozilla.org/integration/mozilla-inbound/pushloghtml?fromchange=07e357311cf3&tochange=76e799f77069

Regressed by: 
76e799f77069	Ralph Giles — Bug 887978 - Enable WebVTT. r=cpearce, bz New global interfaces need to be added to test_interfaces.html and marked with the same conditional as the pref. The test must also be updated if the conditional is changed on the pref. We now pass the track element portion of the imptests.

Disabled Bug 887978 by Bug 981280 in Firefox28, 29 and 30 but not Firefox31+

Comment 2

[Alice0775 White]

[Tracking Requested - why for this release]:

Comment 3

[Alexander Farkas]

This is not a bug.

If I run the following code: 

``
$('video').prop('textTracks')[0].mode = 'disabled';
``

The textTrack is hidden as expected. As it turns out the JS of this site is just bad. If you look into the JS it reveals the problem (i.e: http://www.mozilla.jp/static/scripts/mj/global-20140512.js at line 1026-1034):

``
          if (native_track_supported) {
            // TODO: HTML5 TextTrack 対応
          } else {
            if (!$(this).attr('lang')) {
              $('#video-subtitles p').aria('hidden', true);
            } else {
              $('#video-subtitles p[lang="' + $(this).attr('lang') + '"]').aria('hidden', false);
            }
          }
``

Comment 4

[Sylvestre Ledru [:Sylvestre]]

change of component to get the tracking flags back.

Comment 5

[Kohei Yoshino [:kohei]]

The custom video player has been removed.