Title: [282408] trunk
Revision
282408
Author
[email protected]
Date
2021-09-14 13:21:43 -0700 (Tue, 14 Sep 2021)

Log Message

Enable RTCRtpScriptTransform by default
https://bugs.webkit.org/show_bug.cgi?id=230220
Source/WebCore:

Reviewed by Eric Carlson.

No change of behavior, renaming WebRTCInsertableStreamsEnabled to WebRTCEncodedTransformEnabled
and WebRTCSFrameTransformEnabled.

* Modules/mediastream/RTCEncodedAudioFrame.idl:
* Modules/mediastream/RTCEncodedVideoFrame.idl:
* Modules/mediastream/RTCRtpReceiver+Transform.idl:
* Modules/mediastream/RTCRtpSFrameTransform.idl:
* Modules/mediastream/RTCRtpSFrameTransformErrorEvent.idl:
* Modules/mediastream/RTCRtpScriptTransform.idl:
* Modules/mediastream/RTCRtpScriptTransformer.idl:
* Modules/mediastream/RTCRtpSender+Transform.idl:
* Modules/mediastream/RTCTransformEvent.idl:
* workers/DedicatedWorkerGlobalScope.idl:

Source/WTF:

Reviewed by Eric Carlson.

Splitting  WebRTCInsertableStreamsEnabled to WebRTCEncodedTransformEnabled and WebRTCSFrameTransformEnabled.
Keeping WebRTCSFrameTransformEnabled as off by default until SFrame solidifies and mark WebRTCEncodedTransformEnabled as on by default.

* Scripts/Preferences/WebPreferencesExperimental.yaml:

LayoutTests:

<rdar://problem/83090247>

Reviewed by Eric Carlson.

* http/wpt/webrtc/no-webrtc-transform.html:
Migrate to new internal setting.

Modified Paths

Diff

Modified: trunk/LayoutTests/ChangeLog (282407 => 282408)


--- trunk/LayoutTests/ChangeLog	2021-09-14 20:18:38 UTC (rev 282407)
+++ trunk/LayoutTests/ChangeLog	2021-09-14 20:21:43 UTC (rev 282408)
@@ -1,3 +1,14 @@
+2021-09-14  Youenn Fablet  <[email protected]>
+
+        Enable RTCRtpScriptTransform by default
+        https://bugs.webkit.org/show_bug.cgi?id=230220
+        <rdar://problem/83090247>
+
+        Reviewed by Eric Carlson.
+
+        * http/wpt/webrtc/no-webrtc-transform.html:
+        Migrate to new internal setting.
+
 2021-09-14  Ayumi Kojima  <[email protected]>
 
         [ iOS Mac Debug ] imported/w3c/web-platform-tests/html/cross-origin-opener-policy/coop-navigated-history-popup.https.html is a flaky failure.

Modified: trunk/LayoutTests/http/wpt/webrtc/no-webrtc-transform.html (282407 => 282408)


--- trunk/LayoutTests/http/wpt/webrtc/no-webrtc-transform.html	2021-09-14 20:18:38 UTC (rev 282407)
+++ trunk/LayoutTests/http/wpt/webrtc/no-webrtc-transform.html	2021-09-14 20:21:43 UTC (rev 282408)
@@ -9,7 +9,7 @@
         <script>
 promise_test(async (test) => {
     if (window.internals)
-        internals.settings.setWebRTCInsertableStreamsEnabled(true);
+        internals.settings.setWebRTCEncodedTransformEnabled(true);
 
     const worker = new Worker('no-transform.js');
     const data = "" new Promise(resolve => worker._onmessage_ = (event) => resolve(event.data));
@@ -18,7 +18,7 @@
 
 promise_test(async (test) => {
     if (window.internals)
-        internals.settings.setWebRTCInsertableStreamsEnabled(false);
+        internals.settings.setWebRTCEncodedTransformEnabled(false);
 
     const worker = new Worker('no-transform.js');
     const data = "" new Promise(resolve => worker._onmessage_ = (event) => resolve(event.data));

Modified: trunk/Source/WTF/ChangeLog (282407 => 282408)


--- trunk/Source/WTF/ChangeLog	2021-09-14 20:18:38 UTC (rev 282407)
+++ trunk/Source/WTF/ChangeLog	2021-09-14 20:21:43 UTC (rev 282408)
@@ -1,3 +1,15 @@
+2021-09-14  Youenn Fablet  <[email protected]>
+
+        Enable RTCRtpScriptTransform by default
+        https://bugs.webkit.org/show_bug.cgi?id=230220
+
+        Reviewed by Eric Carlson.
+
+        Splitting  WebRTCInsertableStreamsEnabled to WebRTCEncodedTransformEnabled and WebRTCSFrameTransformEnabled.
+        Keeping WebRTCSFrameTransformEnabled as off by default until SFrame solidifies and mark WebRTCEncodedTransformEnabled as on by default.
+
+        * Scripts/Preferences/WebPreferencesExperimental.yaml:
+
 2021-09-14  Fujii Hironori  <[email protected]>
 
         StackTrace outputs a garbage prefix because WTF::StackTrace::m_prefix is filled with the stack content

Modified: trunk/Source/WTF/Scripts/Preferences/WebPreferencesExperimental.yaml (282407 => 282408)


--- trunk/Source/WTF/Scripts/Preferences/WebPreferencesExperimental.yaml	2021-09-14 20:18:38 UTC (rev 282407)
+++ trunk/Source/WTF/Scripts/Preferences/WebPreferencesExperimental.yaml	2021-09-14 20:21:43 UTC (rev 282408)
@@ -1392,6 +1392,19 @@
     WebKit:
       default: WebKit::defaultWebMParserEnabled()
 
+WebRTCEncodedTransformEnabled:
+  type: bool
+  condition: ENABLE(WEB_RTC)
+  humanReadableName: "WebRTC Encoded Transform API"
+  humanReadableDescription: "Enable WebRTC Encoded Transform API"
+  defaultValue:
+    WebKitLegacy:
+      default: false
+    WebKit:
+      default: true
+    WebCore:
+      default: true
+
 # FIXME: Is this implemented for WebKitLegacy? If not, this should be excluded from WebKitLegacy entirely.
 WebRTCH264LowLatencyEncoderEnabled:
   type: bool
@@ -1418,19 +1431,6 @@
     WebKit:
       default: false
 
-WebRTCInsertableStreamsEnabled:
-  type: bool
-  condition: ENABLE(WEB_RTC)
-  humanReadableName: "WebRTC Insertable Streams API"
-  humanReadableDescription: "Enable WebRTC Insertable Streams API"
-  defaultValue:
-    WebKitLegacy:
-      default: false
-    WebKit:
-      default: false
-    WebCore:
-      default: false
-
 # FIXME: This is not relevent for WebKitLegacy, so should be excluded from WebKitLegacy entirely.
 WebRTCPlatformCodecsInGPUProcessEnabled:
   type: bool
@@ -1465,6 +1465,19 @@
     WebKit:
       default: true
 
+WebRTCSFrameTransformEnabled:
+  type: bool
+  condition: ENABLE(WEB_RTC)
+  humanReadableName: "WebRTC SFrame Transform API"
+  humanReadableDescription: "Enable WebRTC SFrame Transform API"
+  defaultValue:
+    WebKitLegacy:
+      default: false
+    WebKit:
+      default: false
+    WebCore:
+      default: false
+
 WebRTCSocketsProxyingEnabled:
   type: bool
   humanReadableName: "WebRTC Sockets Proxying"

Modified: trunk/Source/WebCore/ChangeLog (282407 => 282408)


--- trunk/Source/WebCore/ChangeLog	2021-09-14 20:18:38 UTC (rev 282407)
+++ trunk/Source/WebCore/ChangeLog	2021-09-14 20:21:43 UTC (rev 282408)
@@ -1,3 +1,24 @@
+2021-09-14  Youenn Fablet  <[email protected]>
+
+        Enable RTCRtpScriptTransform by default
+        https://bugs.webkit.org/show_bug.cgi?id=230220
+
+        Reviewed by Eric Carlson.
+
+        No change of behavior, renaming WebRTCInsertableStreamsEnabled to WebRTCEncodedTransformEnabled
+        and WebRTCSFrameTransformEnabled.
+
+        * Modules/mediastream/RTCEncodedAudioFrame.idl:
+        * Modules/mediastream/RTCEncodedVideoFrame.idl:
+        * Modules/mediastream/RTCRtpReceiver+Transform.idl:
+        * Modules/mediastream/RTCRtpSFrameTransform.idl:
+        * Modules/mediastream/RTCRtpSFrameTransformErrorEvent.idl:
+        * Modules/mediastream/RTCRtpScriptTransform.idl:
+        * Modules/mediastream/RTCRtpScriptTransformer.idl:
+        * Modules/mediastream/RTCRtpSender+Transform.idl:
+        * Modules/mediastream/RTCTransformEvent.idl:
+        * workers/DedicatedWorkerGlobalScope.idl:
+
 2021-09-14  Darin Adler  <[email protected]>
 
         URLs in CSS variables must be resolved against the base URL of the stylesheet, not the document

Modified: trunk/Source/WebCore/Modules/mediastream/RTCEncodedAudioFrame.idl (282407 => 282408)


--- trunk/Source/WebCore/Modules/mediastream/RTCEncodedAudioFrame.idl	2021-09-14 20:18:38 UTC (rev 282407)
+++ trunk/Source/WebCore/Modules/mediastream/RTCEncodedAudioFrame.idl	2021-09-14 20:21:43 UTC (rev 282408)
@@ -34,7 +34,7 @@
 
 [
     Conditional=WEB_RTC,
-    EnabledBySetting=WebRTCInsertableStreams,
+    EnabledBySetting=WebRTCEncodedTransform,
     Exposed=DedicatedWorker,
     ImplementationLacksVTable,
 ] interface RTCEncodedAudioFrame {

Modified: trunk/Source/WebCore/Modules/mediastream/RTCEncodedVideoFrame.idl (282407 => 282408)


--- trunk/Source/WebCore/Modules/mediastream/RTCEncodedVideoFrame.idl	2021-09-14 20:18:38 UTC (rev 282407)
+++ trunk/Source/WebCore/Modules/mediastream/RTCEncodedVideoFrame.idl	2021-09-14 20:21:43 UTC (rev 282408)
@@ -43,7 +43,7 @@
 
 [
     Conditional=WEB_RTC,
-    EnabledBySetting=WebRTCInsertableStreams,
+    EnabledBySetting=WebRTCEncodedTransform,
     Exposed=DedicatedWorker,
     ImplementationLacksVTable,
 ] interface RTCEncodedVideoFrame {

Modified: trunk/Source/WebCore/Modules/mediastream/RTCRtpReceiver+Transform.idl (282407 => 282408)


--- trunk/Source/WebCore/Modules/mediastream/RTCRtpReceiver+Transform.idl	2021-09-14 20:18:38 UTC (rev 282407)
+++ trunk/Source/WebCore/Modules/mediastream/RTCRtpReceiver+Transform.idl	2021-09-14 20:21:43 UTC (rev 282408)
@@ -27,7 +27,7 @@
 
 [
     Conditional=WEB_RTC,
-    EnabledBySetting=WebRTCInsertableStreams,
+    EnabledBySetting=WebRTCEncodedTransform,
     Exposed=Window,
     ImplementedBy=RTCRtpReceiverWithTransform,
 ] partial interface RTCRtpReceiver {

Modified: trunk/Source/WebCore/Modules/mediastream/RTCRtpSFrameTransform.idl (282407 => 282408)


--- trunk/Source/WebCore/Modules/mediastream/RTCRtpSFrameTransform.idl	2021-09-14 20:18:38 UTC (rev 282407)
+++ trunk/Source/WebCore/Modules/mediastream/RTCRtpSFrameTransform.idl	2021-09-14 20:21:43 UTC (rev 282408)
@@ -43,7 +43,7 @@
 [
     ActiveDOMObject,
     Conditional=WEB_RTC,
-    EnabledBySetting=WebRTCInsertableStreams,
+    EnabledBySetting=WebRTCSFrameTransform,
     ExportMacro=WEBCORE_EXPORT,
     Exposed=(Window,DedicatedWorker),
     InterfaceName=SFrameTransform,

Modified: trunk/Source/WebCore/Modules/mediastream/RTCRtpSFrameTransformErrorEvent.idl (282407 => 282408)


--- trunk/Source/WebCore/Modules/mediastream/RTCRtpSFrameTransformErrorEvent.idl	2021-09-14 20:18:38 UTC (rev 282407)
+++ trunk/Source/WebCore/Modules/mediastream/RTCRtpSFrameTransformErrorEvent.idl	2021-09-14 20:21:43 UTC (rev 282408)
@@ -36,7 +36,7 @@
 
 [
     Conditional=WEB_RTC,
-    EnabledBySetting=WebRTCInsertableStreams,
+    EnabledBySetting=WebRTCSFrameTransform,
     Exposed=(Window,DedicatedWorker),
     InterfaceName=SFrameTransformErrorEvent
 ] interface RTCRtpSFrameTransformErrorEvent : Event {

Modified: trunk/Source/WebCore/Modules/mediastream/RTCRtpScriptTransform.idl (282407 => 282408)


--- trunk/Source/WebCore/Modules/mediastream/RTCRtpScriptTransform.idl	2021-09-14 20:18:38 UTC (rev 282407)
+++ trunk/Source/WebCore/Modules/mediastream/RTCRtpScriptTransform.idl	2021-09-14 20:21:43 UTC (rev 282408)
@@ -26,7 +26,7 @@
 [
     ActiveDOMObject,
     Conditional=WEB_RTC,
-    EnabledBySetting=WebRTCInsertableStreams,
+    EnabledBySetting=WebRTCEncodedTransform,
     Exposed=Window,
     JSGenerateToNativeObject,
 ] interface RTCRtpScriptTransform : EventTarget {

Modified: trunk/Source/WebCore/Modules/mediastream/RTCRtpScriptTransformer.idl (282407 => 282408)


--- trunk/Source/WebCore/Modules/mediastream/RTCRtpScriptTransformer.idl	2021-09-14 20:18:38 UTC (rev 282407)
+++ trunk/Source/WebCore/Modules/mediastream/RTCRtpScriptTransformer.idl	2021-09-14 20:21:43 UTC (rev 282408)
@@ -26,7 +26,7 @@
 [
     ActiveDOMObject,
     Conditional=WEB_RTC,
-    EnabledBySetting=WebRTCInsertableStreams,
+    EnabledBySetting=WebRTCEncodedTransform,
     Exposed=DedicatedWorker,
     JSGenerateToNativeObject,
 ] interface RTCRtpScriptTransformer {

Modified: trunk/Source/WebCore/Modules/mediastream/RTCRtpSender+Transform.idl (282407 => 282408)


--- trunk/Source/WebCore/Modules/mediastream/RTCRtpSender+Transform.idl	2021-09-14 20:18:38 UTC (rev 282407)
+++ trunk/Source/WebCore/Modules/mediastream/RTCRtpSender+Transform.idl	2021-09-14 20:21:43 UTC (rev 282408)
@@ -27,7 +27,7 @@
 
 [
     Conditional=WEB_RTC,
-    EnabledBySetting=WebRTCInsertableStreams,
+    EnabledBySetting=WebRTCEncodedTransform,
     Exposed=Window,
     ImplementedBy=RTCRtpSenderWithTransform,
 ] partial interface RTCRtpSender {

Modified: trunk/Source/WebCore/Modules/mediastream/RTCTransformEvent.idl (282407 => 282408)


--- trunk/Source/WebCore/Modules/mediastream/RTCTransformEvent.idl	2021-09-14 20:18:38 UTC (rev 282407)
+++ trunk/Source/WebCore/Modules/mediastream/RTCTransformEvent.idl	2021-09-14 20:21:43 UTC (rev 282408)
@@ -24,7 +24,7 @@
 
 [
     Conditional=WEB_RTC,
-    EnabledBySetting=WebRTCInsertableStreams,
+    EnabledBySetting=WebRTCEncodedTransform,
     Exposed=Worker,
     InterfaceName=RTCTransformEvent,
 ] interface RTCTransformEvent : Event {

Modified: trunk/Source/WebCore/workers/DedicatedWorkerGlobalScope.idl (282407 => 282408)


--- trunk/Source/WebCore/workers/DedicatedWorkerGlobalScope.idl	2021-09-14 20:18:38 UTC (rev 282407)
+++ trunk/Source/WebCore/workers/DedicatedWorkerGlobalScope.idl	2021-09-14 20:21:43 UTC (rev 282408)
@@ -46,4 +46,4 @@
 };
 
 [Conditional=OFFSCREEN_CANVAS_IN_WORKERS, EnabledAtRuntime=OffscreenCanvasInWorkers] DedicatedWorkerGlobalScope includes AnimationFrameProvider;
-[Conditional=WEB_RTC, EnabledBySetting=WebRTCInsertableStreams] DedicatedWorkerGlobalScope includes RTCRtpScriptTransformProvider;
+[Conditional=WEB_RTC, EnabledBySetting=WebRTCEncodedTransform] DedicatedWorkerGlobalScope includes RTCRtpScriptTransformProvider;
_______________________________________________
webkit-changes mailing list
[email protected]
https://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to