[WebRTC] Add support for runtime PeeConnection setting, required after r209757
authorcommit-queue@webkit.org <commit-queue@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Thu, 22 Dec 2016 09:22:42 +0000 (09:22 +0000)
committercommit-queue@webkit.org <commit-queue@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Thu, 22 Dec 2016 09:22:42 +0000 (09:22 +0000)
https://bugs.webkit.org/show_bug.cgi?id=166346

Patch by Alejandro G. Castro <alex@igalia.com> on 2016-12-22
Reviewed by Carlos Garcia Campos.

Source/WebKit2:

After that commit we have to enable the peerConnection runtime
flag if we want webrtc to work. We are not adding a new API for
the moment to control the peerConnection feature.

* UIProcess/API/gtk/WebKitSettings.cpp:
(webkit_settings_set_enable_media_stream): Use the mediaStream
setting to control the peerConnection

Tools:

Enable WebRTC by default to simplify testing.

* MiniBrowser/gtk/main.c:
(main):

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@210096 268f45cc-cd09-0410-ab3c-d52691b4dbfc

Source/WebKit2/ChangeLog
Source/WebKit2/UIProcess/API/gtk/WebKitSettings.cpp
Tools/ChangeLog
Tools/MiniBrowser/gtk/main.c

index 6d88e99..7b3dcc5 100644 (file)
@@ -1,3 +1,18 @@
+2016-12-22  Alejandro G. Castro  <alex@igalia.com>
+
+        [WebRTC] Add support for runtime PeeConnection setting, required after r209757
+        https://bugs.webkit.org/show_bug.cgi?id=166346
+
+        Reviewed by Carlos Garcia Campos.
+
+        After that commit we have to enable the peerConnection runtime
+        flag if we want webrtc to work. We are not adding a new API for
+        the moment to control the peerConnection feature.
+
+        * UIProcess/API/gtk/WebKitSettings.cpp:
+        (webkit_settings_set_enable_media_stream): Use the mediaStream
+        setting to control the peerConnection
+
 2016-12-21  Brent Fulgham  <bfulgham@apple.com>
 
         [Mac][WK2] Stop using file* rules in WebProcess sandbox profiles
index d48d928..e233c91 100644 (file)
@@ -2973,6 +2973,7 @@ void webkit_settings_set_enable_media_stream(WebKitSettings* settings, gboolean
         return;
 
     priv->preferences->setMediaStreamEnabled(enabled);
+    priv->preferences->setPeerConnectionEnabled(enabled);
     g_object_notify(G_OBJECT(settings), "enable-media-stream");
 }
 
index 00f4a65..a9530f9 100644 (file)
@@ -1,3 +1,15 @@
+2016-12-22  Alejandro G. Castro  <alex@igalia.com>
+
+        [WebRTC] Add support for runtime PeeConnection setting, required after r209757
+        https://bugs.webkit.org/show_bug.cgi?id=166346
+
+        Reviewed by Carlos Garcia Campos.
+
+        Enable WebRTC by default to simplify testing.
+
+        * MiniBrowser/gtk/main.c:
+        (main):
+
 2016-12-21  Alexey Proskuryakov  <ap@apple.com>
 
         REGRESSION: API test failure: WKWebView.LocalStorageClear
index 9af3286..34b9a85 100644 (file)
@@ -269,6 +269,7 @@ int main(int argc, char *argv[])
     WebKitSettings *webkitSettings = webkit_settings_new();
     webkit_settings_set_enable_developer_extras(webkitSettings, TRUE);
     webkit_settings_set_enable_webgl(webkitSettings, TRUE);
+    webkit_settings_set_enable_media_stream(webkitSettings, TRUE);
     if (!addSettingsGroupToContext(context, webkitSettings))
         g_clear_object(&webkitSettings);