Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
[Mac] .webkitClosedCaptionsVisible doesn't work with "Automatic" capt…
…ion mode https://bugs.webkit.org/show_bug.cgi?id=118319 Reviewed by Jer Noble. Source/WebCore: Track when captions are enabled with the old 'webkitClosedCaptionsVisible' attribute and consider that when determining which track should be enabled. Test: media/track/track-legacyapi-with-automatic-mode.html * html/HTMLMediaElement.cpp: (WebCore::HTMLMediaElement::HTMLMediaElement): Initialize m_webkitLegacyClosedCaptionOverride. (WebCore::HTMLMediaElement::setWebkitClosedCaptionsVisible): Set m_webkitLegacyClosedCaptionOverride. (WebCore::HTMLMediaElement::webkitClosedCaptionsVisible): Return m_webkitLegacyClosedCaptionOverride. * html/HTMLMediaElement.h: * page/CaptionUserPreferencesMediaAF.cpp: (WebCore::CaptionUserPreferencesMediaAF::textTrackSelectionScore): Don't apply "automatic" logic when captions were enabled with .webkitClosedCaptionsVisible. LayoutTests: * media/track/track-legacyapi-with-automatic-mode-expected.txt: Added. * media/track/track-legacyapi-with-automatic-mode.html: Added. Canonical link: https://commits.webkit.org/136413@main git-svn-id: https://svn.webkit.org/repository/webkit/trunk@152422 268f45cc-cd09-0410-ab3c-d52691b4dbfc
- Loading branch information
1 parent
55eefa5
commit 97a3125
Showing
7 changed files
with
134 additions
and
2 deletions.
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
30 changes: 30 additions & 0 deletions
30
LayoutTests/media/track/track-legacyapi-with-automatic-mode-expected.txt
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,30 @@ | ||
Test that enabling a track with .webkitClosedCaptionsVisible succeeds when in "automatic" mode. | ||
|
||
|
||
** Setup | ||
RUN(internals.setCaptionDisplayMode('Automatic')) | ||
RUN(trackElement = document.createElement('track')) | ||
RUN(trackElement.kind = 'captions') | ||
RUN(trackElement.src = 'captions-webvtt/tc004-webvtt-file.vtt') | ||
RUN(video.appendChild(trackElement)) | ||
|
||
EVENT(canplaythrough) | ||
|
||
** Test initial state | ||
EXPECTED (video.textTracks.length == '1') OK | ||
EXPECTED (video.textTracks[0].mode == 'disabled') OK | ||
|
||
** Enable cues, allow them to load | ||
RUN(video.webkitClosedCaptionsVisible = true) | ||
EVENT(load) | ||
EXPECTED (track.readyState == '2') OK | ||
RUN(video.currentTime = 0.5) | ||
|
||
EVENT(seeked) | ||
|
||
** Check to make sure captions are displaying | ||
EXPECTED (video.textTracks[0].cues != 'null') OK | ||
EXPECTED (textTrackDisplayElement(video, 'cue').textContent != 'null') OK | ||
|
||
END OF TEST | ||
|
67 changes: 67 additions & 0 deletions
67
LayoutTests/media/track/track-legacyapi-with-automatic-mode.html
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,67 @@ | ||
<!DOCTYPE html> | ||
<html> | ||
<head> | ||
<title>Enabling a track with legacy API succeeds when in "Automatic" mode</title> | ||
|
||
<script src=../media-file.js></script> | ||
<script src=../video-test.js></script> | ||
<script src=../trackmenu-test.js></script> | ||
<script src=../media-controls.js></script> | ||
<script> | ||
|
||
function seeked() | ||
{ | ||
consoleWrite("<br><i>** Check to make sure captions are displaying<" + "/i>"); | ||
|
||
testExpected("video.textTracks[0].cues", null, '!='); | ||
testExpected("textTrackDisplayElement(video, 'cue').textContent", null, '!='); | ||
|
||
consoleWrite(""); | ||
endTest(); | ||
} | ||
|
||
function trackLoaded() | ||
{ | ||
consoleWrite("EVENT(load)"); | ||
track = event.target; | ||
testExpected("track.readyState", HTMLTrackElement.LOADED); | ||
run("video.currentTime = 0.5"); | ||
waitForEvent('seeked', seeked); | ||
consoleWrite(""); | ||
} | ||
|
||
function canplaythrough() | ||
{ | ||
consoleWrite("<br><i>** Test initial state<" + "/i>"); | ||
testExpected("video.textTracks.length", 1); | ||
testExpected("video.textTracks[0].mode", "disabled"); | ||
|
||
consoleWrite("<br><i>** Enable cues, allow them to load<" + "/i>"); | ||
run("video.webkitClosedCaptionsVisible = true"); | ||
} | ||
|
||
function setup() | ||
{ | ||
consoleWrite("<br><i>** Setup<" + "/i>"); | ||
findMediaElement(); | ||
|
||
if (window.internals) | ||
run("internals.setCaptionDisplayMode('Automatic')"); | ||
run("trackElement = document.createElement('track')"); | ||
run("trackElement.kind = 'captions'"); | ||
run("trackElement.src = 'captions-webvtt/tc004-webvtt-file.vtt'"); | ||
run("video.appendChild(trackElement)"); | ||
|
||
video.src = findMediaFile('video', '../content/test'); | ||
trackElement.setAttribute('onload', 'trackLoaded()'); | ||
waitForEvent('canplaythrough', canplaythrough); | ||
consoleWrite(""); | ||
} | ||
|
||
</script> | ||
</head> | ||
<body onload="setup()"> | ||
<video controls width=500></video> | ||
<p>Test that enabling a track with .webkitClosedCaptionsVisible succeeds when in "automatic" mode.</p> | ||
</body> | ||
</html> |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters