Convert MessageLoopProxy to TaskRunner in FileUtilProxy

Convert MessageLoopProxy to TaskRunner in FileUtilProxy

Also adding unittests to FileUtilProxy.

BUG=123558, 77454
TEST=FileUtilProxyTest.*


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

[willchan no longer on Chromium, 2012-04-21 06:58:39]

http://codereview.chromium.org/10095028/diff/1/base/file_util_proxy_unittest.cc
File base/file_util_proxy_unittest.cc (right):

http://codereview.chromium.org/10095028/diff/1/base/file_util_proxy_unittest....
base/file_util_proxy_unittest.cc:8: #include "base/file_util_proxy.h"
This should go above <map> according to style guide. both foo.cc and foo_test.cc
should include foo.h first.

http://codereview.chromium.org/10095028/diff/1/base/file_util_proxy_unittest....
base/file_util_proxy_unittest.cc:269: int expected_bytes =
ARRAYSIZE_UNSAFE(expected_data);
You should be able to use arraysize(), not ARRAYSIZE_UNSAFE().

[kinuko, 2012-04-23 09:01:33]

http://codereview.chromium.org/10095028/diff/1/base/file_util_proxy_unittest.cc
File base/file_util_proxy_unittest.cc (right):

http://codereview.chromium.org/10095028/diff/1/base/file_util_proxy_unittest....
base/file_util_proxy_unittest.cc:8: #include "base/file_util_proxy.h"
On 2012/04/21 06:58:39, willchan wrote:
> This should go above <map> according to style guide. both foo.cc and
foo_test.cc
> should include foo.h first.

Done.

http://codereview.chromium.org/10095028/diff/1/base/file_util_proxy_unittest....
base/file_util_proxy_unittest.cc:269: int expected_bytes =
ARRAYSIZE_UNSAFE(expected_data);
On 2012/04/21 06:58:39, willchan wrote:
> You should be able to use arraysize(), not ARRAYSIZE_UNSAFE().

Done.

[willchan no longer on Chromium, 2012-04-23 14:22:55]

lgtm

[commit-bot: I haz the power, 2012-04-23 16:33:38]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/kinuko@chromium.org/10095028/9001

[commit-bot: I haz the power, 2012-04-23 18:41:19]

Change committed as 133484