Quota: Add proxy for QuotaManager::GetUsageAndQuota

Quota: Add proxy for QuotaManager::GetUsageAndQuota

This introduces proxy implementation for QuotaManager::GetUsageAndQuota in
QuotaManagerProxy.

The proxy will be used from QuotaBackend living on the File thread so that
the backend can make sure of available quota space before reserving quota.


BUG=303443
TEST=n/a
NOTRY=true

Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=233254

[nhiroki, 2013-11-06 06:02:16]

ptal

https://codereview.chromium.org/59753008/diff/30001/webkit/browser/quota/mock...
File webkit/browser/quota/mock_quota_manager.h (right):

https://codereview.chromium.org/59753008/diff/30001/webkit/browser/quota/mock...
webkit/browser/quota/mock_quota_manager.h:165: // id SetUsageCacheEnabled()
Will be removed.

[kinuko, 2013-11-06 06:07:26]

https://codereview.chromium.org/59753008/diff/30001/webkit/browser/appcache/a...
File webkit/browser/appcache/appcache_storage_impl_unittest.cc (right):

https://codereview.chromium.org/59753008/diff/30001/webkit/browser/appcache/a...
webkit/browser/appcache/appcache_storage_impl_unittest.cc:329:
base::SingleThreadTaskRunner* original_task_runner,
Is this going to be always single-threaded? Then maybe we can just use
base::MessageLoopProxy::current() rather than passing it. Or did you mean
SequencedTaskRunner?

[tzik, 2013-11-06 06:10:43]

Looks good

https://codereview.chromium.org/59753008/diff/30001/webkit/browser/appcache/a...
File webkit/browser/appcache/appcache_storage_impl_unittest.cc (right):

https://codereview.chromium.org/59753008/diff/30001/webkit/browser/appcache/a...
webkit/browser/appcache/appcache_storage_impl_unittest.cc:329:
base::SingleThreadTaskRunner* original_task_runner,
Can we use SequencedTaskRunner instead of SingleThreadTaskRunner?

[nhiroki, 2013-11-06 06:20:16]

Thanks! Update.

https://codereview.chromium.org/59753008/diff/30001/webkit/browser/appcache/a...
File webkit/browser/appcache/appcache_storage_impl_unittest.cc (right):

https://codereview.chromium.org/59753008/diff/30001/webkit/browser/appcache/a...
webkit/browser/appcache/appcache_storage_impl_unittest.cc:329:
base::SingleThreadTaskRunner* original_task_runner,
On 2013/11/06 06:10:43, tzik wrote:
> Can we use SequencedTaskRunner instead of SingleThreadTaskRunner?

Oops... I meant SequencedTaskRunner. Thanks! Done.

[tzik, 2013-11-06 06:25:41]

lgtm

[kinuko, 2013-11-06 06:40:50]

lgtm2
2013/11/06 15:25 <tzik@chromium.org>:

> lgtm
>
>
>
> https://codereview.chromium.org/59753008/
>

To unsubscribe from this group and stop receiving emails from it, send an email
to chromium-reviews+unsubscribe@chromium.org.

[commit-bot: I haz the power, 2013-11-06 06:47:02]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/nhiroki@chromium.org/59753008/140001

[commit-bot: I haz the power, 2013-11-06 10:23:23]

Retried try job too often on linux_aura for step(s) browser_tests
http://build.chromium.org/p/tryserver.chromium/buildstatus?builder=linux_aura...

[commit-bot: I haz the power, 2013-11-06 10:31:13]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/nhiroki@chromium.org/59753008/140001

[commit-bot: I haz the power, 2013-11-06 10:33:06]

Change committed as 233254