Changeset 246937 in webkit

Timestamp:

Jun 28, 2019 2:01:59 PM (5 years ago)

Author:

Alan Coon

Message:

Cherry-pick r246835. rdar://problem/51787961

[ContentChangeObserver] Dispatch synthetic mouse event asynchronously in completePendingSyntheticClickForContentChangeObserver
​https://bugs.webkit.org/show_bug.cgi?id=199220
<rdar://problem/51787961>

Reviewed by Simon Fraser.

Source/WebKit:

WebPage::completePendingSyntheticClickForContentChangeObserver should not dispatch mouse events synchronously.
Mouse events, through style updates could destroy the element that initiated this change.
WebPage::handleSyntheticClick() already implements this pattern.

• WebProcess/WebPage/ios/WebPageIOS.mm: (WebKit::WebPage::completePendingSyntheticClickForContentChangeObserver):

LayoutTests:

• fast/events/touch/ios/content-observation/animation-end-with-visiblity-change-crash-expected.txt: Added.
• fast/events/touch/ios/content-observation/animation-end-with-visiblity-change-crash.html: Added.

git-svn-id: ​https://svn.webkit.org/repository/webkit/trunk@246835 268f45cc-cd09-0410-ab3c-d52691b4dbfc

Location:

tags/Safari-608.1.32.1

Files:

• LayoutTests/ChangeLog (modified) (1 diff)
• LayoutTests/fast/events/touch/ios/content-observation/animation-end-with-visiblity-change-crash-expected.txt (added)
• LayoutTests/fast/events/touch/ios/content-observation/animation-end-with-visiblity-change-crash.html (added)
• Source/WebKit/ChangeLog (modified) (1 diff)
• Source/WebKit/WebProcess/WebPage/ios/WebPageIOS.mm (modified) (1 diff)

tags/Safari-608.1.32.1/LayoutTests/ChangeLog

@@ +1 @@
+2019-06-28  Kocsen Chung  <kocsen_chung@apple.com>
+
+        Cherry-pick r246835. rdar://problem/51787961
+
+    [ContentChangeObserver] Dispatch synthetic mouse event asynchronously in completePendingSyntheticClickForContentChangeObserver
+    https://bugs.webkit.org/show_bug.cgi?id=199220
+    <rdar://problem/51787961>
+    
+    Reviewed by Simon Fraser.
+    
+    Source/WebKit:
+    
+    WebPage::completePendingSyntheticClickForContentChangeObserver should not dispatch mouse events synchronously.
+    Mouse events, through style updates could destroy the element that initiated this change.
+    WebPage::handleSyntheticClick() already implements this pattern.
+    
+    * WebProcess/WebPage/ios/WebPageIOS.mm:
+    (WebKit::WebPage::completePendingSyntheticClickForContentChangeObserver):
+    
+    LayoutTests:
+    
+    * fast/events/touch/ios/content-observation/animation-end-with-visiblity-change-crash-expected.txt: Added.
+    * fast/events/touch/ios/content-observation/animation-end-with-visiblity-change-crash.html: Added.
+    
+    
+    git-svn-id: https://svn.webkit.org/repository/webkit/trunk@246835 268f45cc-cd09-0410-ab3c-d52691b4dbfc
+
+    2019-06-26  Zalan Bujtas  <zalan@apple.com>
+
+            [ContentChangeObserver] Dispatch synthetic mouse event asynchronously in completePendingSyntheticClickForContentChangeObserver
+            https://bugs.webkit.org/show_bug.cgi?id=199220
+            <rdar://problem/51787961>
+
+            Reviewed by Simon Fraser.
+
+            * fast/events/touch/ios/content-observation/animation-end-with-visiblity-change-crash-expected.txt: Added.
+            * fast/events/touch/ios/content-observation/animation-end-with-visiblity-change-crash.html: Added.
+
 2019-06-25  Russell Epstein  <russell_e@apple.com>
 

tags/Safari-608.1.32.1/Source/WebKit/ChangeLog

@@ +1 @@
+2019-06-28  Kocsen Chung  <kocsen_chung@apple.com>
+
+        Cherry-pick r246835. rdar://problem/51787961
+
+    [ContentChangeObserver] Dispatch synthetic mouse event asynchronously in completePendingSyntheticClickForContentChangeObserver
+    https://bugs.webkit.org/show_bug.cgi?id=199220
+    <rdar://problem/51787961>
+    
+    Reviewed by Simon Fraser.
+    
+    Source/WebKit:
+    
+    WebPage::completePendingSyntheticClickForContentChangeObserver should not dispatch mouse events synchronously.
+    Mouse events, through style updates could destroy the element that initiated this change.
+    WebPage::handleSyntheticClick() already implements this pattern.
+    
+    * WebProcess/WebPage/ios/WebPageIOS.mm:
+    (WebKit::WebPage::completePendingSyntheticClickForContentChangeObserver):
+    
+    LayoutTests:
+    
+    * fast/events/touch/ios/content-observation/animation-end-with-visiblity-change-crash-expected.txt: Added.
+    * fast/events/touch/ios/content-observation/animation-end-with-visiblity-change-crash.html: Added.
+    
+    
+    git-svn-id: https://svn.webkit.org/repository/webkit/trunk@246835 268f45cc-cd09-0410-ab3c-d52691b4dbfc
+
+    2019-06-26  Zalan Bujtas  <zalan@apple.com>
+
+            [ContentChangeObserver] Dispatch synthetic mouse event asynchronously in completePendingSyntheticClickForContentChangeObserver
+            https://bugs.webkit.org/show_bug.cgi?id=199220
+            <rdar://problem/51787961>
+
+            Reviewed by Simon Fraser.
+
+            WebPage::completePendingSyntheticClickForContentChangeObserver should not dispatch mouse events synchronously.
+            Mouse events, through style updates could destroy the element that initiated this change.
+            WebPage::handleSyntheticClick() already implements this pattern.
+
+            * WebProcess/WebPage/ios/WebPageIOS.mm:
+            (WebKit::WebPage::completePendingSyntheticClickForContentChangeObserver):
+
 2019-06-28  Kocsen Chung  <kocsen_chung@apple.com>
 

tags/Safari-608.1.32.1/Source/WebKit/WebProcess/WebPage/ios/WebPageIOS.mm

@@ -715 +715 @@
         return;
     auto observedContentChange = m_pendingSyntheticClickNode->document().contentChangeObserver().observedContentChange();
-    // Only dispatch the click if the document didn't get changed by any timers started by the move event.
-    if (observedContentChange == WKContentNoChange) {
-        LOG(ContentObservation, "No chage was observed -> click.");
-        completeSyntheticClick(*m_pendingSyntheticClickNode, m_pendingSyntheticClickLocation, m_pendingSyntheticClickModifiers, WebCore::OneFingerTap, m_pendingSyntheticClickPointerId);
-    } else {
+    callOnMainThread([protectedThis = makeRefPtr(this), targetNode = Ref<Node>(*m_pendingSyntheticClickNode), originalDocument = makeWeakPtr(m_pendingSyntheticClickNode->document()), observedContentChange, location = m_pendingSyntheticClickLocation, modifiers = m_pendingSyntheticClickModifiers, pointerId = m_pendingSyntheticClickPointerId] {
+        if (protectedThis->m_isClosed || !protectedThis->corePage())
+            return;
+        if (!originalDocument || &targetNode->document() != originalDocument)
+            return;
+
+        // Only dispatch the click if the document didn't get changed by any timers started by the move event.
+        if (observedContentChange == WKContentNoChange) {
+            LOG(ContentObservation, "No chage was observed -> click.");
+            protectedThis->completeSyntheticClick(targetNode, location, modifiers, WebCore::OneFingerTap, pointerId);
+            return;
+        }
         // Ensure that the mouse is on the most recent content.
-        dispatchSyntheticMouseMove(m_page->mainFrame(), m_pendingSyntheticClickLocation, m_pendingSyntheticClickModifiers, m_pendingSyntheticClickPointerId);
         LOG(ContentObservation, "Observed meaningful visible change -> hover.");
-    }
-
+        dispatchSyntheticMouseMove(protectedThis->corePage()->mainFrame(), location, modifiers, pointerId);
+    });
     m_pendingSyntheticClickNode = nullptr;
-    m_pendingSyntheticClickLocation = FloatPoint();
+    m_pendingSyntheticClickLocation = { };
     m_pendingSyntheticClickModifiers = { };
     m_pendingSyntheticClickPointerId = 0;