Attachment 348624 Details for Bug 189188 – Patch

[patch] Patch

bug-189188-20180831113838.patch (text/plain), 3.02 KB, created by Frédéric Wang (:fredw) on 2018-08-31 02:38:39 PDT

(hide)

Description:

Filename:

MIME Type:

Creator: Frédéric Wang (:fredw)

Created: 2018-08-31 02:38:39 PDT

Size: 3.02 KB

patch

obsolete

>Subversion Revision: 235540
>diff --git a/Source/WebKit/ChangeLog b/Source/WebKit/ChangeLog
>index f82d9f4ee2f863a2c363f6ea5b9317b85a465f73..f8c1de4de2a97f21d11c5da55825d7a6b9b8dfdb 100644
>--- a/Source/WebKit/ChangeLog
>+++ b/Source/WebKit/ChangeLog
>@@ -1,3 +1,18 @@
>+2018-08-31  Frederic Wang  <fwang@igalia.com>
>+
>+        REGRESSION(r235484) _WKPreferencesGetWebRTCLegacyAPIEnabled symbol not found when running Safari with external SDK build
>+        https://bugs.webkit.org/show_bug.cgi?id=189188
>+
>+        Reviewed by NOBODY (OOPS!).
>+
>+        WebRTCLegacyAPI preference has been removed in r235484. However, the C API is still used by
>+        Safari when running external SDK builds. Hence we restore some dummy getter/setter for now.
>+
>+        * UIProcess/API/C/WKPreferences.cpp:
>+        (WKPreferencesSetWebRTCLegacyAPIEnabled): no-op.
>+        (WKPreferencesGetWebRTCLegacyAPIEnabled): Always return false.
>+        * UIProcess/API/C/WKPreferencesRef.h:
>+
> 2018-08-31  Frederic Wang  <fwang@igalia.com>
> 
>         Bug 182053 - [CSSOM View] Implement standard behavior for scrollingElement
>diff --git a/Source/WebKit/UIProcess/API/C/WKPreferences.cpp b/Source/WebKit/UIProcess/API/C/WKPreferences.cpp
>index 2e1ce2206bb50aa58b698be768f3d624d0ef18ad..dd271b59df760890445fed1d63338f1d9f608871 100644
>--- a/Source/WebKit/UIProcess/API/C/WKPreferences.cpp
>+++ b/Source/WebKit/UIProcess/API/C/WKPreferences.cpp
>@@ -1428,6 +1428,15 @@ bool WKPreferencesGetPeerConnectionEnabled(WKPreferencesRef preferencesRef)
>     return toImpl(preferencesRef)->peerConnectionEnabled();
> }
> 
>+void WKPreferencesSetWebRTCLegacyAPIEnabled(WKPreferencesRef, bool)
>+{
>+}
>+
>+bool WKPreferencesGetWebRTCLegacyAPIEnabled(WKPreferencesRef)
>+{
>+    return false;
>+}
>+
> void WKPreferencesSetWebRTCMDNSICECandidatesEnabled(WKPreferencesRef preferencesRef, bool enabled)
> {
>     toImpl(preferencesRef)->setMDNSICECandidatesEnabled(enabled);
>diff --git a/Source/WebKit/UIProcess/API/C/WKPreferencesRef.h b/Source/WebKit/UIProcess/API/C/WKPreferencesRef.h
>index 1c2fbb4aea2f69d5bf16f445d382f300570baf63..5a088db3c9101b728e847078b23fe77eb89d06c4 100644
>--- a/Source/WebKit/UIProcess/API/C/WKPreferencesRef.h
>+++ b/Source/WebKit/UIProcess/API/C/WKPreferencesRef.h
>@@ -268,6 +268,11 @@ WK_EXPORT bool WKPreferencesGetMediaDevicesEnabled(WKPreferencesRef preferencesR
> WK_EXPORT void WKPreferencesSetPeerConnectionEnabled(WKPreferencesRef preferencesRef, bool enabled);
> WK_EXPORT bool WKPreferencesGetPeerConnectionEnabled(WKPreferencesRef preferencesRef);
> 
>+// This is for backward compatibility with external SDK builds (https://webkit.org/b/189188).
>+// FIXME: Remove these API functions.
>+WK_EXPORT void WKPreferencesSetWebRTCLegacyAPIEnabled(WKPreferencesRef preferencesRef, bool enabled);
>+WK_EXPORT bool WKPreferencesGetWebRTCLegacyAPIEnabled(WKPreferencesRef preferencesRef);
>+
> // Defaults to false
> WK_EXPORT void WKPreferencesSetWebRTCMDNSICECandidatesEnabled(WKPreferencesRef preferencesRef, bool enabled);
> WK_EXPORT bool WKPreferencesGetWebRTCMDNSICECandidatesEnabled(WKPreferencesRef preferencesRef);

Actions: View | Formatted Diff | Diff

Attachments on bug 189188: 348624 | 348637