[webkit-changes] [WebKit/WebKit] 5e62d5: Versioning.
MyahCobbs
noreply at github.com
Tue May 14 16:13:26 PDT 2024
Branch: refs/heads/safari-7617.2.1.10-branch
Home: https://github.com/WebKit/WebKit
Commit: 5e62d504367c3cb6d07a8c394ecc28cc17666a85
https://github.com/WebKit/WebKit/commit/5e62d504367c3cb6d07a8c394ecc28cc17666a85
Author: Myah Cobbs <mcobbs at apple.com>
Date: 2023-12-04 (Mon, 04 Dec 2023)
Changed paths:
M Configurations/Version.xcconfig
Log Message:
-----------
Versioning.
WebKit-7617.2.1.10.1
Identifier: 267815.600 at safari-7617.2.1.10-branch
Commit: 56fa3a19d532d73e9fb00c4761f8747c65dbbc66
https://github.com/WebKit/WebKit/commit/56fa3a19d532d73e9fb00c4761f8747c65dbbc66
Author: Myah Cobbs <mcobbs at apple.com>
Date: 2023-12-04 (Mon, 04 Dec 2023)
Changed paths:
A LayoutTests/media/content/test-inband-captions.mp4
A LayoutTests/media/track/track-in-band-layout-expected.txt
A LayoutTests/media/track/track-in-band-layout.html
M LayoutTests/platform/glib/TestExpectations
M Source/WebCore/html/track/InbandGenericTextTrack.cpp
M Source/WebCore/html/track/TextTrackCueGeneric.cpp
M Source/WebCore/html/track/TextTrackCueGeneric.h
M Source/WebCore/html/track/VTTCue.h
M Source/WebCore/platform/graphics/InbandGenericCue.cpp
M Source/WebCore/platform/graphics/InbandGenericCue.h
M Source/WebCore/platform/graphics/avfoundation/InbandTextTrackPrivateAVF.cpp
M Source/WebKit/Shared/WebCoreArgumentCoders.serialization.in
Log Message:
-----------
Cherry-pick 53b6a4522e45. rdar://119138261
REGRESSION(265596 at main): [Cocoa] In-band captions generate large, line-wrapped cues
https://bugs.webkit.org/show_bug.cgi?id=265635
rdar://119024855
Reviewed by Eric Carlson.
When modernizing VTTCue, TextTrackCueGeneric was not sufficiently updated to take advantage
of the correct layout implemented by VTTCue. Specifically, TextTrackCueGeneric still queried
VTTCueBox::fontSizeFromCaptionUserPrefs(), which was no longer set, and caused the cue width
to be incorrectly set to zero.
Remove that dead method, and remove most of the implementation of
TextTrackCueGenericBoxElement::applyCSSProperties, relying instead upon VTTCueBoxElement's
implementation.
In order to correctly position TextTrackCueGeneric objects created from AVFoundation, set
the resulting cue's positionAlign() to "Center" by way of GenericCueData. This causes VTTCue
to position the cue relative to the cue's center point, which is the same way AVFoundation
represents the cue's position.
* LayoutTests/media/content/test-inband-captions.mp4: Added.
* LayoutTests/media/track/track-in-band-layout-expected.txt: Added.
* LayoutTests/media/track/track-in-band-layout.html: Added.
* Source/WebCore/html/track/InbandGenericTextTrack.cpp:
(WebCore::InbandGenericTextTrack::updateCueFromCueData):
* Source/WebCore/html/track/TextTrackCueGeneric.cpp:
(WebCore::TextTrackCueGenericBoxElement::applyCSSProperties):
(WebCore::TextTrackCueGeneric::setBaseFontSizeRelativeToVideoHeight):
(WebCore::TextTrackCueGeneric::setFontSizeMultiplier):
(WebCore::TextTrackCueGeneric::setFontSize): Deleted.
* Source/WebCore/html/track/TextTrackCueGeneric.h:
* Source/WebCore/html/track/VTTCue.h:
(WebCore::VTTCueBox::setFontSizeFromCaptionUserPrefs): Deleted.
(WebCore::VTTCueBox::fontSizeFromCaptionUserPrefs const): Deleted.
* Source/WebCore/platform/graphics/InbandGenericCue.cpp:
(WebCore::InbandGenericCue::toJSONString const):
* Source/WebCore/platform/graphics/InbandGenericCue.h:
(WebCore::GenericCueData::GenericCueData):
(WebCore::InbandGenericCue::positionAlign const):
(WebCore::InbandGenericCue::setPositionAlign):
* Source/WebCore/platform/graphics/avfoundation/InbandTextTrackPrivateAVF.cpp:
(WebCore::InbandTextTrackPrivateAVF::processAttributedStrings):
* Source/WebKit/Shared/WebCoreArgumentCoders.serialization.in:
Canonical link: https://commits.webkit.org/271415@main
Identifier: 267815.602 at safari-7617.2.1.10-branch
Commit: dbcc506e57c89e133e02dfa713e4c532971c9929
https://github.com/WebKit/WebKit/commit/dbcc506e57c89e133e02dfa713e4c532971c9929
Author: Myah Cobbs <mcobbs at apple.com>
Date: 2023-12-06 (Wed, 06 Dec 2023)
Changed paths:
M Configurations/Version.xcconfig
Log Message:
-----------
Versioning.
WebKit-7617.2.1.10.2
Identifier: 267815.603 at safari-7617.2.1.10-branch
Commit: 37ca3b3d13f866a753163b3c3154f352c56bf8e5
https://github.com/WebKit/WebKit/commit/37ca3b3d13f866a753163b3c3154f352c56bf8e5
Author: Myah Cobbs <mcobbs at apple.com>
Date: 2023-12-07 (Thu, 07 Dec 2023)
Changed paths:
M Configurations/Version.xcconfig
Log Message:
-----------
Versioning.
WebKit-7617.2.1.10.3
Identifier: 267815.604 at safari-7617.2.1.10-branch
Compare: https://github.com/WebKit/WebKit/compare/5e62d504367c%5E...37ca3b3d13f8
To unsubscribe from these emails, change your notification settings at https://github.com/WebKit/WebKit/settings/notifications
More information about the webkit-changes
mailing list