Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(503)

Issue 242493002: [fsp] [recommit] Add an initial AsyncFileUtil. (Closed)

Created:
6 years, 8 months ago by mtomasz
Modified:
6 years, 8 months ago
Reviewers:
kinaba
CC:
chromium-reviews, nkostylev+watch_chromium.org, tzik, nhiroki, oshima+watch_chromium.org, kinuko+watch, stevenjb+watch_chromium.org, davemoore+watch_chromium.org
Visibility:
Public.

Description

[fsp] [recommit] Add an initial AsyncFileUtil. This patch adds the AsyncFileUtil implementation for the file system provider. It does not communicate with the file system provider service yet. Currently, for not supported operations SECURITY error is returned. For allowed, but not implemented operations, it is NOT_FOUND error. Unit tests are quite trivial at this stage, but they will grow once more operations are implemented. This patch has been previously committed and reverted due to test failures. See: https://codereview.chromium.org/236303017/ TEST=unit_test: FileSystemProviderAsyncFileUtilTest* BUG=248427 Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=264778

Patch Set 1 #

Messages

Total messages: 9 (0 generated)
mtomasz
@kinaba: This identical to https://codereview.chromium.org/236303017/, but required to recommit via CQ. PTAL. Thanks.
6 years, 8 months ago (2014-04-18 03:54:06 UTC) #1
kinaba
lgtm
6 years, 8 months ago (2014-04-18 03:54:44 UTC) #2
mtomasz
The CQ bit was checked by mtomasz@chromium.org
6 years, 8 months ago (2014-04-18 03:55:41 UTC) #3
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/mtomasz@chromium.org/242493002/1
6 years, 8 months ago (2014-04-18 03:55:55 UTC) #4
commit-bot: I haz the power
The CQ bit was unchecked by commit-bot@chromium.org
6 years, 8 months ago (2014-04-18 05:40:00 UTC) #5
commit-bot: I haz the power
Try jobs failed on following builders: tryserver.chromium on win_chromium_rel
6 years, 8 months ago (2014-04-18 05:40:01 UTC) #6
mtomasz
The CQ bit was checked by mtomasz@chromium.org
6 years, 8 months ago (2014-04-18 05:41:13 UTC) #7
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/mtomasz@chromium.org/242493002/1
6 years, 8 months ago (2014-04-18 05:41:32 UTC) #8
commit-bot: I haz the power
6 years, 8 months ago (2014-04-18 12:29:47 UTC) #9
Message was sent while issue was closed.
Change committed as 264778

Powered by Google App Engine
This is Rietveld 408576698