Title: [288647] branches/safari-611.3.10.1-branch/Source

Diff

Modified: branches/safari-611.3.10.1-branch/Source/WTF/ChangeLog (288646 => 288647)


--- branches/safari-611.3.10.1-branch/Source/WTF/ChangeLog	2022-01-26 22:35:47 UTC (rev 288646)
+++ branches/safari-611.3.10.1-branch/Source/WTF/ChangeLog	2022-01-26 22:49:36 UTC (rev 288647)
@@ -1,42 +1,3 @@
-2021-08-21  Kocsen Chung  <[email protected]>
-
-        Cherry-pick r281384. rdar://problem/82195071
-
-    IndexedDB: crash when triggering IDBOpenRequest completion back on a worker thread
-    https://bugs.webkit.org/show_bug.cgi?id=229375
-    
-    Source/WebCore:
-    
-    Reviewed by Brady Eidson.
-    
-    Client may dispatch custom events to an IDBRequest, and we should only change request state based on events
-    created internally.
-    
-    * Modules/indexeddb/IDBRequest.cpp:
-    (WebCore::IDBRequest::dispatchEvent):
-    
-    Source/WTF:
-    
-    Protect callee in CrossThreadTask if it inherits from ThreadSafeRefCounted<T>.
-    
-    Reviewed by Brady Eidson.
-    
-    * wtf/CrossThreadTask.h:
-    
-    
-    git-svn-id: https://svn.webkit.org/repository/webkit/trunk@281384 268f45cc-cd09-0410-ab3c-d52691b4dbfc
-
-    2021-08-21  Sihui Liu  <[email protected]>
-
-            IndexedDB: crash when triggering IDBOpenRequest completion back on a worker thread
-            https://bugs.webkit.org/show_bug.cgi?id=229375
-
-            Protect callee in CrossThreadTask if it inherits from ThreadSafeRefCounted<T>.
-
-            Reviewed by Brady Eidson.
-
-            * wtf/CrossThreadTask.h:
-
 2021-06-10  Russell Epstein  <[email protected]>
 
         Cherry-pick r278177. rdar://problem/79144899

Modified: branches/safari-611.3.10.1-branch/Source/WTF/wtf/CrossThreadTask.h (288646 => 288647)


--- branches/safari-611.3.10.1-branch/Source/WTF/wtf/CrossThreadTask.h	2022-01-26 22:35:47 UTC (rev 288646)
+++ branches/safari-611.3.10.1-branch/Source/WTF/wtf/CrossThreadTask.h	2022-01-26 22:49:36 UTC (rev 288647)
@@ -85,7 +85,7 @@
     callMemberFunctionForCrossThreadTaskImpl(object, function, std::forward<ArgsTuple>(args), ArgsIndicies());
 }
 
-template<typename T, typename std::enable_if<std::is_base_of<ThreadSafeRefCountedBase, T>::value, int>::type = 0, typename... Parameters, typename... Arguments>
+template<typename T, typename std::enable_if<std::is_base_of<ThreadSafeRefCounted<T>, T>::value, int>::type = 0, typename... Parameters, typename... Arguments>
 CrossThreadTask createCrossThreadTask(T& callee, void (T::*method)(Parameters...), const Arguments&... arguments)
 {
     return CrossThreadTask([callee = makeRefPtr(&callee), method, arguments = std::make_tuple(crossThreadCopy(arguments)...)]() mutable {
@@ -93,7 +93,7 @@
     });
 }
 
-template<typename T, typename std::enable_if<!std::is_base_of<ThreadSafeRefCountedBase, T>::value, int>::type = 0, typename... Parameters, typename... Arguments>
+template<typename T, typename std::enable_if<!std::is_base_of<ThreadSafeRefCounted<T>, T>::value, int>::type = 0, typename... Parameters, typename... Arguments>
 CrossThreadTask createCrossThreadTask(T& callee, void (T::*method)(Parameters...), const Arguments&... arguments)
 {
     return CrossThreadTask([callee = &callee, method, arguments = std::make_tuple(crossThreadCopy(arguments)...)]() mutable {

Modified: branches/safari-611.3.10.1-branch/Source/WebCore/ChangeLog (288646 => 288647)


--- branches/safari-611.3.10.1-branch/Source/WebCore/ChangeLog	2022-01-26 22:35:47 UTC (rev 288646)
+++ branches/safari-611.3.10.1-branch/Source/WebCore/ChangeLog	2022-01-26 22:49:36 UTC (rev 288647)
@@ -43,47 +43,6 @@
             * html/parser/HTMLConstructionSite.cpp:
             (WebCore::HTMLConstructionSite::mergeAttributesFromTokenIntoElement):
 
-2021-08-21  Kocsen Chung  <[email protected]>
-
-        Cherry-pick r281384. rdar://problem/82195071
-
-    IndexedDB: crash when triggering IDBOpenRequest completion back on a worker thread
-    https://bugs.webkit.org/show_bug.cgi?id=229375
-    
-    Source/WebCore:
-    
-    Reviewed by Brady Eidson.
-    
-    Client may dispatch custom events to an IDBRequest, and we should only change request state based on events
-    created internally.
-    
-    * Modules/indexeddb/IDBRequest.cpp:
-    (WebCore::IDBRequest::dispatchEvent):
-    
-    Source/WTF:
-    
-    Protect callee in CrossThreadTask if it inherits from ThreadSafeRefCounted<T>.
-    
-    Reviewed by Brady Eidson.
-    
-    * wtf/CrossThreadTask.h:
-    
-    
-    git-svn-id: https://svn.webkit.org/repository/webkit/trunk@281384 268f45cc-cd09-0410-ab3c-d52691b4dbfc
-
-    2021-08-21  Sihui Liu  <[email protected]>
-
-            IndexedDB: crash when triggering IDBOpenRequest completion back on a worker thread
-            https://bugs.webkit.org/show_bug.cgi?id=229375
-
-            Reviewed by Brady Eidson.
-
-            Client may dispatch custom events to an IDBRequest, and we should only change request state based on events
-            created internally.
-
-            * Modules/indexeddb/IDBRequest.cpp:
-            (WebCore::IDBRequest::dispatchEvent):
-
 2021-07-02  Ruben Turcios  <[email protected]>
 
         Cherry-pick r278964. rdar://problem/79474157

Modified: branches/safari-611.3.10.1-branch/Source/WebCore/Modules/indexeddb/IDBRequest.cpp (288646 => 288647)


--- branches/safari-611.3.10.1-branch/Source/WebCore/Modules/indexeddb/IDBRequest.cpp	2022-01-26 22:35:47 UTC (rev 288646)
+++ branches/safari-611.3.10.1-branch/Source/WebCore/Modules/indexeddb/IDBRequest.cpp	2022-01-26 22:49:36 UTC (rev 288647)
@@ -310,8 +310,7 @@
     else if (m_transaction && !m_transaction->didDispatchAbortOrCommit())
         targets = { this, m_transaction.get(), &m_transaction->database() };
 
-    if (event.isTrusted())
-        m_hasPendingActivity = false;
+    m_hasPendingActivity = false;
 
     {
         TransactionActivator activator(m_transaction.get());
_______________________________________________
webkit-changes mailing list
[email protected]
https://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to