IndexedDB: Add --level-db flag, and pipe through to WebIDBFactory::open.

IndexedDB: Add --indexeddb-use-leveldb flag, and pipe through to WebIDBFactory::open.

BUG=64078
TEST=none

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

[hans, 2011-03-08 18:28:51]

This depends on https://bugs.webkit.org/show_bug.cgi?id=55948

Please review.

[jorlow, 2011-03-08 18:45:54]

http://codereview.chromium.org/6627081/diff/1/chrome/common/chrome_switches.cc
File chrome/common/chrome_switches.cc (right):

http://codereview.chromium.org/6627081/diff/1/chrome/common/chrome_switches.c...
chrome/common/chrome_switches.cc:268: const char kLevelDBIndexedDatabase[]      
 = "level-db";
Let's make this a bit more descriptive.

--indexeddb-use-leveldb ?

http://codereview.chromium.org/6627081/diff/1/chrome/common/chrome_switches.h
File chrome/common/chrome_switches.h (right):

http://codereview.chromium.org/6627081/diff/1/chrome/common/chrome_switches.h...
chrome/common/chrome_switches.h:92: extern const char kLevelDBIndexedDatabase[];
I believe it should go in content switches now.  And alpha order.

[hans, 2011-03-09 12:22:09]

Thanks for the review. Will run try jobs when webkit rolls.

http://codereview.chromium.org/6627081/diff/1/chrome/common/chrome_switches.cc
File chrome/common/chrome_switches.cc (right):

http://codereview.chromium.org/6627081/diff/1/chrome/common/chrome_switches.c...
chrome/common/chrome_switches.cc:268: const char kLevelDBIndexedDatabase[]      
 = "level-db";
On 2011/03/08 18:45:55, jorlow wrote:
> Let's make this a bit more descriptive.
> 
> --indexeddb-use-leveldb ?

Done.

http://codereview.chromium.org/6627081/diff/1/chrome/common/chrome_switches.h
File chrome/common/chrome_switches.h (right):

http://codereview.chromium.org/6627081/diff/1/chrome/common/chrome_switches.h...
chrome/common/chrome_switches.h:92: extern const char kLevelDBIndexedDatabase[];
On 2011/03/08 18:45:55, jorlow wrote:
> I believe it should go in content switches now.  And alpha order.

Done.

[hans, 2011-03-10 11:41:47]

On 2011/03/09 12:22:09, hans wrote:
> Thanks for the review. Will run try jobs when webkit rolls.
> 
> http://codereview.chromium.org/6627081/diff/1/chrome/common/chrome_switches.cc
> File chrome/common/chrome_switches.cc (right):
> 
>
http://codereview.chromium.org/6627081/diff/1/chrome/common/chrome_switches.c...
> chrome/common/chrome_switches.cc:268: const char kLevelDBIndexedDatabase[]    
 
>  = "level-db";
> On 2011/03/08 18:45:55, jorlow wrote:
> > Let's make this a bit more descriptive.
> > 
> > --indexeddb-use-leveldb ?
> 
> Done.
> 
> http://codereview.chromium.org/6627081/diff/1/chrome/common/chrome_switches.h
> File chrome/common/chrome_switches.h (right):
> 
>
http://codereview.chromium.org/6627081/diff/1/chrome/common/chrome_switches.h...
> chrome/common/chrome_switches.h:92: extern const char
kLevelDBIndexedDatabase[];
> On 2011/03/08 18:45:55, jorlow wrote:
> > I believe it should go in content switches now.  And alpha order.
> 
> Done.

Ping?

[andreip3000, 2011-03-10 11:47:02]

LGTM