Title: [226790] trunk/Source
Revision
226790
Author
[email protected]
Date
2018-01-11 11:48:36 -0800 (Thu, 11 Jan 2018)

Log Message

Replace WebRTCLegacyAPIDisabled by WebRTCLegacyAPIEnabled and switch off WebRTC legacy flag by default
https://bugs.webkit.org/show_bug.cgi?id=181480

Patch by Youenn Fablet <[email protected]> on 2018-01-11
Reviewed by Eric Carlson.

Source/WebCore:

No change of behavior.

* page/RuntimeEnabledFeatures.h: Set default value to false.

Source/WebKit:

Renaming preference to WebRTCLegacyAPIEnabled for simplification and removing it from experimental feature.
Set it to off by default.

* Shared/WebPreferences.yaml:
* UIProcess/API/C/WKPreferences.cpp:
(WKPreferencesSetWebRTCLegacyAPIEnabled):
(WKPreferencesGetWebRTCLegacyAPIEnabled):
* UIProcess/API/Cocoa/WKPreferences.mm:
(-[WKPreferences _webRTCLegacyAPIEnabled]):
(-[WKPreferences _setWebRTCLegacyAPIEnabled:]):
* UIProcess/WebProcessPool.cpp:
(WebKit::WebProcessPool::ensureNetworkProcess):
* WebProcess/WebPage/WebPage.cpp:
(WebKit::WebPage::updatePreferences):

Source/WebKitLegacy/mac:

* WebView/WebPreferences.mm:
(+[WebPreferences initialize]):

Modified Paths

Diff

Modified: trunk/Source/WebCore/ChangeLog (226789 => 226790)


--- trunk/Source/WebCore/ChangeLog	2018-01-11 19:22:46 UTC (rev 226789)
+++ trunk/Source/WebCore/ChangeLog	2018-01-11 19:48:36 UTC (rev 226790)
@@ -1,3 +1,14 @@
+2018-01-11  Youenn Fablet  <[email protected]>
+
+        Replace WebRTCLegacyAPIDisabled by WebRTCLegacyAPIEnabled and switch off WebRTC legacy flag by default
+        https://bugs.webkit.org/show_bug.cgi?id=181480
+
+        Reviewed by Eric Carlson.
+
+        No change of behavior.
+
+        * page/RuntimeEnabledFeatures.h: Set default value to false.
+
 2018-01-11  Wenson Hsieh  <[email protected]>
 
         [Attachment Support] Support moving attachment elements in editable areas using drag and drop

Modified: trunk/Source/WebCore/page/RuntimeEnabledFeatures.h (226789 => 226790)


--- trunk/Source/WebCore/page/RuntimeEnabledFeatures.h	2018-01-11 19:22:46 UTC (rev 226789)
+++ trunk/Source/WebCore/page/RuntimeEnabledFeatures.h	2018-01-11 19:48:36 UTC (rev 226790)
@@ -277,7 +277,7 @@
 
 #if ENABLE(WEB_RTC)
     bool m_isPeerConnectionEnabled { true };
-    bool m_webRTCLegacyAPIEnabled { true };
+    bool m_webRTCLegacyAPIEnabled { false };
 #endif
 
 #if ENABLE(LEGACY_CSS_VENDOR_PREFIXES)

Modified: trunk/Source/WebKit/ChangeLog (226789 => 226790)


--- trunk/Source/WebKit/ChangeLog	2018-01-11 19:22:46 UTC (rev 226789)
+++ trunk/Source/WebKit/ChangeLog	2018-01-11 19:48:36 UTC (rev 226790)
@@ -1,3 +1,25 @@
+2018-01-11  Youenn Fablet  <[email protected]>
+
+        Replace WebRTCLegacyAPIDisabled by WebRTCLegacyAPIEnabled and switch off WebRTC legacy flag by default
+        https://bugs.webkit.org/show_bug.cgi?id=181480
+
+        Reviewed by Eric Carlson.
+
+        Renaming preference to WebRTCLegacyAPIEnabled for simplification and removing it from experimental feature.
+        Set it to off by default.
+
+        * Shared/WebPreferences.yaml:
+        * UIProcess/API/C/WKPreferences.cpp:
+        (WKPreferencesSetWebRTCLegacyAPIEnabled):
+        (WKPreferencesGetWebRTCLegacyAPIEnabled):
+        * UIProcess/API/Cocoa/WKPreferences.mm:
+        (-[WKPreferences _webRTCLegacyAPIEnabled]):
+        (-[WKPreferences _setWebRTCLegacyAPIEnabled:]):
+        * UIProcess/WebProcessPool.cpp:
+        (WebKit::WebProcessPool::ensureNetworkProcess):
+        * WebProcess/WebPage/WebPage.cpp:
+        (WebKit::WebPage::updatePreferences):
+
 2018-01-11  Alex Christensen  <[email protected]>
 
         Merge sync and async code paths for getting context menus

Modified: trunk/Source/WebKit/Shared/WebPreferences.yaml (226789 => 226790)


--- trunk/Source/WebKit/Shared/WebPreferences.yaml	2018-01-11 19:22:46 UTC (rev 226789)
+++ trunk/Source/WebKit/Shared/WebPreferences.yaml	2018-01-11 19:48:36 UTC (rev 226790)
@@ -1103,13 +1103,12 @@
   category: experimental
   webcoreBinding: RuntimeEnabledFeatures
 
-WebRTCLegacyAPIDisabled:
+WebRTCLegacyAPIEnabled:
   type: bool
-  defaultValue: DEFAULT_EXPERIMENTAL_FEATURES_ENABLED
-  humanReadableName: "Remove Legacy WebRTC API"
-  humanReadableDescription: "Remove Legacy WebRTC API"
-  category: experimental
-  webcoreBinding: custom
+  defaultValue: false
+  humanReadableName: "Enable Legacy WebRTC API"
+  humanReadableDescription: "Enable Legacy WebRTC API"
+  webcoreBinding: RuntimeEnabledFeatures
   condition: ENABLE(WEB_RTC)
 
 IsSecureContextAttributeEnabled:

Modified: trunk/Source/WebKit/UIProcess/API/C/WKPreferences.cpp (226789 => 226790)


--- trunk/Source/WebKit/UIProcess/API/C/WKPreferences.cpp	2018-01-11 19:22:46 UTC (rev 226789)
+++ trunk/Source/WebKit/UIProcess/API/C/WKPreferences.cpp	2018-01-11 19:48:36 UTC (rev 226790)
@@ -1416,12 +1416,12 @@
 
 void WKPreferencesSetWebRTCLegacyAPIEnabled(WKPreferencesRef preferencesRef, bool enabled)
 {
-    toImpl(preferencesRef)->setWebRTCLegacyAPIDisabled(!enabled);
+    toImpl(preferencesRef)->setWebRTCLegacyAPIEnabled(enabled);
 }
 
 bool WKPreferencesGetWebRTCLegacyAPIEnabled(WKPreferencesRef preferencesRef)
 {
-    return !toImpl(preferencesRef)->webRTCLegacyAPIDisabled();
+    return toImpl(preferencesRef)->webRTCLegacyAPIEnabled();
 }
 
 void WKPreferencesSetSpatialNavigationEnabled(WKPreferencesRef preferencesRef, bool enabled)

Modified: trunk/Source/WebKit/UIProcess/API/Cocoa/WKPreferences.mm (226789 => 226790)


--- trunk/Source/WebKit/UIProcess/API/Cocoa/WKPreferences.mm	2018-01-11 19:22:46 UTC (rev 226789)
+++ trunk/Source/WebKit/UIProcess/API/Cocoa/WKPreferences.mm	2018-01-11 19:48:36 UTC (rev 226790)
@@ -660,12 +660,12 @@
 
 - (BOOL)_webRTCLegacyAPIEnabled
 {
-    return !_preferences->webRTCLegacyAPIDisabled();
+    return _preferences->webRTCLegacyAPIEnabled();
 }
 
 - (void)_setWebRTCLegacyAPIEnabled:(BOOL)enabled
 {
-    _preferences->setWebRTCLegacyAPIDisabled(!enabled);
+    _preferences->setWebRTCLegacyAPIEnabled(enabled);
 }
 
 - (void)_setJavaScriptCanAccessClipboard:(BOOL)_javascript_CanAccessClipboard

Modified: trunk/Source/WebKit/WebProcess/WebPage/WebPage.cpp (226789 => 226790)


--- trunk/Source/WebKit/WebProcess/WebPage/WebPage.cpp	2018-01-11 19:22:46 UTC (rev 226789)
+++ trunk/Source/WebKit/WebProcess/WebPage/WebPage.cpp	2018-01-11 19:48:36 UTC (rev 226790)
@@ -3080,10 +3080,6 @@
     settings.setDataDetectorTypes(static_cast<DataDetectorTypes>(store.getUInt32ValueForKey(WebPreferencesKey::dataDetectorTypesKey())));
 #endif
 
-#if ENABLE(WEB_RTC)
-    RuntimeEnabledFeatures::sharedFeatures().setWebRTCLegacyAPIEnabled(!store.getBoolValueForKey(WebPreferencesKey::webRTCLegacyAPIDisabledKey()));
-#endif
-
     RuntimeEnabledFeatures::sharedFeatures().setWebAuthenticationEnabled(store.getBoolValueForKey(WebPreferencesKey::webAuthenticationEnabledKey()));
 
     DatabaseManager::singleton().setIsAvailable(store.getBoolValueForKey(WebPreferencesKey::databasesEnabledKey()));

Modified: trunk/Source/WebKitLegacy/mac/ChangeLog (226789 => 226790)


--- trunk/Source/WebKitLegacy/mac/ChangeLog	2018-01-11 19:22:46 UTC (rev 226789)
+++ trunk/Source/WebKitLegacy/mac/ChangeLog	2018-01-11 19:48:36 UTC (rev 226790)
@@ -1,3 +1,13 @@
+2018-01-11  Youenn Fablet  <[email protected]>
+
+        Replace WebRTCLegacyAPIDisabled by WebRTCLegacyAPIEnabled and switch off WebRTC legacy flag by default
+        https://bugs.webkit.org/show_bug.cgi?id=181480
+
+        Reviewed by Eric Carlson.
+
+        * WebView/WebPreferences.mm:
+        (+[WebPreferences initialize]):
+
 2018-01-08  David Kilzer  <[email protected]>
 
         Fix check-webkit-style issues with WebNSDataExtras.mm

Modified: trunk/Source/WebKitLegacy/mac/WebView/WebPreferences.mm (226789 => 226790)


--- trunk/Source/WebKitLegacy/mac/WebView/WebPreferences.mm	2018-01-11 19:22:46 UTC (rev 226789)
+++ trunk/Source/WebKitLegacy/mac/WebView/WebPreferences.mm	2018-01-11 19:48:36 UTC (rev 226790)
@@ -658,7 +658,7 @@
 #endif
 #if ENABLE(WEB_RTC)
         [NSNumber numberWithBool:YES], WebKitPeerConnectionEnabledPreferenceKey,
-        [NSNumber numberWithBool:YES], WebKitWebRTCLegacyAPIEnabledPreferenceKey,
+        [NSNumber numberWithBool:NO], WebKitWebRTCLegacyAPIEnabledPreferenceKey,
 #endif
 #if ENABLE(INTERSECTION_OBSERVER)
         @NO, WebKitIntersectionObserverEnabledPreferenceKey,
_______________________________________________
webkit-changes mailing list
[email protected]
https://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to