IndexedDB API cleanup.

IndexedDB API cleanup.

Fix Chromium impls and calls to WebIDB* interfaces to match
current WebKit implementation.  No change in behavior, just
fewer wrapped function calls.

R=dgrogan@chromium.org,darin@chromium.org

Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=97687

[dgrogan, 2011-08-19 21:30:52]

LGTM

http://codereview.chromium.org/7692006/diff/1/content/browser/in_process_webk...
File content/browser/in_process_webkit/indexed_db_callbacks.h (right):

http://codereview.chromium.org/7692006/diff/1/content/browser/in_process_webk...
content/browser/in_process_webkit/indexed_db_callbacks.h:23: template <class
Type> struct WebIDBToMsgHelper { };
Can you add a TODO to (or just) delete WebIDBToMsgHelper?  It was already lame
with just one remaining specialization, but I think now it's just dead code.

[adamk, 2011-08-19 22:24:24]

http://codereview.chromium.org/7692006/diff/1/content/browser/in_process_webk...
File content/browser/in_process_webkit/indexed_db_callbacks.h (right):

http://codereview.chromium.org/7692006/diff/1/content/browser/in_process_webk...
content/browser/in_process_webkit/indexed_db_callbacks.h:23: template <class
Type> struct WebIDBToMsgHelper { };
On 2011/08/19 21:30:53, dgrogan wrote:
> Can you add a TODO to (or just) delete WebIDBToMsgHelper?  It was already lame
> with just one remaining specialization, but I think now it's just dead code.

I've re-templatized two of these specializations, so I'm now using this again
(saves about 40 lines of code).

[dgrogan, 2011-08-19 22:48:18]

LGTM

Thanks for the cleanup.

[adamk, 2011-08-20 01:05:35]

Thanks, David.  Still need OWNER approval for content/renderer/, adding jam &
brettw.

[brettw, 2011-08-21 15:57:54]

LGTM rubberstamp, please only add one person for owners review rather than
spamming everybody.