Title: [229885] trunk/Source/WebKit
Revision
229885
Author
mcatanz...@igalia.com
Date
2018-03-23 00:56:08 -0700 (Fri, 23 Mar 2018)

Log Message

[WPE][GTK] Fix -Wswitch warnings after r229778
https://bugs.webkit.org/show_bug.cgi?id=183927

Reviewed by Carlos Garcia Campos.

* NetworkProcess/soup/NetworkDataTaskSoup.cpp:
(WebKit::NetworkDataTaskSoup::dispatchDidReceiveResponse): Log a warning if Suspend is
received as a PolicyAction
* UIProcess/API/glib/WebKitNavigationClient.cpp: Ignore the ProcessSwap termination reason,
which should be invisible to WebKit clients

Modified Paths

Diff

Modified: trunk/Source/WebKit/ChangeLog (229884 => 229885)


--- trunk/Source/WebKit/ChangeLog	2018-03-23 07:49:14 UTC (rev 229884)
+++ trunk/Source/WebKit/ChangeLog	2018-03-23 07:56:08 UTC (rev 229885)
@@ -1,3 +1,16 @@
+2018-03-23  Michael Catanzaro  <mcatanz...@igalia.com>
+
+        [WPE][GTK] Fix -Wswitch warnings after r229778
+        https://bugs.webkit.org/show_bug.cgi?id=183927
+
+        Reviewed by Carlos Garcia Campos.
+
+        * NetworkProcess/soup/NetworkDataTaskSoup.cpp:
+        (WebKit::NetworkDataTaskSoup::dispatchDidReceiveResponse): Log a warning if Suspend is
+        received as a PolicyAction
+        * UIProcess/API/glib/WebKitNavigationClient.cpp: Ignore the ProcessSwap termination reason,
+        which should be invisible to WebKit clients
+
 2018-03-22  Chris Dumez  <cdu...@apple.com>
 
         Include security origin in the service worker process name

Modified: trunk/Source/WebKit/NetworkProcess/soup/NetworkDataTaskSoup.cpp (229884 => 229885)


--- trunk/Source/WebKit/NetworkProcess/soup/NetworkDataTaskSoup.cpp	2018-03-23 07:49:14 UTC (rev 229884)
+++ trunk/Source/WebKit/NetworkProcess/soup/NetworkDataTaskSoup.cpp	2018-03-23 07:56:08 UTC (rev 229885)
@@ -385,6 +385,9 @@
                 ASSERT_NOT_REACHED();
 
             break;
+        case PolicyAction::Suspend:
+            LOG_ERROR("PolicyAction::Suspend encountered - Treating as PolicyAction::Ignore for now");
+            FALLTHROUGH;
         case PolicyAction::Ignore:
             clearRequest();
             break;

Modified: trunk/Source/WebKit/UIProcess/API/glib/WebKitNavigationClient.cpp (229884 => 229885)


--- trunk/Source/WebKit/UIProcess/API/glib/WebKitNavigationClient.cpp	2018-03-23 07:49:14 UTC (rev 229884)
+++ trunk/Source/WebKit/UIProcess/API/glib/WebKitNavigationClient.cpp	2018-03-23 07:56:08 UTC (rev 229885)
@@ -116,6 +116,7 @@
             break;
         case ProcessTerminationReason::ExceededCPULimit:
         case ProcessTerminationReason::RequestedByClient:
+        case ProcessTerminationReason::NavigationSwap:
             break;
         }
     }
_______________________________________________
webkit-changes mailing list
webkit-changes@lists.webkit.org
https://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to