IndexedDB: Eliminate WebIDBCursor wrapper

IndexedDB: Eliminate WebIDBCursor wrapper

Cruft from the Blink->Chromium backend move.

BUG=234278
TBR=jam@chromium.org
R=alecflett@chromium.org

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

[jsbell, 2013-06-27 22:24:28]

I'm not sure how I feel about IDMap<scoped_refptr<IndexedDBCursor>,
IDMapOwnPointer> - it does the right thing, but perhaps I should wrap
IDMap<scoped_refptr<T>, IDMapOwnPointer> in its own type within
indexed_db_dispatcher.cc that eliminates the icky double indirection to extract
the pointer.

Thoughts?

[jsbell, 2013-06-27 23:24:24]

On 2013/06/27 22:24:28, jsbell wrote:
> I'm not sure how I feel about IDMap<scoped_refptr<IndexedDBCursor>,
> IDMapOwnPointer> - it does the right thing, but perhaps I should wrap
> IDMap<scoped_refptr<T>, IDMapOwnPointer> in its own type within
> indexed_db_dispatcher.cc that eliminates the icky double indirection to
extract
> the pointer.
> 
> Thoughts?

Since I'll need it for eliminating WebIDBDatabaseImpl I went ahead and added
RefIDMap.

[jam, 2013-06-28 16:16:29]

lgtm

[alecflett, 2013-06-28 23:59:08]

lgtm as well

You could move RefIDMap as well, but the lgtm stands without that

[alecflett, 2013-06-28 23:59:16]

https://chromiumcodereview.appspot.com/18145004/diff/5001/content/browser/ind...
File content/browser/indexed_db/indexed_db_dispatcher_host.h (right):

https://chromiumcodereview.appspot.com/18145004/diff/5001/content/browser/ind...
content/browser/indexed_db/indexed_db_dispatcher_host.h:95: // IDMap for
RefCounted types
seems like this is worth moving along side the existing IDMap

[commit-bot: I haz the power, 2013-06-29 00:03:13]

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

[commit-bot: I haz the power, 2013-06-29 20:46:00]

Change committed as 209335