Title: [233616] trunk/Source/WebKit
Revision
233616
Author
[email protected]
Date
2018-07-07 10:50:54 -0700 (Sat, 07 Jul 2018)

Log Message

REGRESSION (r233561): MobileMail crashes when replying to an email
https://bugs.webkit.org/show_bug.cgi?id=187436
rdar://problem/41931915

Reviewed by Alan Bujtas.

r233561 added a RELEASE_ASSERT(isMainThread()) in this function. This is fine most of the time,
until Mail spawns a WebThread for mail compose, at which point the meaning of isMainThread changes
and the RELEASE_ASSERT fires.

For now, just remove the RELEASE_ASSERT.

* UIProcess/API/Cocoa/WKWebView.mm:
(-[WKWebView _scheduleVisibleContentRectUpdateAfterScrollInView:]):

Modified Paths

Diff

Modified: trunk/Source/WebKit/ChangeLog (233615 => 233616)


--- trunk/Source/WebKit/ChangeLog	2018-07-07 16:20:30 UTC (rev 233615)
+++ trunk/Source/WebKit/ChangeLog	2018-07-07 17:50:54 UTC (rev 233616)
@@ -1,3 +1,20 @@
+2018-07-07  Simon Fraser  <[email protected]>
+
+        REGRESSION (r233561): MobileMail crashes when replying to an email
+        https://bugs.webkit.org/show_bug.cgi?id=187436
+        rdar://problem/41931915
+
+        Reviewed by Alan Bujtas.
+        
+        r233561 added a RELEASE_ASSERT(isMainThread()) in this function. This is fine most of the time,
+        until Mail spawns a WebThread for mail compose, at which point the meaning of isMainThread changes
+        and the RELEASE_ASSERT fires.
+        
+        For now, just remove the RELEASE_ASSERT.
+
+        * UIProcess/API/Cocoa/WKWebView.mm:
+        (-[WKWebView _scheduleVisibleContentRectUpdateAfterScrollInView:]):
+
 2018-07-06  Timothy Hatcher  <[email protected]>
 
         Semantic colors don't update when accessibility Increase Contrast mode is enabled.

Modified: trunk/Source/WebKit/UIProcess/API/Cocoa/WKWebView.mm (233615 => 233616)


--- trunk/Source/WebKit/UIProcess/API/Cocoa/WKWebView.mm	2018-07-07 16:20:30 UTC (rev 233615)
+++ trunk/Source/WebKit/UIProcess/API/Cocoa/WKWebView.mm	2018-07-07 17:50:54 UTC (rev 233616)
@@ -2730,8 +2730,6 @@
 
 - (void)_scheduleVisibleContentRectUpdateAfterScrollInView:(UIScrollView *)scrollView
 {
-    RELEASE_ASSERT(isMainThread());
-
     _visibleContentRectUpdateScheduledFromScrollViewInStableState = [self _scrollViewIsInStableState:scrollView];
 
     if (_hasScheduledVisibleRectUpdate) {
_______________________________________________
webkit-changes mailing list
[email protected]
https://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to