New issue
Advanced search Search tips

Issue 805597 link

Starred by 1 user

Issue metadata

Status: Fixed
Owner:
Closed: Jan 2018
Components:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 3
Type: Task



Sign in to add a comment

[mojo-blobs] Remove blob acking from IndexedDB

Project Member Reported by mek@chromium.org, Jan 24 2018

Issue description

With mojo blobs blob acking should no longer be needed.
 
Project Member

Comment 1 by bugdroid1@chromium.org, Jan 27 2018

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/c1a69b495a9519506aa07abd4fbef4957799c0ba

commit c1a69b495a9519506aa07abd4fbef4957799c0ba
Author: Marijn Kruisselbrink <mek@chromium.org>
Date: Sat Jan 27 03:51:18 2018

IndexedDB: Remove Blob-acking.

With mojo blobs a blob handle is passed all the way across from the
browser process to blink so there should be no need for blob acking
anymore.

Bug:  805597 
Change-Id: I11a59f4af80df4c13c73ee5d7761a07a1a38129a
Reviewed-on: https://chromium-review.googlesource.com/883980
Reviewed-by: Kentaro Hara <haraken@chromium.org>
Reviewed-by: Daniel Murphy <dmurph@chromium.org>
Reviewed-by: Tom Sepez <tsepez@chromium.org>
Reviewed-by: Victor Costan <pwnall@chromium.org>
Commit-Queue: Marijn Kruisselbrink <mek@chromium.org>
Cr-Commit-Position: refs/heads/master@{#532151}
[modify] https://crrev.com/c1a69b495a9519506aa07abd4fbef4957799c0ba/content/browser/indexed_db/cursor_impl.cc
[modify] https://crrev.com/c1a69b495a9519506aa07abd4fbef4957799c0ba/content/browser/indexed_db/cursor_impl.h
[modify] https://crrev.com/c1a69b495a9519506aa07abd4fbef4957799c0ba/content/browser/indexed_db/database_impl.cc
[modify] https://crrev.com/c1a69b495a9519506aa07abd4fbef4957799c0ba/content/browser/indexed_db/database_impl.h
[modify] https://crrev.com/c1a69b495a9519506aa07abd4fbef4957799c0ba/content/browser/indexed_db/indexed_db_callbacks.cc
[modify] https://crrev.com/c1a69b495a9519506aa07abd4fbef4957799c0ba/content/browser/indexed_db/indexed_db_dispatcher_host.cc
[modify] https://crrev.com/c1a69b495a9519506aa07abd4fbef4957799c0ba/content/browser/indexed_db/indexed_db_dispatcher_host.h
[modify] https://crrev.com/c1a69b495a9519506aa07abd4fbef4957799c0ba/content/browser/indexed_db/indexed_db_dispatcher_host_unittest.cc
[modify] https://crrev.com/c1a69b495a9519506aa07abd4fbef4957799c0ba/content/common/indexed_db/indexed_db.mojom
[modify] https://crrev.com/c1a69b495a9519506aa07abd4fbef4957799c0ba/content/renderer/indexed_db/webidbcursor_impl.cc
[modify] https://crrev.com/c1a69b495a9519506aa07abd4fbef4957799c0ba/content/renderer/indexed_db/webidbcursor_impl_unittest.cc
[modify] https://crrev.com/c1a69b495a9519506aa07abd4fbef4957799c0ba/content/renderer/indexed_db/webidbdatabase_impl.cc
[modify] https://crrev.com/c1a69b495a9519506aa07abd4fbef4957799c0ba/content/renderer/indexed_db/webidbdatabase_impl.h
[modify] https://crrev.com/c1a69b495a9519506aa07abd4fbef4957799c0ba/third_party/WebKit/Source/modules/exported/WebIDBValue.cpp
[modify] https://crrev.com/c1a69b495a9519506aa07abd4fbef4957799c0ba/third_party/WebKit/Source/modules/indexeddb/IDBRequest.cpp
[modify] https://crrev.com/c1a69b495a9519506aa07abd4fbef4957799c0ba/third_party/WebKit/Source/modules/indexeddb/IDBRequest.h
[modify] https://crrev.com/c1a69b495a9519506aa07abd4fbef4957799c0ba/third_party/WebKit/Source/modules/indexeddb/IDBRequestTest.cpp
[modify] https://crrev.com/c1a69b495a9519506aa07abd4fbef4957799c0ba/third_party/WebKit/Source/modules/indexeddb/IDBTransactionTest.cpp
[modify] https://crrev.com/c1a69b495a9519506aa07abd4fbef4957799c0ba/third_party/WebKit/Source/modules/indexeddb/IDBValue.cpp
[modify] https://crrev.com/c1a69b495a9519506aa07abd4fbef4957799c0ba/third_party/WebKit/Source/modules/indexeddb/IDBValue.h
[modify] https://crrev.com/c1a69b495a9519506aa07abd4fbef4957799c0ba/third_party/WebKit/Source/modules/indexeddb/MockWebIDBDatabase.h
[modify] https://crrev.com/c1a69b495a9519506aa07abd4fbef4957799c0ba/third_party/WebKit/public/platform/modules/indexeddb/WebIDBDatabase.h
[modify] https://crrev.com/c1a69b495a9519506aa07abd4fbef4957799c0ba/third_party/WebKit/public/platform/modules/indexeddb/WebIDBValue.h

Comment 2 by mek@chromium.org, Jan 27 2018

Status: Fixed (was: Started)

Sign in to add a comment