Attachment 358803 Details for Bug 193167 – Patch

[patch] Patch

bug-193167-20190110100533.patch (text/plain), 3.79 KB, created by Sihui Liu on 2019-01-10 10:05:34 PST

(hide)

Description:

Filename:

MIME Type:

Creator: Sihui Liu

Created: 2019-01-10 10:05:34 PST

Size: 3.79 KB

patch

obsolete

>Subversion Revision: 239778
>diff --git a/Source/WebCore/ChangeLog b/Source/WebCore/ChangeLog
>index 63394d701f2531582bcb2ddc2a6d087d7b10b596..6c7326d1b12c2d2579970535b46c6a4187777bde 100644
>--- a/Source/WebCore/ChangeLog
>+++ b/Source/WebCore/ChangeLog
>@@ -1,3 +1,23 @@
>+2019-01-10  Sihui Liu  <sihui_liu@apple.com>
>+
>+        IndexedDB: leak IDBTransaction, TransactionOperation and IDBRequest in layout tests
>+        https://bugs.webkit.org/show_bug.cgi?id=193167
>+        <rdar://problem/46891688>
>+
>+        Reviewed by NOBODY (OOPS!).
>+
>+        Do some cleanup to break retain cycle when context is stopped. 
>+
>+        * Modules/indexeddb/IDBOpenDBRequest.cpp:
>+        (WebCore::IDBOpenDBRequest::fireSuccessAfterVersionChangeCommit):
>+        (WebCore::IDBOpenDBRequest::cancelForStop):
>+        * Modules/indexeddb/IDBOpenDBRequest.h:
>+        * Modules/indexeddb/IDBTransaction.cpp:
>+        (WebCore::IDBTransaction::abortOnServerAndCancelRequests):
>+        (WebCore::IDBTransaction::stop):
>+        * Modules/indexeddb/client/TransactionOperation.h:
>+        (WebCore::IDBClient::TransactionOperation::doComplete):
>+
> 2019-01-09  Zalan Bujtas  <zalan@apple.com>
> 
>         [LFC][BFC][MarginCollapsing] Add support for peculiar cases.
>diff --git a/Source/WebCore/Modules/indexeddb/IDBOpenDBRequest.cpp b/Source/WebCore/Modules/indexeddb/IDBOpenDBRequest.cpp
>index 91ce9c4cb9454cebaa104f8943cb42bf51c635d2..e91f2937c4ed0f327cb045006f47c02a04f8309a 100644
>--- a/Source/WebCore/Modules/indexeddb/IDBOpenDBRequest.cpp
>+++ b/Source/WebCore/Modules/indexeddb/IDBOpenDBRequest.cpp
>@@ -115,6 +115,9 @@ void IDBOpenDBRequest::fireErrorAfterVersionChangeCompletion()
> void IDBOpenDBRequest::cancelForStop()
> {
>     connectionProxy().openDBRequestCancelled({ connectionProxy(), *this });
>+    
>+    if (m_transaction && m_transaction->isVersionChange())
>+        m_transaction->removeRequest(*this);
> }
> 
> void IDBOpenDBRequest::dispatchEvent(Event& event)
>diff --git a/Source/WebCore/Modules/indexeddb/IDBTransaction.cpp b/Source/WebCore/Modules/indexeddb/IDBTransaction.cpp
>index b82b66a1d23f7af54383b2933c253ec433613626..b869177c066ba03832e3b8052c1aec0617a83742 100644
>--- a/Source/WebCore/Modules/indexeddb/IDBTransaction.cpp
>+++ b/Source/WebCore/Modules/indexeddb/IDBTransaction.cpp
>@@ -297,11 +297,12 @@ void IDBTransaction::abortOnServerAndCancelRequests(IDBClient::TransactionOperat
>     abortInProgressOperations(error);
> 
>     for (auto& operation : m_abortQueue) {
>-        m_currentlyCompletingRequest = nullptr;
>         m_transactionOperationsInProgressQueue.append(operation.get());
>         operation->doComplete(IDBResultData::error(operation->identifier(), error));
>+        m_currentlyCompletingRequest = nullptr;
>     }
> 
>+    m_abortQueue.clear();
>     // Since we're aborting, it should be impossible to have queued any further operations.
>     ASSERT(m_pendingTransactionOperationQueue.isEmpty());
> }
>@@ -338,6 +339,9 @@ void IDBTransaction::stop()
> 
>     m_contextStopped = true;
> 
>+    if (isVersionChange())
>+        m_openDBRequest = nullptr;
>+
>     if (isFinishedOrFinishing())
>         return;
> 
>diff --git a/Source/WebCore/Modules/indexeddb/client/TransactionOperation.h b/Source/WebCore/Modules/indexeddb/client/TransactionOperation.h
>index 8ddb56e178b72ef21a666eb169dfc16ee9892de3..a46a1f411f593b459172d52c6faeee428e0251a1 100644
>--- a/Source/WebCore/Modules/indexeddb/client/TransactionOperation.h
>+++ b/Source/WebCore/Modules/indexeddb/client/TransactionOperation.h
>@@ -98,6 +98,8 @@ public:
>         // so we need to do this trick to null it out without first destroying it.
>         WTF::Function<void (const IDBResultData&)> oldCompleteFunction;
>         std::swap(m_completeFunction, oldCompleteFunction);
>+        
>+        m_performFunction = { };
>     }
> 
>     const IDBResourceIdentifier& identifier() const { return m_identifier; }

Actions: View | Formatted Diff | Diff

Attachments on bug 193167: 358421 | 358803 | 358815