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

Issue 14493007: Add Transient FileSystem which goes away after its snapsot file is gone (Closed)

Created:
7 years, 8 months ago by kinuko
Modified:
7 years, 8 months ago
Reviewers:
tzik
CC:
chromium-reviews, joi+watch-content_chromium.org, darin-cc_chromium.org, kinuko+watch, jam, tzik+watch_chromium.org
Visibility:
Public.

Description

Add Transient FileSystem which goes away after its snapsot file is gone - Adding a new filesystem type, kFileSystemTypeForTransientFile, which is supposed to only contain a single temporary file and must be go away after the last reference of the snapshot file for the temporary file is dropped. I'm planning to use this filesystem for creating Blobs for manual conflict handling API. BUG=177159 TEST=TransientFileTest.TransientFile Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=196670

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+235 lines, -2 lines) Patch
M content/content_tests.gypi View 1 chunk +2 lines, -1 line 0 comments Download
M webkit/fileapi/file_system_types.h View 1 chunk +7 lines, -0 lines 0 comments Download
M webkit/fileapi/file_system_util.cc View 1 chunk +2 lines, -0 lines 0 comments Download
M webkit/fileapi/isolated_mount_point_provider.h View 1 chunk +1 line, -0 lines 0 comments Download
M webkit/fileapi/isolated_mount_point_provider.cc View 4 chunks +8 lines, -1 line 0 comments Download
A webkit/fileapi/transient_file_util.h View 1 chunk +36 lines, -0 lines 0 comments Download
A webkit/fileapi/transient_file_util.cc View 1 chunk +55 lines, -0 lines 0 comments Download
A webkit/fileapi/transient_file_util_unittest.cc View 1 chunk +122 lines, -0 lines 0 comments Download
M webkit/fileapi/webkit_fileapi.gypi View 1 chunk +2 lines, -0 lines 0 comments Download

Messages

Total messages: 3 (0 generated)
kinuko
Can you take a look?
7 years, 8 months ago (2013-04-25 06:29:27 UTC) #1
tzik
lgtm
7 years, 8 months ago (2013-04-25 08:06:40 UTC) #2
kinuko
7 years, 8 months ago (2013-04-26 08:11:03 UTC) #3
Message was sent while issue was closed.
Committed patchset #1 manually as r196670.

Powered by Google App Engine
This is Rietveld 408576698