Comment fix for r144115 (Manage IsolatedContext with reference counts)

webkit/fileapi/isolated_context.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_ISOLATED_CONTEXT_H_
 #define WEBKIT_FILEAPI_ISOLATED_CONTEXT_H_
 
 #include <map>
 #include <set>
 #include <string>
@@ skipping 18 matching lines @@
   // The instance is lazily created per browser process.
   static IsolatedContext* GetInstance();
 
   // Registers a new file isolated filesystem with the given set of files
   // and returns the new filesystem_id.  The files are registered with their
   // basenames as their keys so that later we can resolve the full paths
   // for the given file name in the isolated filesystem.  We only expose the
   // key and the ID for the newly created filesystem to the renderer for
   // the sake of security.
   //
+  // Note that all the given paths in |fileset| must be absolute paths.
+  //
   // The renderer will be sending filesystem requests with a virtual path like
   // '/<filesystem_id>/<relative_path_from_the_basename_of_dropped_path>'
   // for which we could crack in the browser by calling CrackIsolatedPath to
   // get the full path.
   //
   // For example: if a dropped file has a path like '/a/b/foo' we register
   // the path with the key 'foo' in the newly created filesystem.
   // Later if the context is asked to crack a virtual path like '/<fsid>/foo'
   // it can properly return the original path '/a/b/foo' by looking up the
   // internal mapping.  Similarly if a dropped entry is a directory and its
   // path is like '/a/b/dir' a virtual path like '/<fsid>/dir/foo' can be
   // cracked into '/a/b/dir/foo'.
-  //
-  // This may return an empty string (thus invalid as an ID) if the given
-  // file set contains non absolute paths.
   std::string RegisterIsolatedFileSystem(const std::set<FilePath>& fileset);
 
   // Revokes filesystem specified by the given filesystem_id.
   void RevokeIsolatedFileSystem(const std::string& filesystem_id);
 
   // Cracks the given |virtual_path| (which should look like
   // "/<filesystem_id>/<relative_path>") and populates the |filesystem_id|
   // and |platform_path| if the embedded <filesystem_id> is registerred
   // to this context.  |root_path| is also populated to have the platform
   // root (toplevel) path for the |virtual_path|
@@ skipping 53 matching lines @@
   // Detailed filesystem permission may be provided by an external
   // security policy manager, e.g. ChildProcessSecurityPolicy.
   std::set<std::string> writable_ids_;
 
   DISALLOW_COPY_AND_ASSIGN(IsolatedContext);
 };
 
 }  // namespace fileapi
 
 #endif  // WEBKIT_FILEAPI_ISOLATED_CONTEXT_H_