Title: [271965] trunk/Source
- Revision
- 271965
- Author
- [email protected]
- Date
- 2021-01-27 10:36:53 -0800 (Wed, 27 Jan 2021)
Log Message
[macOS] Re-enable ENABLE_SET_WEBCONTENT_PROCESS_INFORMATION_IN_NETWORK_PROCESS
https://bugs.webkit.org/show_bug.cgi?id=221039
<rdar://problem/73665061>
Source/WebCore:
Reviewed by Brent Fulgham.
The commit r271907 introduced a crash, since it denied access to the Launch Services daemon, which AppKit is using to get and set
application information, for example whether the app is foreground or not. To work around this, the WebCoreThemeWindow has been made
borderless, and the method +[NSWindow invalidateRestorableState] is overridden to do nothing. Running the default
+[NSWindow invalidateRestorableState] will exercise code which tries to reach out to the Launch Services daemon, and crashes if
unable to do so. This functionality should not be required in the WebContent process, and can be disabled.
* platform/mac/ThemeMac.mm:
(-[WebCoreThemeWindow invalidateRestorableState]):
(-[WebCoreThemeView window]):
Source/WebKit:
Reviewed by Brent Fulgham.
Work around crash introduced in r271907, by disabling persistent UI. Having persistent UI enabled will exercise code
which tries to connect with the Launch Services daemon, and crash if that fails.
* WebProcess/cocoa/WebProcessCocoa.mm:
(WebKit::WebProcess::platformInitializeWebProcess):
Source/WTF:
Reviewed by Brent Fulgham.
* wtf/PlatformEnableCocoa.h:
Modified Paths
Diff
Modified: trunk/Source/WTF/ChangeLog (271964 => 271965)
--- trunk/Source/WTF/ChangeLog 2021-01-27 18:07:39 UTC (rev 271964)
+++ trunk/Source/WTF/ChangeLog 2021-01-27 18:36:53 UTC (rev 271965)
@@ -1,3 +1,13 @@
+2021-01-27 Per Arne <[email protected]>
+
+ [macOS] Re-enable ENABLE_SET_WEBCONTENT_PROCESS_INFORMATION_IN_NETWORK_PROCESS
+ https://bugs.webkit.org/show_bug.cgi?id=221039
+ <rdar://problem/73665061>
+
+ Reviewed by Brent Fulgham.
+
+ * wtf/PlatformEnableCocoa.h:
+
2021-01-27 Antti Koivisto <[email protected]>
Implement visibility based resource load scheduling for low priority resources
Modified: trunk/Source/WTF/wtf/PlatformEnableCocoa.h (271964 => 271965)
--- trunk/Source/WTF/wtf/PlatformEnableCocoa.h 2021-01-27 18:07:39 UTC (rev 271964)
+++ trunk/Source/WTF/wtf/PlatformEnableCocoa.h 2021-01-27 18:36:53 UTC (rev 271965)
@@ -630,7 +630,7 @@
#define ENABLE_VORBIS 1
#endif
-#if !defined(ENABLE_SET_WEBCONTENT_PROCESS_INFORMATION_IN_NETWORK_PROCESS) && PLATFORM(MAC) && __MAC_OS_X_VERSION_MIN_REQUIRED > 120000
+#if !defined(ENABLE_SET_WEBCONTENT_PROCESS_INFORMATION_IN_NETWORK_PROCESS) && PLATFORM(MAC) && __MAC_OS_X_VERSION_MIN_REQUIRED >= 120000
#define ENABLE_SET_WEBCONTENT_PROCESS_INFORMATION_IN_NETWORK_PROCESS 1
#endif
Modified: trunk/Source/WebCore/ChangeLog (271964 => 271965)
--- trunk/Source/WebCore/ChangeLog 2021-01-27 18:07:39 UTC (rev 271964)
+++ trunk/Source/WebCore/ChangeLog 2021-01-27 18:36:53 UTC (rev 271965)
@@ -1,3 +1,21 @@
+2021-01-27 Per Arne <[email protected]>
+
+ [macOS] Re-enable ENABLE_SET_WEBCONTENT_PROCESS_INFORMATION_IN_NETWORK_PROCESS
+ https://bugs.webkit.org/show_bug.cgi?id=221039
+ <rdar://problem/73665061>
+
+ Reviewed by Brent Fulgham.
+
+ The commit r271907 introduced a crash, since it denied access to the Launch Services daemon, which AppKit is using to get and set
+ application information, for example whether the app is foreground or not. To work around this, the WebCoreThemeWindow has been made
+ borderless, and the method +[NSWindow invalidateRestorableState] is overridden to do nothing. Running the default
+ +[NSWindow invalidateRestorableState] will exercise code which tries to reach out to the Launch Services daemon, and crashes if
+ unable to do so. This functionality should not be required in the WebContent process, and can be disabled.
+
+ * platform/mac/ThemeMac.mm:
+ (-[WebCoreThemeWindow invalidateRestorableState]):
+ (-[WebCoreThemeView window]):
+
2021-01-27 Sihui Liu <[email protected]>
Use SFTranscription instead of SFTranscriptionSegment for SpeechRecognitionAlternative
Modified: trunk/Source/WebCore/platform/mac/ThemeMac.mm (271964 => 271965)
--- trunk/Source/WebCore/platform/mac/ThemeMac.mm 2021-01-27 18:07:39 UTC (rev 271964)
+++ trunk/Source/WebCore/platform/mac/ThemeMac.mm 2021-01-27 18:36:53 UTC (rev 271965)
@@ -67,6 +67,9 @@
return themeWindowHasKeyAppearance;
}
+- (void)invalidateRestorableState
+{
+}
@end
@implementation WebCoreThemeView
@@ -76,7 +79,7 @@
// Using defer:YES prevents us from wasting any window server resources for this window, since we're not actually
// going to draw into it. The other arguments match what you get when calling -[NSWindow init].
static WebCoreThemeWindow *window = [[WebCoreThemeWindow alloc] initWithContentRect:NSMakeRect(100, 100, 100, 100)
- styleMask:NSWindowStyleMaskTitled backing:NSBackingStoreBuffered defer:YES];
+ styleMask:NSWindowStyleMaskBorderless backing:NSBackingStoreBuffered defer:YES];
return window;
}
Modified: trunk/Source/WebKit/ChangeLog (271964 => 271965)
--- trunk/Source/WebKit/ChangeLog 2021-01-27 18:07:39 UTC (rev 271964)
+++ trunk/Source/WebKit/ChangeLog 2021-01-27 18:36:53 UTC (rev 271965)
@@ -1,5 +1,19 @@
2021-01-27 Per Arne <[email protected]>
+ [macOS] Re-enable ENABLE_SET_WEBCONTENT_PROCESS_INFORMATION_IN_NETWORK_PROCESS
+ https://bugs.webkit.org/show_bug.cgi?id=221039
+ <rdar://problem/73665061>
+
+ Reviewed by Brent Fulgham.
+
+ Work around crash introduced in r271907, by disabling persistent UI. Having persistent UI enabled will exercise code
+ which tries to connect with the Launch Services daemon, and crash if that fails.
+
+ * WebProcess/cocoa/WebProcessCocoa.mm:
+ (WebKit::WebProcess::platformInitializeWebProcess):
+
+2021-01-27 Per Arne <[email protected]>
+
[iOS] Fix sandbox violation file-read-data /dev/dtracehelper in the Networking process
https://bugs.webkit.org/show_bug.cgi?id=221046
<rdar://problem/73625718>
Modified: trunk/Source/WebKit/WebProcess/cocoa/WebProcessCocoa.mm (271964 => 271965)
--- trunk/Source/WebKit/WebProcess/cocoa/WebProcessCocoa.mm 2021-01-27 18:07:39 UTC (rev 271964)
+++ trunk/Source/WebKit/WebProcess/cocoa/WebProcessCocoa.mm 2021-01-27 18:36:53 UTC (rev 271965)
@@ -293,7 +293,7 @@
// We don't need to talk to the Dock.
[NSApplication _preventDockConnections];
- [[NSUserDefaults standardUserDefaults] registerDefaults:@{ @"NSApplicationCrashOnExceptions" : @YES }];
+ [[NSUserDefaults standardUserDefaults] registerDefaults:@{ @"NSApplicationCrashOnExceptions" : @YES, @"ApplePersistence" : @NO }];
// rdar://9118639 accessibilityFocusedUIElement in NSApplication defaults to use the keyWindow. Since there's
// no window in WK2, NSApplication needs to use the focused page's focused element.
_______________________________________________
webkit-changes mailing list
[email protected]
https://lists.webkit.org/mailman/listinfo/webkit-changes
