kFileSystemTypeIsolated should be only used in the URL exposed to renderer

webkit/fileapi/file_system_util.h

 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #ifndef WEBKIT_FILEAPI_FILE_SYSTEM_UTIL_H_
 #define WEBKIT_FILEAPI_FILE_SYSTEM_UTIL_H_
 
 #include <string>
 #include <vector>
 
@@ skipping 48 matching lines @@
       std::vector<FilePath::StringType>* components);
 };
 
 // Returns the root URI of the filesystem that can be specified by a pair of
 // |origin_url| and |type|.  The returned URI can be used as a root path
 // of the filesystem (e.g. <returned_URI> + "/relative/path" will compose
 // a path pointing to the entry "/relative/path" in the filesystem).
 //
 // For Isolated filesystem this returns the 'common' root part, e.g.
 // returns URL without the filesystem ID.
+//
+// |type| needs to be public type as the returned URI is given to the renderer.
 FILEAPI_EXPORT GURL GetFileSystemRootURI(const GURL& origin_url,
                                          FileSystemType type);
 
 // Returns the name for the filesystem that is specified by a pair of
 // |origin_url| and |type|.
 // (The name itself is neither really significant nor a formal identifier
 // but can be read as the .name field of the returned FileSystem object
 // as a user-friendly name in the javascript layer).
 //
+// |type| needs to be public type as the returned name is given to the renderer.
+//
 // Example:
 //   The name for a TEMPORARY filesystem of "http://www.example.com:80/"
 //   should look like: "http_www.example.host_80:temporary"
 FILEAPI_EXPORT std::string GetFileSystemName(const GURL& origin_url,
                                              FileSystemType type);
 
 // Converts FileSystemType |type| to/from the StorageType |storage_type| that
 // is used for the unified quota system.
 // (Basically this naively maps TEMPORARY storage type to TEMPORARY filesystem
 // type, PERSISTENT storage type to PERSISTENT filesystem type and vice versa.)
@@ skipping 45 matching lines @@
 // platform apps. This function will return false if the file system name is
 // not of the form {origin}:Isolated_{id}, and will also check that there is an
 // origin and id present. It will not check that the origin or id are valid.
 FILEAPI_EXPORT bool CrackIsolatedFileSystemName(
     const std::string& filesystem_name,
     std::string* filesystem_id);
 
 }  // namespace fileapi
 
 #endif  // WEBKIT_FILEAPI_FILE_SYSTEM_UTIL_H_