[webkit-changes] [WebKit/WebKit] 27ac54: Use consistent settings between WebKit/WebKitLegac...
Brent Fulgham
noreply at github.com
Fri Jan 20 20:24:22 PST 2023
Branch: refs/heads/main
Home: https://github.com/WebKit/WebKit
Commit: 27ac548bef6ea93a00857b33ca9bbefa811391ef
https://github.com/WebKit/WebKit/commit/27ac548bef6ea93a00857b33ca9bbefa811391ef
Author: Brent Fulgham <bfulgham at apple.com>
Date: 2023-01-20 (Fri, 20 Jan 2023)
Changed paths:
M LayoutTests/http/tests/cache/disk-cache/disk-cache-media-small.html
M LayoutTests/media/ios/autoplay-only-in-main-document.html
M LayoutTests/media/loadedmetadata-fires-without-user-gesture-when-setRequiresUserGestureToLoadVideo-false.html
M LayoutTests/media/no-autoplay-with-user-gesture-requirement.html
M LayoutTests/media/playlist-inherits-user-gesture-expected.txt
M LayoutTests/media/playlist-inherits-user-gesture.html
M LayoutTests/media/video-add-autoplay-user-gesture-expected.txt
M LayoutTests/media/video-add-autoplay-user-gesture.html
M LayoutTests/media/video-create-with-user-gesture-expected.txt
M LayoutTests/media/video-create-with-user-gesture.html
M LayoutTests/media/video-load-require-user-gesture.html
M LayoutTests/media/video-source-before-src-expected.txt
M LayoutTests/media/video-source-before-src.html
M LayoutTests/media/video-user-gesture-tracking-expected.txt
M LayoutTests/media/video-user-gesture-tracking.html
M LayoutTests/platform/ios/media/video-source-before-src-expected.txt
M Source/WTF/Scripts/Preferences/UnifiedWebPreferences.yaml
M Source/WebCore/Modules/speech/SpeechSynthesis.cpp
M Source/WebCore/Modules/webaudio/AudioContext.cpp
M Source/WebCore/dom/Document.cpp
M Source/WebCore/dom/Document.h
M Source/WebCore/html/HTMLMediaElement.cpp
M Source/WebCore/page/Settings.yaml
M Source/WebKit/WebProcess/WebPage/WebPage.cpp
M Source/WebKitLegacy/mac/WebView/WebView.mm
M Source/WebKitLegacy/win/WebView.cpp
Log Message:
-----------
Use consistent settings between WebKit/WebKitLegacy/WebCore for audio and video gesture requirements
https://bugs.webkit.org/show_bug.cgi?id=250858
<rdar://problem/104440786>
Reviewed by Jer Noble.
We currently have different names used for WebCore-related media gesture requirements, and the
preferences in WebKitLegacy and WebKit with custom code to toggle the WebCore bits in response
to the WebKit/WebKitLegacy settings.
Instead, we should let the generator code create the WebCore settings code, which will
automatically sync it when WebKit/WebKitLegacy preferences are adjusted.
Since the internal keys used to control the settings are unchanged, this should not
modify behavior.
* LayoutTests/http/tests/cache/disk-cache/disk-cache-media-small.html: Update generated 'internals'
method call to match Settings.
* LayoutTests/media/ios/autoplay-only-in-main-document.html: Ditto.
* LayoutTests/media/loadedmetadata-fires-without-user-gesture-when-setRequiresUserGestureToLoadVideo-false.html: Ditto.
* LayoutTests/media/no-autoplay-with-user-gesture-requirement.html: Ditto.
* LayoutTests/media/playlist-inherits-user-gesture-expected.txt: Ditto.
* LayoutTests/media/playlist-inherits-user-gesture.html: Ditto.
* LayoutTests/media/video-add-autoplay-user-gesture-expected.txt: Ditto.
* LayoutTests/media/video-add-autoplay-user-gesture.html: Ditto.
* LayoutTests/media/video-create-with-user-gesture-expected.txt: Ditto.
* LayoutTests/media/video-create-with-user-gesture.html: Ditto.
* LayoutTests/media/video-load-require-user-gesture.html: Ditto.
* LayoutTests/media/video-source-before-src-expected.txt: Ditto.
* LayoutTests/media/video-source-before-src.html: Ditto.
* LayoutTests/media/video-user-gesture-tracking-expected.txt: Ditto.
* LayoutTests/media/video-user-gesture-tracking.html: Ditto.
* LayoutTests/platform/ios/media/video-source-before-src-expected.txt: Ditto.
* Source/WTF/Scripts/Preferences/UnifiedWebPreferences.yaml:
* Source/WebCore/Modules/speech/SpeechSynthesis.cpp:
(WebCore::SpeechSynthesis::SpeechSynthesis):
* Source/WebCore/Modules/webaudio/AudioContext.cpp:
(WebCore::AudioContext::constructCommon):
* Source/WebCore/dom/Document.cpp:
(WebCore::Document::requiresUserGestureForAudioPlayback const):
(WebCore::Document::requiresUserGestureForVideoPlayback const):
(WebCore::Document::audioPlaybackRequiresUserGesture const): Deleted.
(WebCore::Document::videoPlaybackRequiresUserGesture const): Deleted.
* Source/WebCore/dom/Document.h:
* Source/WebCore/html/HTMLMediaElement.cpp:
(WebCore::HTMLMediaElement::HTMLMediaElement):
(WebCore::HTMLMediaElement::initializeMediaSession):
(WebCore::HTMLMediaElement::updateRateChangeRestrictions):
* Source/WebCore/page/Settings.yaml:
* Source/WebKit/WebProcess/WebPage/WebPage.cpp:
(WebKit::WebPage::updatePreferences): Update for generated Settings labels.
* Source/WebKitLegacy/mac/WebView/WebView.mm:
(-[WebView _preferencesChanged:]): Ditto.
* Source/WebKitLegacy/win/WebView.cpp:
(WebView::notifyPreferencesChanged): Ditto.
Canonical link: https://commits.webkit.org/259163@main
More information about the webkit-changes
mailing list