Title: [225871] tags/Safari-605.1.18/Source

Diff

Modified: tags/Safari-605.1.18/Source/WebCore/ChangeLog (225870 => 225871)


--- tags/Safari-605.1.18/Source/WebCore/ChangeLog	2017-12-13 21:46:24 UTC (rev 225870)
+++ tags/Safari-605.1.18/Source/WebCore/ChangeLog	2017-12-13 21:58:19 UTC (rev 225871)
@@ -1,3 +1,20 @@
+2017-12-13  Jason Marcell  <[email protected]>
+
+        Cherry-pick r225863. rdar://problem/35996158
+
+    2017-12-13  Per Arne Vollan  <[email protected]>
+
+            REGRESSION(225597): Can't select a text box or web view on a page when VO is on.
+            https://bugs.webkit.org/show_bug.cgi?id=180756
+            <rdar://problem/35996158>
+
+            Reviewed by Brent Fulgham.
+
+            No new tests.
+
+            * platform/mac/EventLoopMac.mm:
+            (WebCore::EventLoop::cycle):
+
 2017-12-12  Zalan Bujtas  <[email protected]>
 
         is<HTMLTextFormControlElement> reports the input type.

Modified: tags/Safari-605.1.18/Source/WebCore/platform/mac/EventLoopMac.mm (225870 => 225871)


--- tags/Safari-605.1.18/Source/WebCore/platform/mac/EventLoopMac.mm	2017-12-13 21:46:24 UTC (rev 225870)
+++ tags/Safari-605.1.18/Source/WebCore/platform/mac/EventLoopMac.mm	2017-12-13 21:58:19 UTC (rev 225871)
@@ -30,10 +30,12 @@
 
 void EventLoop::cycle()
 {
+#if __MAC_OS_X_VERSION_MIN_REQUIRED >= 101400
     if (![NSApp isRunning]) {
         [[NSRunLoop currentRunLoop] runUntilDate:[NSDate dateWithTimeIntervalSinceNow:0.05]];
         return;
     }
+#endif
     [NSApp setWindowsNeedUpdate:YES];
     if (NSEvent *event = [NSApp nextEventMatchingMask:NSEventMaskAny untilDate:[NSDate dateWithTimeIntervalSinceNow:0.05] inMode:NSDefaultRunLoopMode dequeue:YES])
         [NSApp sendEvent:event];

Modified: tags/Safari-605.1.18/Source/WebKit/ChangeLog (225870 => 225871)


--- tags/Safari-605.1.18/Source/WebKit/ChangeLog	2017-12-13 21:46:24 UTC (rev 225870)
+++ tags/Safari-605.1.18/Source/WebKit/ChangeLog	2017-12-13 21:58:19 UTC (rev 225871)
@@ -1,3 +1,23 @@
+2017-12-13  Jason Marcell  <[email protected]>
+
+        Cherry-pick r225863. rdar://problem/35996158
+
+    2017-12-13  Per Arne Vollan  <[email protected]>
+
+            REGRESSION(225597): Can't select a text box or web view on a page when VO is on.
+            https://bugs.webkit.org/show_bug.cgi?id=180756
+            <rdar://problem/35996158>
+
+            Reviewed by Brent Fulgham.
+
+            * Platform/IPC/mac/ConnectionMac.mm:
+            (IPC::AccessibilityProcessSuspendedNotification):
+            * UIProcess/Launcher/mac/ProcessLauncherMac.mm:
+            (WebKit::shouldLeakBoost):
+            * WebProcess/EntryPoint/mac/XPCService/WebContentService/Info-OSX.plist:
+            * WebProcess/WebProcess.cpp:
+            (WebKit::WebProcess::initializeProcess):
+
 2017-12-13  Carlos Garcia Campos  <[email protected]>
 
         Unreviewed. Update OptionsGTK.cmake and NEWS for 2.19.3 release.

Modified: tags/Safari-605.1.18/Source/WebKit/Platform/IPC/mac/ConnectionMac.mm (225870 => 225871)


--- tags/Safari-605.1.18/Source/WebKit/Platform/IPC/mac/ConnectionMac.mm	2017-12-13 21:46:24 UTC (rev 225870)
+++ tags/Safari-605.1.18/Source/WebKit/Platform/IPC/mac/ConnectionMac.mm	2017-12-13 21:58:19 UTC (rev 225871)
@@ -586,9 +586,11 @@
 static void AccessibilityProcessSuspendedNotification(bool suspended)
 {
 #if PLATFORM(MAC)
+#if __MAC_OS_X_VERSION_MIN_REQUIRED >= 101400
     // Calling _AXUIElementNotifyProcessSuspendStatus will crash if the NSApplication event loop is not running.
     if (![NSApp isRunning])
         return;
+#endif
     _AXUIElementNotifyProcessSuspendStatus(suspended ? AXSuspendStatusSuspended : AXSuspendStatusRunning);
 #elif PLATFORM(IOS)
     UIAccessibilityPostNotification(kAXPidStatusChangedNotification, @{ @"pid" : @(getpid()), @"suspended" : @(suspended) });

Modified: tags/Safari-605.1.18/Source/WebKit/UIProcess/Launcher/mac/ProcessLauncherMac.mm (225870 => 225871)


--- tags/Safari-605.1.18/Source/WebKit/UIProcess/Launcher/mac/ProcessLauncherMac.mm	2017-12-13 21:46:24 UTC (rev 225870)
+++ tags/Safari-605.1.18/Source/WebKit/UIProcess/Launcher/mac/ProcessLauncherMac.mm	2017-12-13 21:58:19 UTC (rev 225871)
@@ -76,9 +76,13 @@
     UNUSED_PARAM(launchOptions);
     return true;
 #else
+#if __MAC_OS_X_VERSION_MIN_REQUIRED >= 101400
+    // Boost the WebContent process if the NSApplication run loop is not used.
+    if (launchOptions.processType == ProcessLauncher::ProcessType::Web)
+        return true;
+#endif
     // On Mac, leak a boost onto the NetworkProcess.
-    return launchOptions.processType == ProcessLauncher::ProcessType::Network
-        || launchOptions.processType == ProcessLauncher::ProcessType::Web;
+    return launchOptions.processType == ProcessLauncher::ProcessType::Network;
 #endif
 }
 

Modified: tags/Safari-605.1.18/Source/WebKit/WebProcess/EntryPoint/mac/XPCService/WebContentService/Info-OSX.plist (225870 => 225871)


--- tags/Safari-605.1.18/Source/WebKit/WebProcess/EntryPoint/mac/XPCService/WebContentService/Info-OSX.plist	2017-12-13 21:46:24 UTC (rev 225870)
+++ tags/Safari-605.1.18/Source/WebKit/WebProcess/EntryPoint/mac/XPCService/WebContentService/Info-OSX.plist	2017-12-13 21:58:19 UTC (rev 225871)
@@ -45,7 +45,7 @@
 		<key>ServiceType</key>
 		<string>Application</string>
 		<key>RunLoopType</key>
-		<string>NSRunLoop</string>
+		<string>_NSApplicationMain</string>
 		<key>_MultipleInstances</key>
 		<true/>
 		<key>_HighBitsASLR</key>

Modified: tags/Safari-605.1.18/Source/WebKit/WebProcess/WebProcess.cpp (225870 => 225871)


--- tags/Safari-605.1.18/Source/WebKit/WebProcess/WebProcess.cpp	2017-12-13 21:46:24 UTC (rev 225870)
+++ tags/Safari-605.1.18/Source/WebKit/WebProcess/WebProcess.cpp	2017-12-13 21:58:19 UTC (rev 225871)
@@ -216,7 +216,7 @@
 
 void WebProcess::initializeProcess(const ChildProcessInitializationParameters& parameters)
 {
-#if PLATFORM(COCOA) && !PLATFORM(IOS)
+#if PLATFORM(MAC) && __MAC_OS_X_VERSION_MIN_REQUIRED >= 101400
     // This call is needed when the WebProcess is not running the NSApplication event loop.
     // Otherwise, calling enableSandboxStyleFileQuarantine() will fail.
     launchServicesCheckIn();
_______________________________________________
webkit-changes mailing list
[email protected]
https://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to