[IndexedDB] Hold referenced to IDB::put blobs in the IDBRequest

third_party/WebKit/Source/modules/indexeddb/IDBRequest.cpp

Index: third_party/WebKit/Source/modules/indexeddb/IDBRequest.cpp
diff --git a/third_party/WebKit/Source/modules/indexeddb/IDBRequest.cpp b/third_party/WebKit/Source/modules/indexeddb/IDBRequest.cpp
index bea4fbb3cd42c11d279decb0bb7cab603383e34f..be09a519d66b214dc7de65490d46ac4137e9e9d3 100644
--- a/third_party/WebKit/Source/modules/indexeddb/IDBRequest.cpp
+++ b/third_party/WebKit/Source/modules/indexeddb/IDBRequest.cpp
@@ -241,6 +241,7 @@ bool IDBRequest::ShouldEnqueueEvent() const {
 
 void IDBRequest::OnError(DOMException* error) {
   IDB_TRACE("IDBRequest::onError()");
+  ClearTransitBlobHandles();
   if (!ShouldEnqueueEvent())
     return;
 
@@ -289,6 +290,7 @@ void IDBRequest::OnSuccess(std::unique_ptr<WebIDBCursor> backend,
 
 void IDBRequest::OnSuccess(IDBKey* idb_key) {
   IDB_TRACE("IDBRequest::onSuccess(IDBKey)");
+  ClearTransitBlobHandles();

[michaeln, 2017/04/25 23:37:00]

There are many OnSuccess variants, no doubt this is the one that get's called
for Put(), maybe to ensure and make clear this method isn't needed for the other
variants, add a DCHECK(!transit_blobs) to OnSucessInternal.

[dmurph, 2017/04/27 22:04:12]

Done.

   if (!ShouldEnqueueEvent())
     return;