Title: [175736] branches/safari-600.2-branch/Source/_javascript_Core
Revision
175736
Author
matthew_han...@apple.com
Date
2014-11-06 18:51:31 -0800 (Thu, 06 Nov 2014)

Log Message

Rollout r173238. rdar://problem/18495806

Modified Paths

Diff

Modified: branches/safari-600.2-branch/Source/_javascript_Core/ChangeLog (175735 => 175736)


--- branches/safari-600.2-branch/Source/_javascript_Core/ChangeLog	2014-11-07 02:27:49 UTC (rev 175735)
+++ branches/safari-600.2-branch/Source/_javascript_Core/ChangeLog	2014-11-07 02:51:31 UTC (rev 175736)
@@ -1,21 +1,7 @@
-2014-10-21  Dana Burkart  <dburk...@apple.com>
+2014-11-06  Matthew Hanson  <matthew_han...@apple.com>
 
-        Merge r173238
+        Rollout r173238. rdar://problem/18495806
 
-    2014-09-03  Joseph Pecoraro  <pecor...@apple.com>
-
-            Avoid warning if a process does not have access to com.apple.webinspector
-            https://bugs.webkit.org/show_bug.cgi?id=136473
-
-            Reviewed by Alexey Proskuryakov.
-
-            Pre-check for access to the mach port to avoid emitting warnings
-            in syslog for processes that do not have access.
-
-            * inspector/remote/RemoteInspector.mm:
-            (Inspector::canAccessWebInspectorMachPort):
-            (Inspector::RemoteInspector::shared):
-
 2014-08-21  Matthew Hanson  <matthew_han...@apple.com>
 
         Merge r172707. <rdar://problem/18043281>

Modified: branches/safari-600.2-branch/Source/_javascript_Core/inspector/remote/RemoteInspector.mm (175735 => 175736)


--- branches/safari-600.2-branch/Source/_javascript_Core/inspector/remote/RemoteInspector.mm	2014-11-07 02:27:49 UTC (rev 175735)
+++ branches/safari-600.2-branch/Source/_javascript_Core/inspector/remote/RemoteInspector.mm	2014-11-07 02:51:31 UTC (rev 175736)
@@ -39,28 +39,12 @@
 #import <wtf/text/WTFString.h>
 #import <xpc/xpc.h>
 
-#if __has_include(<sandbox/private.h>)
-#import <sandbox/private.h>
-#else
-enum sandbox_filter_type {
-    SANDBOX_FILTER_GLOBAL_NAME,
-};
-#endif
-extern "C" {
-    int sandbox_check(pid_t, const char *operation, enum sandbox_filter_type, ...);
-}
-
 #if PLATFORM(IOS)
 #import <wtf/ios/WebCoreThread.h>
 #endif
 
 namespace Inspector {
 
-static bool canAccessWebInspectorMachPort()
-{
-    return sandbox_check(getpid(), "mach-lookup", SANDBOX_FILTER_GLOBAL_NAME, WIRXPCMachPortName) == 0;
-}
-
 static void dispatchAsyncOnQueueSafeForAnyDebuggable(void (^block)())
 {
 #if PLATFORM(IOS)
@@ -86,11 +70,9 @@
 
     static dispatch_once_t once;
     dispatch_once(&once, ^{
-        if (canAccessWebInspectorMachPort()) {
-            JSC::initializeThreading();
-            if (RemoteInspector::startEnabled)
-                shared.get().start();
-        }
+        JSC::initializeThreading();
+        if (RemoteInspector::startEnabled)
+            shared.get().start();
     });
 
     return shared;
_______________________________________________
webkit-changes mailing list
webkit-changes@lists.webkit.org
https://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to