Allow IndexedDB to send messages from any thread in the renderer process.

Allow IndexedDB to send messages from any thread in the renderer process, not just the main thread.

BUG=64054
TEST=


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

[dgrogan, 2011-11-30 03:52:18]

First and small part of chromium side IDB-on-workers changes.

[dgrogan, 2011-11-30 05:54:10]

+jam, for content/renderer/OWNERS

I think this change is straightforward enough to add jam before hearing from
anyone else.

If I click commit after getting green trybot runs, will the commit queue run all
the tests again?

[hans, 2011-11-30 13:49:12]

LGTM

On 2011/11/30 05:54:10, dgrogan wrote:
> If I click commit after getting green trybot runs, will the commit queue run
all
> the tests again?

Yes, as far as I know it will run them again.

[jam, 2011-11-30 18:32:10]

if this code is meant to be used in the renderer and worker processes, then it
needs to move to content/common

http://codereview.chromium.org/8745003/diff/1/content/renderer/indexed_db_dis...
File content/renderer/indexed_db_dispatcher.cc (right):

http://codereview.chromium.org/8745003/diff/1/content/renderer/indexed_db_dis...
content/renderer/indexed_db_dispatcher.cc:335: idb_object_store_id, response_id,
nit: i believe the tabbing here should be 4 spaces. also above

[dgrogan, 2011-11-30 19:19:15]

On 2011/11/30 18:32:10, John Abd-El-Malek wrote:
> if this code is meant to be used in the renderer and worker processes, then it
> needs to move to content/common

My comment was misleading - _eventually_ this will be used in the worker
process, but for the time being it will only be used from the renderer process,
but on dedicated worker threads in addition to the main thread.  When IDB is
made to be available from shared workers a bunch of stuff will be moved to
common, these files included.

>
http://codereview.chromium.org/8745003/diff/1/content/renderer/indexed_db_dis...
> File content/renderer/indexed_db_dispatcher.cc (right):
> 
>
http://codereview.chromium.org/8745003/diff/1/content/renderer/indexed_db_dis...
> content/renderer/indexed_db_dispatcher.cc:335: idb_object_store_id,
response_id,
> nit: i believe the tabbing here should be 4 spaces. also above

Fixed.

[jam, 2011-11-30 20:32:58]

On 2011/11/30 19:19:15, dgrogan wrote:
> On 2011/11/30 18:32:10, John Abd-El-Malek wrote:
> > if this code is meant to be used in the renderer and worker processes, then
it
> > needs to move to content/common
> 
> My comment was misleading - _eventually_ this will be used in the worker
> process, but for the time being it will only be used from the renderer
process,
> but on dedicated worker threads in addition to the main thread.  When IDB is
> made to be available from shared workers a bunch of stuff will be moved to
> common, these files included.

got it, lgtm
> 
> >
>
http://codereview.chromium.org/8745003/diff/1/content/renderer/indexed_db_dis...
> > File content/renderer/indexed_db_dispatcher.cc (right):
> > 
> >
>
http://codereview.chromium.org/8745003/diff/1/content/renderer/indexed_db_dis...
> > content/renderer/indexed_db_dispatcher.cc:335: idb_object_store_id,
> response_id,
> > nit: i believe the tabbing here should be 4 spaces. also above
> 
> Fixed.

[commit-bot: I haz the power, 2011-11-30 20:34:12]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/dgrogan@chromium.org/8745003/6001

[commit-bot: I haz the power, 2011-12-01 00:12:09]

Change committed as 112348