Add plumbing for passing the indexed database quota via IDBFactory::open

Add plumbing for passing the indexed database quota via IDBFactory::open


BUG=
TEST=

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

[jorlow, 2010-10-12 02:08:04]

http://codereview.chromium.org/3729003/diff/1/2
File chrome/browser/in_process_webkit/indexed_db_dispatcher_host.cc (right):

http://codereview.chromium.org/3729003/diff/1/2#newcode53
chrome/browser/in_process_webkit/indexed_db_dispatcher_host.cc:53: static const
uint64 kDatabaseQuota = 5 * 1024 * 1024;
it's unsigned in webkit, right?  Might as well just make this unsigned?

http://codereview.chromium.org/3729003/diff/1/5
File chrome/renderer/indexed_db_dispatcher.cc (right):

http://codereview.chromium.org/3729003/diff/1/5#newcode106
chrome/renderer/indexed_db_dispatcher.cc:106: WebFrame* web_frame, unsigned long
long maximum_size) {
split these lines

http://codereview.chromium.org/3729003/diff/1/6
File chrome/renderer/indexed_db_dispatcher.h (right):

http://codereview.chromium.org/3729003/diff/1/6#newcode38
chrome/renderer/indexed_db_dispatcher.h:38: WebKit::WebFrame* web_frame,
unsigned long long maximum_size);
Split all these lines.

http://codereview.chromium.org/3729003/diff/1/8
File chrome/renderer/renderer_webidbfactory_impl.h (right):

http://codereview.chromium.org/3729003/diff/1/8#newcode32
chrome/renderer/renderer_webidbfactory_impl.h:32: unsigned long long
maximum_size);
include base/basictypes.h and use the same type as you use in the rest of
chromium

[andreip3000, 2010-10-12 02:33:55]

http://codereview.chromium.org/3729003/diff/1/2
File chrome/browser/in_process_webkit/indexed_db_dispatcher_host.cc (right):

http://codereview.chromium.org/3729003/diff/1/2#newcode53
chrome/browser/in_process_webkit/indexed_db_dispatcher_host.cc:53: static const
uint64 kDatabaseQuota = 5 * 1024 * 1024;
On 2010/10/12 02:08:04, jorlow wrote:
> it's unsigned in webkit, right?  Might as well just make this unsigned?

It's a Uint64, so it is unsigned :)

http://codereview.chromium.org/3729003/diff/1/5
File chrome/renderer/indexed_db_dispatcher.cc (right):

http://codereview.chromium.org/3729003/diff/1/5#newcode106
chrome/renderer/indexed_db_dispatcher.cc:106: WebFrame* web_frame, unsigned long
long maximum_size) {
On 2010/10/12 02:08:04, jorlow wrote:
> split these lines

Done.

http://codereview.chromium.org/3729003/diff/1/6
File chrome/renderer/indexed_db_dispatcher.h (right):

http://codereview.chromium.org/3729003/diff/1/6#newcode38
chrome/renderer/indexed_db_dispatcher.h:38: WebKit::WebFrame* web_frame,
unsigned long long maximum_size);
On 2010/10/12 02:08:04, jorlow wrote:
> Split all these lines.

Done.

http://codereview.chromium.org/3729003/diff/1/8
File chrome/renderer/renderer_webidbfactory_impl.h (right):

http://codereview.chromium.org/3729003/diff/1/8#newcode32
chrome/renderer/renderer_webidbfactory_impl.h:32: unsigned long long
maximum_size);
On 2010/10/12 02:08:04, jorlow wrote:
> include base/basictypes.h and use the same type as you use in the rest of
> chromium

Done.

[jorlow, 2010-10-12 02:42:45]

LGTM