Attachment 362402 Details for Bug 194806 – Patch

[patch] Patch

bug-194806-20190219113027.patch (text/plain), 23.51 KB, created by Sihui Liu on 2019-02-19 11:30:28 PST

(hide)

Description:

Filename:

MIME Type:

Creator: Sihui Liu

Created: 2019-02-19 11:30:28 PST

Size: 23.51 KB

patch

obsolete

>Subversion Revision: 241761
>diff --git a/Source/WebCore/ChangeLog b/Source/WebCore/ChangeLog
>index a6da53fe8e04ad79dd06c117c410a040b2b97e76..9034176ace6ad41d4b21dd987d7c95636be1d958 100644
>--- a/Source/WebCore/ChangeLog
>+++ b/Source/WebCore/ChangeLog
>@@ -1,3 +1,37 @@
>+2019-02-19  Sihui Liu  <sihui_liu@apple.com>
>+
>+        IndexedDB: re-enable some leak tests
>+        https://bugs.webkit.org/show_bug.cgi?id=194806
>+
>+        Protected JSIDBCursor object when advance/continue request on IDBCursor is not finished, because after the 
>+        advance operation completes on success, we need to return the same JSIDBCursor object as before the advance, 
>+        and during the wait for advance operation to complete, we need to return error as the result. 
>+ 
>+        Covered by existing tests.
>+
>+        * Modules/indexeddb/IDBCursor.cpp:
>+        (WebCore::IDBCursor::setGetResult): Marked the wrapper as stale instead of assigning it a new value which may
>+        require a lock.
>+
>+        * Modules/indexeddb/IDBCursor.h:
>+        * Modules/indexeddb/IDBRequest.cpp:
>+        (WebCore::IDBRequest::setResult): Removed unused lock.
>+        (WebCore::IDBRequest::setResultToStructuredClone): Ditto.
>+        (WebCore::IDBRequest::setResultToUndefined): Ditto.
>+        (WebCore::IDBRequest::willIterateCursor):
>+        (WebCore::IDBRequest::didOpenOrIterateCursor):
>+        * Modules/indexeddb/IDBRequest.h:
>+        (WebCore::IDBRequest::cursorWrapper):
>+        * bindings/js/JSIDBRequestCustom.cpp:
>+        (WebCore::JSIDBRequest::visitAdditionalChildren):
>+
>+        * bindings/js/JSValueInWrappedObject.h: 
>+        (WebCore::JSValueInWrappedObject::setIsStale): Added a way to mark the value invalid and ask for re-cache.
>+        (WebCore::JSValueInWrappedObject::isStale):
>+        (WebCore::JSValueInWrappedObject::JSValueInWrappedObject):
>+        (WebCore::JSValueInWrappedObject::operator=):
>+        (WebCore::cachedPropertyValue):subl 
>+
> 2019-02-19  Commit Queue  <commit-queue@webkit.org>
> 
>         Unreviewed, rolling out r241722.
>diff --git a/Source/WebCore/Modules/indexeddb/IDBCursor.cpp b/Source/WebCore/Modules/indexeddb/IDBCursor.cpp
>index 253a3b158bf5041af1aeb865011ecfbcf206f3cc..b7cc3061552db14dc8c73104fce38063e2ad32bc 100644
>--- a/Source/WebCore/Modules/indexeddb/IDBCursor.cpp
>+++ b/Source/WebCore/Modules/indexeddb/IDBCursor.cpp
>@@ -309,14 +309,14 @@ ExceptionOr<Ref<WebCore::IDBRequest>> IDBCursor::deleteFunction(ExecState& state
>     return WTFMove(request);
> }
> 
>-void IDBCursor::setGetResult(IDBRequest&, const IDBGetResult& getResult)
>+bool IDBCursor::setGetResult(IDBRequest&, const IDBGetResult& getResult)
> {
>     LOG(IndexedDB, "IDBCursor::setGetResult - current key %s", getResult.keyData().loggingString().substring(0, 100).utf8().data());
>     ASSERT(&effectiveObjectStore().transaction().database().originThread() == &Thread::current());
> 
>-    m_keyWrapper = { };
>-    m_primaryKeyWrapper = { };
>-    m_valueWrapper = { };
>+    m_keyWrapper.setIsStale(true);
>+    m_primaryKeyWrapper.setIsStale(true);
>+    m_valueWrapper.setIsStale(true);
> 
>     if (!getResult.isDefined()) {
>         m_keyData = { };
>@@ -326,7 +326,7 @@ void IDBCursor::setGetResult(IDBRequest&, const IDBGetResult& getResult)
>         m_value = { };
> 
>         m_gotValue = false;
>-        return;
>+        return false;
>     }
> 
>     m_keyData = getResult.keyData();
>@@ -338,6 +338,7 @@ void IDBCursor::setGetResult(IDBRequest&, const IDBGetResult& getResult)
>         m_value = getResult.value();
> 
>     m_gotValue = true;
>+    return true;
> }
> 
> } // namespace WebCore
>diff --git a/Source/WebCore/Modules/indexeddb/IDBCursor.h b/Source/WebCore/Modules/indexeddb/IDBCursor.h
>index 03e6b1eb24fe414df7a7af602055a3c44e753a93..0a378bde940b4514ac9b2a980007bd7ca9ddecc1 100644
>--- a/Source/WebCore/Modules/indexeddb/IDBCursor.h
>+++ b/Source/WebCore/Modules/indexeddb/IDBCursor.h
>@@ -76,7 +76,7 @@ public:
>     void clearRequest() { m_request.clear(); }
>     IDBRequest* request() { return m_request.get(); }
> 
>-    void setGetResult(IDBRequest&, const IDBGetResult&);
>+    bool setGetResult(IDBRequest&, const IDBGetResult&);
> 
>     virtual bool isKeyCursorWithValue() const { return false; }
> 
>diff --git a/Source/WebCore/Modules/indexeddb/IDBRequest.cpp b/Source/WebCore/Modules/indexeddb/IDBRequest.cpp
>index cec7c9a4f898628eff168bc7fa54b0799c18c013..a369df57b5d020764726b5b17875e854a322325e 100644
>--- a/Source/WebCore/Modules/indexeddb/IDBRequest.cpp
>+++ b/Source/WebCore/Modules/indexeddb/IDBRequest.cpp
>@@ -354,66 +354,32 @@ void IDBRequest::setResult(const IDBKeyData& keyData)
> {
>     ASSERT(&originThread() == &Thread::current());
> 
>-    auto* context = scriptExecutionContext();
>-    if (!context)
>-        return;
>-
>-    auto* state = context->execState();
>-    if (!state)
>-        return;
>-
>-    VM& vm = context->vm();
>-    JSLockHolder lock(vm);
>     m_result = keyData;
>-    m_resultWrapper = { };
>+    m_resultWrapper.setIsStale(true);
> }
> 
> void IDBRequest::setResult(const Vector<IDBKeyData>& keyDatas)
> {
>     ASSERT(&originThread() == &Thread::current());
> 
>-    auto* context = scriptExecutionContext();
>-    if (!context)
>-        return;
>-
>-    auto* state = context->execState();
>-    if (!state)
>-        return;
>-
>-    VM& vm = context->vm();
>-    JSLockHolder lock(vm);
>     m_result = keyDatas;
>-    m_resultWrapper = { };
>+    m_resultWrapper.setIsStale(true);
> }
> 
> void IDBRequest::setResult(const Vector<IDBValue>& values)
> {
>     ASSERT(&originThread() == &Thread::current());
> 
>-    auto* context = scriptExecutionContext();
>-    if (!context)
>-        return;
>-
>-    auto* state = context->execState();
>-    if (!state)
>-        return;
>-
>-    VM& vm = context->vm();
>-    JSLockHolder lock(vm);
>     m_result = values;
>-    m_resultWrapper = { };
>+    m_resultWrapper.setIsStale(true);
> }
> 
> void IDBRequest::setResult(uint64_t number)
> {
>     ASSERT(&originThread() == &Thread::current());
> 
>-    auto* context = scriptExecutionContext();
>-    if (!context)
>-        return;
>-
>     m_result = number;
>-    m_resultWrapper = { };
>+    m_resultWrapper.setIsStale(true);
> }
> 
> void IDBRequest::setResultToStructuredClone(const IDBValue& value)
>@@ -422,18 +388,8 @@ void IDBRequest::setResultToStructuredClone(const IDBValue& value)
> 
>     LOG(IndexedDB, "IDBRequest::setResultToStructuredClone");
> 
>-    auto* context = scriptExecutionContext();
>-    if (!context)
>-        return;
>-
>-    auto* state = context->execState();
>-    if (!state)
>-        return;
>-
>-    VM& vm = context->vm();
>-    JSLockHolder lock(vm);
>     m_result = value;
>-    m_resultWrapper = { };
>+    m_resultWrapper.setIsStale(true);
> }
> 
> void IDBRequest::setResultToUndefined()
>@@ -441,7 +397,7 @@ void IDBRequest::setResultToUndefined()
>     ASSERT(&originThread() == &Thread::current());
> 
>     m_result = NullResultType::Undefined;
>-    m_resultWrapper = { };
>+    m_resultWrapper.setIsStale(true);
> }
> 
> IDBCursor* IDBRequest::resultCursor()
>@@ -466,7 +422,9 @@ void IDBRequest::willIterateCursor(IDBCursor& cursor)
>     m_pendingCursor = &cursor;
>     m_hasPendingActivity = true;
>     m_result = NullResultType::Empty;
>-    m_resultWrapper = { };
>+    if (m_resultWrapper && !m_resultWrapper.isStale())
>+        m_cursorWrapper = m_resultWrapper;
>+    m_resultWrapper.setIsStale(true);
>     m_readyState = ReadyState::Pending;
>     m_domError = nullptr;
>     m_idbError = IDBError { };
>@@ -478,10 +436,14 @@ void IDBRequest::didOpenOrIterateCursor(const IDBResultData& resultData)
>     ASSERT(m_pendingCursor);
> 
>     m_result = NullResultType::Empty;
>-    m_resultWrapper = { };
>+    m_resultWrapper.setIsStale(true);
> 
>     if (resultData.type() == IDBResultType::IterateCursorSuccess || resultData.type() == IDBResultType::OpenCursorSuccess) {
>-        m_pendingCursor->setGetResult(*this, resultData.getResult());
>+        if (m_pendingCursor->setGetResult(*this, resultData.getResult()) && m_cursorWrapper) {
>+            m_resultWrapper = m_cursorWrapper;
>+            m_resultWrapper.setIsStale(false);
>+        }
>+
>         if (resultData.getResult().isDefined())
>             m_result = m_pendingCursor;
>     }
>@@ -527,7 +489,7 @@ void IDBRequest::setResult(Ref<IDBDatabase>&& database)
>     ASSERT(&originThread() == &Thread::current());
> 
>     m_result = RefPtr<IDBDatabase> { WTFMove(database) };
>-    m_resultWrapper = { };
>+    m_resultWrapper.setIsStale(true);
> }
> 
> } // namespace WebCore
>diff --git a/Source/WebCore/Modules/indexeddb/IDBRequest.h b/Source/WebCore/Modules/indexeddb/IDBRequest.h
>index 613512b67f756cc1d708e0bdebcb26cef744f630..b37a249e0ee48e5ddf6f143962a0dde45b0db144 100644
>--- a/Source/WebCore/Modules/indexeddb/IDBRequest.h
>+++ b/Source/WebCore/Modules/indexeddb/IDBRequest.h
>@@ -78,6 +78,7 @@ public:
>     using Result = Variant<RefPtr<IDBCursor>, RefPtr<IDBDatabase>, IDBKeyData, Vector<IDBKeyData>, IDBValue, Vector<IDBValue>, uint64_t, NullResultType>;
>     ExceptionOr<Result> result() const;
>     JSValueInWrappedObject& resultWrapper() { return m_resultWrapper; }
>+    JSValueInWrappedObject& cursorWrapper() { return m_cursorWrapper; }
> 
>     using Source = Variant<RefPtr<IDBObjectStore>, RefPtr<IDBIndex>, RefPtr<IDBCursor>>;
>     const Optional<Source>& source() const { return m_source; }
>@@ -171,6 +172,7 @@ private:
>     IDBResourceIdentifier m_resourceIdentifier;
> 
>     JSValueInWrappedObject m_resultWrapper;
>+    JSValueInWrappedObject m_cursorWrapper;
>     Result m_result;
>     Optional<Source> m_source;
> 
>diff --git a/Source/WebCore/bindings/js/JSIDBRequestCustom.cpp b/Source/WebCore/bindings/js/JSIDBRequestCustom.cpp
>index 2e710a1609ad5d0d5353888cc0a9c575df2108a5..5ebb7ccc8e437ed36d9c320f17000317f2ce4eeb 100644
>--- a/Source/WebCore/bindings/js/JSIDBRequestCustom.cpp
>+++ b/Source/WebCore/bindings/js/JSIDBRequestCustom.cpp
>@@ -75,6 +75,7 @@ void JSIDBRequest::visitAdditionalChildren(SlotVisitor& visitor)
> {
>     auto& request = wrapped();
>     request.resultWrapper().visit(visitor);
>+    request.cursorWrapper().visit(visitor);
> }
> 
> }
>diff --git a/Source/WebCore/bindings/js/JSValueInWrappedObject.h b/Source/WebCore/bindings/js/JSValueInWrappedObject.h
>index 07aefb74ab1202ea0a69780ba6945fb0f7f2a87f..cab4fc5178fbc22d57ef195b03b71f8f46a91593 100644
>--- a/Source/WebCore/bindings/js/JSValueInWrappedObject.h
>+++ b/Source/WebCore/bindings/js/JSValueInWrappedObject.h
>@@ -36,10 +36,16 @@ namespace WebCore {
> class JSValueInWrappedObject {
> public:
>     JSValueInWrappedObject(JSC::JSValue = { });
>+    JSValueInWrappedObject(JSValueInWrappedObject&);
>     operator JSC::JSValue() const;
>     explicit operator bool() const;
>+    JSValueInWrappedObject& operator=(JSValueInWrappedObject& other);
>+    JSValueInWrappedObject& operator=(JSC::JSValue);
>     void visit(JSC::SlotVisitor&) const;
> 
>+    void setIsStale(bool isStale) { m_isStale = isStale; };
>+    bool isStale() { return m_isStale; }
>+
> private:
>     // Use a weak pointer here so that if this code or client code has a visiting mistake,
>     // we get null rather than a dangling pointer to a deleted object.
>@@ -48,6 +54,7 @@ private:
>     using Value = Variant<JSC::JSValue, Weak>;
>     static Value makeValue(JSC::JSValue);
>     Value m_value;
>+    bool m_isStale { false };
> };
> 
> JSC::JSValue cachedPropertyValue(JSC::ExecState&, const JSDOMObject& owner, JSValueInWrappedObject& cacheSlot, const WTF::Function<JSC::JSValue()>&);
>@@ -66,6 +73,11 @@ inline auto JSValueInWrappedObject::makeValue(JSC::JSValue value) -> Value
> }
> 
> inline JSValueInWrappedObject::JSValueInWrappedObject(JSC::JSValue value)
>+    : m_value(makeValue(JSC::JSValue(value)))
>+{
>+}
>+
>+inline JSValueInWrappedObject::JSValueInWrappedObject(JSValueInWrappedObject& value)
>     : m_value(makeValue(value))
> {
> }
>@@ -84,6 +96,18 @@ inline JSValueInWrappedObject::operator bool() const
>     return JSC::JSValue { *this }.operator bool();
> }
> 
>+inline JSValueInWrappedObject& JSValueInWrappedObject::operator=(JSValueInWrappedObject& other)
>+{
>+    m_value = makeValue(JSC::JSValue(other));
>+    return *this;
>+}
>+
>+inline JSValueInWrappedObject& JSValueInWrappedObject::operator=(JSC::JSValue value)
>+{
>+    m_value = makeValue(value);
>+    return *this;
>+}
>+
> inline void JSValueInWrappedObject::visit(JSC::SlotVisitor& visitor) const
> {
>     return WTF::switchOn(m_value, [] (JSC::JSValue) {
>@@ -95,9 +119,10 @@ inline void JSValueInWrappedObject::visit(JSC::SlotVisitor& visitor) const
> 
> inline JSC::JSValue cachedPropertyValue(JSC::ExecState& state, const JSDOMObject& owner, JSValueInWrappedObject& cachedValue, const WTF::Function<JSC::JSValue()>& function)
> {
>-    if (cachedValue && isWorldCompatible(state, cachedValue))
>+    if (cachedValue && !cachedValue.isStale() && isWorldCompatible(state, cachedValue))
>         return cachedValue;
>     cachedValue = cloneAcrossWorlds(state, owner, function());
>+    cachedValue.setIsStale(false);
>     ASSERT(isWorldCompatible(state, cachedValue));
>     return cachedValue;
> }
>diff --git a/LayoutTests/ChangeLog b/LayoutTests/ChangeLog
>index e26eaf6e984904ac4398e158332c2c689e209500..4f8b1793df46bda64ed2fe48c0faaf8847d6cfd3 100644
>--- a/LayoutTests/ChangeLog
>+++ b/LayoutTests/ChangeLog
>@@ -1,3 +1,46 @@
>+2019-02-19  Sihui Liu  <sihui_liu@apple.com>
>+
>+        IndexedDB: re-enable some leak tests
>+        https://bugs.webkit.org/show_bug.cgi?id=194806
>+
>+        Reviewed by NOBODY (OOPS!).
>+
>+        We have support for internals.observeGC now.
>+
>+        * TestExpectations:
>+        * platform/win/TestExpectations: skip the tests on win because of unknown bugs failing gc-related tests: 
>+        webkit.org/b/193540.
>+
>+        * storage/indexeddb/connection-leak-expected.txt:
>+        * storage/indexeddb/connection-leak-private-expected.txt:
>+        * storage/indexeddb/cursor-leak-expected.txt:
>+        * storage/indexeddb/cursor-leak-private-expected.txt:
>+        * storage/indexeddb/cursor-request-cycle-expected.txt:
>+        * storage/indexeddb/cursor-request-cycle-private-expected.txt:
>+        * storage/indexeddb/request-leak-expected.txt:
>+        * storage/indexeddb/request-leak-private-expected.txt:
>+        * storage/indexeddb/resources/cursor-request-cycle.js:
>+
>+2019-02-19  Sihui Liu  <sihui_liu@apple.com>
>+
>+        IndexedDB: re-enable some leak tests
>+        https://bugs.webkit.org/show_bug.cgi?id=194806
>+
>+        We have support for internals.observeGC now.
>+
>+        Reviewed by NOBODY (OOPS!).
>+
>+        * TestExpectations:
>+        * storage/indexeddb/connection-leak-expected.txt:
>+        * storage/indexeddb/connection-leak-private-expected.txt:
>+        * storage/indexeddb/cursor-leak-expected.txt:
>+        * storage/indexeddb/cursor-leak-private-expected.txt:
>+        * storage/indexeddb/cursor-request-cycle-expected.txt:
>+        * storage/indexeddb/cursor-request-cycle-private-expected.txt:
>+        * storage/indexeddb/request-leak-expected.txt:
>+        * storage/indexeddb/request-leak-private-expected.txt:
>+        * storage/indexeddb/resources/cursor-request-cycle.js:
>+
> 2019-02-19  Commit Queue  <commit-queue@webkit.org>
> 
>         Unreviewed, rolling out r241722.
>diff --git a/LayoutTests/TestExpectations b/LayoutTests/TestExpectations
>index c2c19fb1153ea659530df86108bd5fff7d9cb26a..bb77d16f32646199d04d94790486809edee84e99 100644
>--- a/LayoutTests/TestExpectations
>+++ b/LayoutTests/TestExpectations
>@@ -1429,18 +1429,6 @@ fast/history/page-cache-indexed-closed-db.html [ Failure ]
> # With Modern IDB and the in-memory backing store, that should change.
> storage/indexeddb/open-db-private-browsing.html [ Failure ]
> 
>-# Relies on internals.observeGC
>-storage/indexeddb/connection-leak-private.html [ Skip ]
>-storage/indexeddb/connection-leak.html [ Skip ]
>-storage/indexeddb/cursor-leak-private.html [ Failure ]
>-storage/indexeddb/cursor-leak.html [ Skip ]
>-storage/indexeddb/cursor-request-cycle-private.html [ Failure ]
>-storage/indexeddb/cursor-request-cycle.html [ Skip ]
>-storage/indexeddb/delete-closed-database-object-private.html [ Skip ]
>-storage/indexeddb/delete-closed-database-object.html [ Skip ]
>-storage/indexeddb/request-leak-private.html [ Failure ]
>-storage/indexeddb/request-leak.html [ Failure ]
>-
> webkit.org/b/154619 storage/indexeddb/odd-strings.html [ Skip ]
> 
> # IDB workers test fails - The worker's attempt to open the database creates a new UniqueIDBDatabase
>diff --git a/LayoutTests/platform/win/TestExpectations b/LayoutTests/platform/win/TestExpectations
>index 9a1ff96d6c627b81660cdf872fd3d4f9d1490353..bb0e38a6411d49b3629bc50dc11ab035b6dcb123 100644
>--- a/LayoutTests/platform/win/TestExpectations
>+++ b/LayoutTests/platform/win/TestExpectations
>@@ -4272,5 +4272,15 @@ webkit.org/b/194451 accessibility/set-value-not-work-for-disabled-sliders.html [
> 
> storage/indexeddb/result-request-cycle.html [ Skip ]
> storage/indexeddb/value-cursor-cycle.html [ Skip ]
>+storage/indexeddb/connection-leak-private.html [ Skip ]
>+storage/indexeddb/connection-leak.html [ Skip ]
>+storage/indexeddb/cursor-leak-private.html [ Skip ]
>+storage/indexeddb/cursor-leak.html [ Skip ]
>+storage/indexeddb/cursor-request-cycle-private.html [ Skip ]
>+storage/indexeddb/cursor-request-cycle.html [ Skip ]
>+storage/indexeddb/delete-closed-database-object-private.html [ Skip ]
>+storage/indexeddb/delete-closed-database-object.html [ Skip ]
>+storage/indexeddb/request-leak-private.html [ Skip ]
>+storage/indexeddb/request-leak.html [ Skip ]
> 
> webkit.org/b/194711 fast/replaced/encrypted-pdf-as-object-and-embed.html [ Failure ]
>diff --git a/LayoutTests/storage/indexeddb/connection-leak-expected.txt b/LayoutTests/storage/indexeddb/connection-leak-expected.txt
>index 9dd0cfa1d67cb2f886e67c9ebad074da162a429f..94c9186614b3ca388b8e2dfc4b2f0c5dcf9823e7 100644
>--- a/LayoutTests/storage/indexeddb/connection-leak-expected.txt
>+++ b/LayoutTests/storage/indexeddb/connection-leak-expected.txt
>@@ -3,7 +3,6 @@ Regression test to ensure that IndexedDB connections don't leak
> On success, you will see a series of "PASS" messages, followed by "TEST COMPLETE".
> 
> 
>-dbname = "connection-leak.html"
> 
> doFirstOpen():
> request = indexedDB.open(dbname, 1)
>diff --git a/LayoutTests/storage/indexeddb/connection-leak-private-expected.txt b/LayoutTests/storage/indexeddb/connection-leak-private-expected.txt
>index 9dd0cfa1d67cb2f886e67c9ebad074da162a429f..94c9186614b3ca388b8e2dfc4b2f0c5dcf9823e7 100644
>--- a/LayoutTests/storage/indexeddb/connection-leak-private-expected.txt
>+++ b/LayoutTests/storage/indexeddb/connection-leak-private-expected.txt
>@@ -3,7 +3,6 @@ Regression test to ensure that IndexedDB connections don't leak
> On success, you will see a series of "PASS" messages, followed by "TEST COMPLETE".
> 
> 
>-dbname = "connection-leak.html"
> 
> doFirstOpen():
> request = indexedDB.open(dbname, 1)
>diff --git a/LayoutTests/storage/indexeddb/cursor-leak-expected.txt b/LayoutTests/storage/indexeddb/cursor-leak-expected.txt
>index 0790c522e7d3dbff8ede8b4ae8b7f431020ac313..aeea70dc943d6497354555e96e80099c17493596 100644
>--- a/LayoutTests/storage/indexeddb/cursor-leak-expected.txt
>+++ b/LayoutTests/storage/indexeddb/cursor-leak-expected.txt
>@@ -5,7 +5,6 @@ On success, you will see a series of "PASS" messages, followed by "TEST COMPLETE
> 
> indexedDB = self.indexedDB || self.webkitIndexedDB || self.mozIndexedDB || self.msIndexedDB || self.OIndexedDB;
> 
>-dbname = "cursor-leak.html"
> indexedDB.deleteDatabase(dbname)
> indexedDB.open(dbname)
> PASS cursorObserver.wasCollected is true
>diff --git a/LayoutTests/storage/indexeddb/cursor-leak-private-expected.txt b/LayoutTests/storage/indexeddb/cursor-leak-private-expected.txt
>index 0790c522e7d3dbff8ede8b4ae8b7f431020ac313..aeea70dc943d6497354555e96e80099c17493596 100644
>--- a/LayoutTests/storage/indexeddb/cursor-leak-private-expected.txt
>+++ b/LayoutTests/storage/indexeddb/cursor-leak-private-expected.txt
>@@ -5,7 +5,6 @@ On success, you will see a series of "PASS" messages, followed by "TEST COMPLETE
> 
> indexedDB = self.indexedDB || self.webkitIndexedDB || self.mozIndexedDB || self.msIndexedDB || self.OIndexedDB;
> 
>-dbname = "cursor-leak.html"
> indexedDB.deleteDatabase(dbname)
> indexedDB.open(dbname)
> PASS cursorObserver.wasCollected is true
>diff --git a/LayoutTests/storage/indexeddb/cursor-request-cycle-expected.txt b/LayoutTests/storage/indexeddb/cursor-request-cycle-expected.txt
>index 99c720509d78e7d4d2f3d2dd34b723426086bebe..6184653dba8c8956dd8ce1c88648aee6cc11382d 100644
>--- a/LayoutTests/storage/indexeddb/cursor-request-cycle-expected.txt
>+++ b/LayoutTests/storage/indexeddb/cursor-request-cycle-expected.txt
>@@ -5,7 +5,6 @@ On success, you will see a series of "PASS" messages, followed by "TEST COMPLETE
> 
> indexedDB = self.indexedDB || self.webkitIndexedDB || self.mozIndexedDB || self.msIndexedDB || self.OIndexedDB;
> 
>-dbname = "cursor-request-cycle.html"
> indexedDB.deleteDatabase(dbname)
> indexedDB.open(dbname)
> 
>@@ -40,6 +39,7 @@ cursor.continue()
> cursor = null
> gc()
> PASS cursorObservation.wasCollected is false
>+PASS cursorRequestObservation.wasCollected is false
> finalRequest = store.get(0)
> 
> cursorContinueSuccess():
>diff --git a/LayoutTests/storage/indexeddb/cursor-request-cycle-private-expected.txt b/LayoutTests/storage/indexeddb/cursor-request-cycle-private-expected.txt
>index 99c720509d78e7d4d2f3d2dd34b723426086bebe..6184653dba8c8956dd8ce1c88648aee6cc11382d 100644
>--- a/LayoutTests/storage/indexeddb/cursor-request-cycle-private-expected.txt
>+++ b/LayoutTests/storage/indexeddb/cursor-request-cycle-private-expected.txt
>@@ -5,7 +5,6 @@ On success, you will see a series of "PASS" messages, followed by "TEST COMPLETE
> 
> indexedDB = self.indexedDB || self.webkitIndexedDB || self.mozIndexedDB || self.msIndexedDB || self.OIndexedDB;
> 
>-dbname = "cursor-request-cycle.html"
> indexedDB.deleteDatabase(dbname)
> indexedDB.open(dbname)
> 
>@@ -40,6 +39,7 @@ cursor.continue()
> cursor = null
> gc()
> PASS cursorObservation.wasCollected is false
>+PASS cursorRequestObservation.wasCollected is false
> finalRequest = store.get(0)
> 
> cursorContinueSuccess():
>diff --git a/LayoutTests/storage/indexeddb/request-leak-expected.txt b/LayoutTests/storage/indexeddb/request-leak-expected.txt
>index e1f164f4a95df03735ddc52e5a03fa91c4b4a9a8..b0386e160adc88a812d74e96b708538e1495f02e 100644
>--- a/LayoutTests/storage/indexeddb/request-leak-expected.txt
>+++ b/LayoutTests/storage/indexeddb/request-leak-expected.txt
>@@ -5,7 +5,6 @@ On success, you will see a series of "PASS" messages, followed by "TEST COMPLETE
> 
> indexedDB = self.indexedDB || self.webkitIndexedDB || self.mozIndexedDB || self.msIndexedDB || self.OIndexedDB;
> 
>-dbname = "request-leak.html"
> indexedDB.deleteDatabase(dbname)
> indexedDB.open(dbname)
> 
>diff --git a/LayoutTests/storage/indexeddb/request-leak-private-expected.txt b/LayoutTests/storage/indexeddb/request-leak-private-expected.txt
>index e1f164f4a95df03735ddc52e5a03fa91c4b4a9a8..b0386e160adc88a812d74e96b708538e1495f02e 100644
>--- a/LayoutTests/storage/indexeddb/request-leak-private-expected.txt
>+++ b/LayoutTests/storage/indexeddb/request-leak-private-expected.txt
>@@ -5,7 +5,6 @@ On success, you will see a series of "PASS" messages, followed by "TEST COMPLETE
> 
> indexedDB = self.indexedDB || self.webkitIndexedDB || self.mozIndexedDB || self.msIndexedDB || self.OIndexedDB;
> 
>-dbname = "request-leak.html"
> indexedDB.deleteDatabase(dbname)
> indexedDB.open(dbname)
> 
>diff --git a/LayoutTests/storage/indexeddb/resources/cursor-request-cycle.js b/LayoutTests/storage/indexeddb/resources/cursor-request-cycle.js
>index bbbec8c773efac2db13844b75f510646cbc1d7c6..646ce1e7b0a0d074d897b23a3a112e7d0cfa8357 100644
>--- a/LayoutTests/storage/indexeddb/resources/cursor-request-cycle.js
>+++ b/LayoutTests/storage/indexeddb/resources/cursor-request-cycle.js
>@@ -66,6 +66,7 @@ function onOpen(evt)
>         evalAndLog("cursor = null");
>         evalAndLog("gc()");
>         shouldBeFalse("cursorObservation.wasCollected");
>+        shouldBeFalse("cursorRequestObservation.wasCollected");
> 
>         evalAndLog("finalRequest = store.get(0)");
>         finalRequest.onsuccess = function finalRequestSuccess(evt) {

Actions: View | Formatted Diff | Diff

Attachments on bug 194806: 362367 | 362368 | 362402 | 363946 | 363955 | 363959 | 363964 | 363992 | 363996 | 363999 | 364000 | 364002 | 364018 | 364294 | 364667 | 364749