diff options
author | kinuko@chromium.org <kinuko@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2013-04-18 01:46:34 +0000 |
---|---|---|
committer | kinuko@chromium.org <kinuko@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2013-04-18 01:46:34 +0000 |
commit | 420fb56759f0ac2d0291ea47397ffb7eb87ade04 (patch) | |
tree | 705c9e6d3b8be986e6e6724605f5573ad1588ddb /webkit/fileapi/file_system_util.cc | |
parent | e3dcf9e71a7c5867a0c3cbbc9c7420d0821d3206 (diff) | |
download | chromium_src-420fb56759f0ac2d0291ea47397ffb7eb87ade04.zip chromium_src-420fb56759f0ac2d0291ea47397ffb7eb87ade04.tar.gz chromium_src-420fb56759f0ac2d0291ea47397ffb7eb87ade04.tar.bz2 |
Make MountPointProvider pluggable from outside webkit/fileapi
Currently we have some layering leakages under webkit/ because
we have several FS modules (e.g. media modules) that
should not be known by WebKit/FileAPI (or content) layer.
Also some module implementors are forced to add extra glue interfaces
just to across layering boundaries from webkit.
This patch changes to:
- Add CanHandleType method to each MountPointProvider
- Make FileSystemContext's ctor take additional MountPointProviders and
MountPoints, so that FS module implementors can implement their own
provider outside webkit/ and plug it into FileAPI layer.
This is the 1st cut of a couple of planned changes.
Brief change plan (internal): https://docs.google.com/a/google.com/document/d/1XEtX0OO_RIA_c0KuKvd9yqK5l3XtsTulwXs3c-mAQ1o/view
BUG=175936
TEST=content_unittests:.*File.*
TEST=content_unittests:.*MountPointProvider.*
Review URL: https://codereview.chromium.org/14096022
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@194747 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'webkit/fileapi/file_system_util.cc')
-rw-r--r-- | webkit/fileapi/file_system_util.cc | 4 |
1 files changed, 4 insertions, 0 deletions
diff --git a/webkit/fileapi/file_system_util.cc b/webkit/fileapi/file_system_util.cc index 468abcf..3918ce0 100644 --- a/webkit/fileapi/file_system_util.cc +++ b/webkit/fileapi/file_system_util.cc @@ -241,6 +241,10 @@ std::string GetFileSystemTypeString(FileSystemType type) { return "Syncable"; case kFileSystemTypeNativeForPlatformApp: return "NativeForPlatformApp"; + case kFileSystemInternalTypeEnumStart: + case kFileSystemInternalTypeEnumEnd: + NOTREACHED(); + // Fall through. case kFileSystemTypeUnknown: return "Unknown"; } |