Refactors FileSystemQuotaClient using base::Callback.

Refactors FileSystemQuotaClient using base::Callback.

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

[nhiroki, 2012-08-06 18:47:08]

Hi kinuko@

Could you review this? Thanks.

[kinuko, 2012-08-06 21:06:27]

lgtm

https://chromiumcodereview.appspot.com/10828177/diff/1/webkit/fileapi/file_sy...
File webkit/fileapi/file_system_quota_client.cc (right):

https://chromiumcodereview.appspot.com/10828177/diff/1/webkit/fileapi/file_sy...
webkit/fileapi/file_system_quota_client.cc:190:
make_scoped_refptr(file_system_context_.get()),
nit: maybe you can just pass file_system_context_ (as refptr) here and elsewhere

[nhiroki, 2012-08-06 21:30:34]

Thank you for reviewing!

https://chromiumcodereview.appspot.com/10828177/diff/1/webkit/fileapi/file_sy...
File webkit/fileapi/file_system_quota_client.cc (right):

https://chromiumcodereview.appspot.com/10828177/diff/1/webkit/fileapi/file_sy...
webkit/fileapi/file_system_quota_client.cc:190:
make_scoped_refptr(file_system_context_.get()),
On 2012/08/06 21:06:27, kinuko wrote:
> nit: maybe you can just pass file_system_context_ (as refptr) here and
elsewhere

Done.

[commit-bot: I haz the power, 2012-08-07 01:17:36]

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

[commit-bot: I haz the power, 2012-08-07 03:59:36]

Failed to apply patch for webkit/fileapi/file_system_quota_client.cc:
While running patch -p1 --forward --force;
patching file webkit/fileapi/file_system_quota_client.cc
Hunk #1 FAILED at 7.
Hunk #2 succeeded at 21 (offset 1 line).
Hunk #3 succeeded at 105 (offset 1 line).
Hunk #4 succeeded at 128 (offset 1 line).
Hunk #5 succeeded at 154 (offset 1 line).
Hunk #6 succeeded at 180 (offset 1 line).
1 out of 6 hunks FAILED -- saving rejects to file
webkit/fileapi/file_system_quota_client.cc.rej

[commit-bot: I haz the power, 2012-08-07 05:39:32]

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

[commit-bot: I haz the power, 2012-08-07 06:49:25]

Change committed as 150290