IDBFactoryBackend -> IndexedDBClient, remove indexeddb/chromium

IDBFactoryBackend -> IndexedDBClient, remove indexeddb/chromium

Tidy up some naming and layering split from the WebKit days.

Committed: https://src.chromium.org/viewvc/blink?view=rev&revision=171425

[jsbell, 2014-04-04 23:46:46]

Inspired by jam@'s crrev.com/225303007 - let's rename/collapse these to match
other client classes.

dglazkov@, jam@ - please take a look?

(jam@'s change can land first)

[jam, 2014-04-04 23:52:38]

just to be clear: i'm not an owner/reviewer in blink

[jsbell, 2014-04-05 00:11:04]

On 2014/04/04 23:52:38, jam wrote:
> just to be clear: i'm not an owner/reviewer in blink

Yep! But since you're touching related blink files around permission clients, I
welcome any vetos.

(Hrm, lingering include reference. New patch coming up...)

[jsbell, 2014-04-07 17:47:23]

Compile fail on android was unrelated.

dglazkov@, abarth@ - PTAL?

[jam, 2014-04-07 18:08:36]

On 2014/04/05 00:11:04, jsbell wrote:
> On 2014/04/04 23:52:38, jam wrote:
> > just to be clear: i'm not an owner/reviewer in blink
> 
> Yep! But since you're touching related blink files around permission clients,
I
> welcome any vetos.

ah, no issues seen, and I wouldn't even be qualified to comment on this code :) 
in blink land, i mostly do just trivial changes

> 
> (Hrm, lingering include reference. New patch coming up...)

[jsbell, 2014-04-09 23:20:30]

huh, not entirely sure why I had added IndexedDB.cpp. Moved it to
IndexedDBClient.cpp instead, which makes more sense and simplifies the patch. 

abarth@, dglazkov@ - please take a look?

[abarth-chromium, 2014-04-10 00:33:44]

lgtm

https://codereview.chromium.org/225023020/diff/60001/Source/modules/indexeddb...
File Source/modules/indexeddb/IDBFactory.h (right):

https://codereview.chromium.org/225023020/diff/60001/Source/modules/indexeddb...
Source/modules/indexeddb/IDBFactory.h:48: static PassRefPtr<IDBFactory>
create(IndexedDBClient* factory)
s/factory/client/ ?

[jsbell, 2014-04-11 23:32:45]

On 2014/04/10 00:33:44, abarth wrote:
> lgtm
> 
>
https://codereview.chromium.org/225023020/diff/60001/Source/modules/indexeddb...
> File Source/modules/indexeddb/IDBFactory.h (right):
> 
>
https://codereview.chromium.org/225023020/diff/60001/Source/modules/indexeddb...
> Source/modules/indexeddb/IDBFactory.h:48: static PassRefPtr<IDBFactory>
> create(IndexedDBClient* factory)
> s/factory/client/ ?

Done. Also updated m_factoryBackend -> m_client in
WorkerGlobalScopeIndexedDatabase.{cpp,h}

[jsbell, 2014-04-11 23:32:50]

The CQ bit was checked by jsbell@chromium.org

[commit-bot: I haz the power, 2014-04-11 23:32:53]

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

[commit-bot: I haz the power, 2014-04-12 00:24:51]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2014-04-12 00:24:52]

Try jobs failed on following builders:
  tryserver.blink on win_blink_rel

[abarth-chromium, 2014-04-12 18:56:33]

The CQ bit was checked by abarth@chromium.org

[commit-bot: I haz the power, 2014-04-12 18:56:44]

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

[commit-bot: I haz the power, 2014-04-12 19:32:54]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2014-04-12 19:32:54]

Try jobs failed on following builders:
  tryserver.blink on win_blink_rel

[jsbell, 2014-04-14 06:09:00]

The CQ bit was checked by jsbell@chromium.org

[commit-bot: I haz the power, 2014-04-14 06:09:08]

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

[commit-bot: I haz the power, 2014-04-14 06:38:17]

Change committed as 171425