Remove IDBCursorBackendInterface::deleteFunction (part 2/3)

Remove IDBCursorBackendInterface::deleteFunction (part 2/3)

Remove IPC plumbing (and new backend impl) now that the front-end
of IDBCursor just relies on IDBDatabaseBackendInterface::deleteRange()

BUG=241526
R=alecflett@chromium.org

Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=203770

[jsbell, 2013-06-03 17:52:27]

alecflett@ - please take a look?

I ran clang-format over the touched files so there is some unrelated churn. I
can undo that if desired, but otherwise this patch is all deletions.

https://codereview.chromium.org/16337010/diff/1/content/browser/indexed_db/in...
File content/browser/indexed_db/indexed_db_cursor_impl.h (left):

https://codereview.chromium.org/16337010/diff/1/content/browser/indexed_db/in...
content/browser/indexed_db/indexed_db_cursor_impl.h:29: int64 object_store_id) {
Surprisingly, this is now unused - the backing store cursor has the object store
ID, so the cursor itself no longer needs it.

https://codereview.chromium.org/16337010/diff/1/content/common_child/indexed_...
File content/common_child/indexed_db/indexed_db_dispatcher.cc (left):

https://codereview.chromium.org/16337010/diff/1/content/common_child/indexed_...
content/common_child/indexed_db/indexed_db_dispatcher.cc:229: void
IndexedDBDispatcher::RequestIDBCursorDelete(
This is the only real change in the file.

https://codereview.chromium.org/16337010/diff/1/content/common_child/indexed_...
File content/common_child/indexed_db/indexed_db_dispatcher.cc (right):

https://codereview.chromium.org/16337010/diff/1/content/common_child/indexed_...
content/common_child/indexed_db/indexed_db_dispatcher.cc:80: void
IndexedDBDispatcher::OnWorkerRunLoopStopped() { delete this; }
more clang-format...

https://codereview.chromium.org/16337010/diff/1/content/common_child/indexed_...
File content/common_child/indexed_db/indexed_db_dispatcher.h (left):

https://codereview.chromium.org/16337010/diff/1/content/common_child/indexed_...
content/common_child/indexed_db/indexed_db_dispatcher.h:98: void
RequestIDBCursorDelete(
This is the only real change in the file.

https://codereview.chromium.org/16337010/diff/1/content/common_child/indexed_...
File content/common_child/indexed_db/indexed_db_dispatcher.h (right):

https://codereview.chromium.org/16337010/diff/1/content/common_child/indexed_...
content/common_child/indexed_db/indexed_db_dispatcher.h:63: void
RequestIDBFactoryGetDatabaseNames(WebKit::WebIDBCallbacks* callbacks,
This (and subsequent diffs) are c/o clang-format.

[alecflett, 2013-06-03 18:39:30]

lgtm

[commit-bot: I haz the power, 2013-06-03 18:42:58]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/jsbell@chromium.org/16337010/1

[commit-bot: I haz the power, 2013-06-03 18:48:06]

Retried try job too often on chromium_presubmit for step(s) presubmit
http://build.chromium.org/p/tryserver.chromium/buildstatus?builder=chromium_p...

[jsbell, 2013-06-03 18:55:38]

tsepez@ - can you review the IPC change? (deleting message)

[Tom Sepez, 2013-06-03 19:02:32]

On 2013/06/03 18:55:38, jsbell wrote:
> tsepez@ - can you review the IPC change? (deleting message)

Deleting message LGTM.

[commit-bot: I haz the power, 2013-06-03 19:27:50]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/jsbell@chromium.org/16337010/1

[commit-bot: I haz the power, 2013-06-03 21:08:06]

Change committed as 203770