Title: [241287] trunk/Source/WebKit
Revision
241287
Author
[email protected]
Date
2019-02-11 17:50:34 -0800 (Mon, 11 Feb 2019)

Log Message

Remove noisy and unnecessary logs added in r241223
https://bugs.webkit.org/show_bug.cgi?id=194427

* UIProcess/Launcher/mac/ProcessLauncherMac.mm:
(WebKit::ProcessLauncher::launchProcess):
Errors occur all the time in the XPC connection's event handler, like when a process terminates.
We don't need to print an "error" every time any process terminates.

Modified Paths

Diff

Modified: trunk/Source/WebKit/ChangeLog (241286 => 241287)


--- trunk/Source/WebKit/ChangeLog	2019-02-12 01:16:19 UTC (rev 241286)
+++ trunk/Source/WebKit/ChangeLog	2019-02-12 01:50:34 UTC (rev 241287)
@@ -1,3 +1,13 @@
+2019-02-11  Alex Christensen  <[email protected]>
+
+        Remove noisy and unnecessary logs added in r241223
+        https://bugs.webkit.org/show_bug.cgi?id=194427
+
+        * UIProcess/Launcher/mac/ProcessLauncherMac.mm:
+        (WebKit::ProcessLauncher::launchProcess):
+        Errors occur all the time in the XPC connection's event handler, like when a process terminates.
+        We don't need to print an "error" every time any process terminates.
+
 2019-02-11  Brian Burg  <[email protected]>
 
         [Cocoa] Web Automation: client callbacks are not called if delegate does not override

Modified: trunk/Source/WebKit/UIProcess/Launcher/mac/ProcessLauncherMac.mm (241286 => 241287)


--- trunk/Source/WebKit/UIProcess/Launcher/mac/ProcessLauncherMac.mm	2019-02-12 01:16:19 UTC (rev 241286)
+++ trunk/Source/WebKit/UIProcess/Launcher/mac/ProcessLauncherMac.mm	2019-02-12 01:50:34 UTC (rev 241287)
@@ -203,9 +203,6 @@
     auto errorHandlerImpl = [weakProcessLauncher = makeWeakPtr(*this), listeningPort] (xpc_object_t event) {
         ASSERT(!event || xpc_get_type(event) == XPC_TYPE_ERROR);
 
-        if (event)
-            LOG_ERROR("Error launching auxiliary process: %s", xpc_dictionary_get_string(event, XPC_ERROR_KEY_DESCRIPTION));
-
         auto processLauncher = weakProcessLauncher.get();
         if (!processLauncher)
             return;
_______________________________________________
webkit-changes mailing list
[email protected]
https://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to