Fix filesystem API file_handlers to work for drive on ChromeOS.

webkit/fileapi/external_mount_points.cc

 // Copyright (c) 2013 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.
 
 #include "webkit/fileapi/external_mount_points.h"
 
 #include "base/files/file_path.h"
 #include "base/lazy_instance.h"
 #include "base/path_service.h"
 #include "base/stl_util.h"
@@ skipping 226 matching lines @@
   std::string mount_name;
   FileSystemType cracked_type;
   base::FilePath cracked_path;
   if (!CrackVirtualPath(path, &mount_name, &cracked_type, &cracked_path))
     return FileSystemURL();
 
   return FileSystemURL(origin, type, path,
                        mount_name, cracked_type, cracked_path);
 }
 
-RemoteFileSystemProxyInterface* ExternalMountPoints::GetRemoteFileSystemProxy(
-    const std::string& mount_name) const {
+RemoteFileSystemProxyInterface*
+ExternalMountPoints::GetRemoteFileSystemProxyForPath(
+    const base::FilePath& full_path) const {
   base::AutoLock locker(lock_);
-  NameToInstance::const_iterator found = instance_map_.find(mount_name);
+
+  std::string mount_point;
+  if (!GetMountPointForPath(full_path, &mount_point, NULL))
+    return NULL;
+
+  // Get mount point instance.
+  NameToInstance::const_iterator found = instance_map_.find(mount_point);
   if (found == instance_map_.end())
     return NULL;
+
   return found->second->remote_proxy();
 }
 
 void ExternalMountPoints::AddMountPointInfosTo(
     std::vector<MountPointInfo>* mount_points) const {
   base::AutoLock locker(lock_);
   DCHECK(mount_points);
   for (NameToInstance::const_iterator iter = instance_map_.begin();
        iter != instance_map_.end(); ++iter) {
     mount_points->push_back(MountPointInfo(iter->first, iter->second->path()));
   }
 }
 
-bool ExternalMountPoints::GetVirtualPath(const base::FilePath& path_in,
+bool ExternalMountPoints::GetVirtualPath(const base::FilePath& path,
                                          base::FilePath* virtual_path) {
   DCHECK(virtual_path);
 
   base::AutoLock locker(lock_);
 
-  base::FilePath path = NormalizeFilePath(path_in);
-  std::map<base::FilePath, std::string>::reverse_iterator iter(
-      path_to_name_map_.upper_bound(path));
-  if (iter == path_to_name_map_.rend())
+  std::string mount_point_name;
+  base::FilePath mount_point_root;
+  if (!GetMountPointForPath(path, &mount_point_name, &mount_point_root))
     return false;
 
-  *virtual_path = CreateVirtualRootPath(iter->second);
-  if (iter->first == path)
+  *virtual_path = CreateVirtualRootPath(mount_point_name);
+  if (mount_point_root == path)
     return true;
-  return iter->first.AppendRelativePath(path, virtual_path);
+  return mount_point_root.AppendRelativePath(path, virtual_path);
 }
 
 base::FilePath ExternalMountPoints::CreateVirtualRootPath(
     const std::string& mount_name) const {
   return base::FilePath().AppendASCII(mount_name);
 }
 
 ExternalMountPoints::ExternalMountPoints() {}
 
 ExternalMountPoints::~ExternalMountPoints() {
@@ skipping 33 matching lines @@
   }
 
   std::map<base::FilePath, std::string>::iterator potential_child =
       path_to_name_map_.upper_bound(path);
   if (potential_child == path_to_name_map_.end())
     return true;
   return !(potential_child->first == path) &&
          !path.IsParent(potential_child->first);
 }
 
+bool ExternalMountPoints::GetMountPointForPath(
+    const base::FilePath& path,
+    std::string* mount_point_name,
+    base::FilePath* mount_point_root) const {
+  DCHECK(mount_point_name);
+
+  // Get mount point the path belongs to. It is the path whose value is smaller
+  // than normalized path, but also parents the normalized_path. Note that
+  // ExternalMountPoints are constrained so there cannot be a root path x for
+  // which the following holds: root_path(path) < x < path.
+  base::FilePath normalized_path = NormalizeFilePath(path);
+  std::map<base::FilePath, std::string>::const_reverse_iterator iter(
+      path_to_name_map_.upper_bound(normalized_path));
+  if (iter == path_to_name_map_.rend() ||
+      (iter->first != normalized_path &&
+       !iter->first.IsParent(normalized_path))) {
+    return false;
+  }
+
+  *mount_point_name = iter->second;
+  if (mount_point_root)
+    *mount_point_root = iter->first;
+  return true;
+}
+
 ScopedExternalFileSystem::ScopedExternalFileSystem(
     const std::string& mount_name,
     FileSystemType type,
     const base::FilePath& path)
     : mount_name_(mount_name) {
   ExternalMountPoints::GetSystemInstance()->RegisterFileSystem(
       mount_name, type, path);
 }
 
 base::FilePath ScopedExternalFileSystem::GetVirtualRootPath() const {
   return ExternalMountPoints::GetSystemInstance()->
       CreateVirtualRootPath(mount_name_);
 }
 
 ScopedExternalFileSystem::~ScopedExternalFileSystem() {
   ExternalMountPoints::GetSystemInstance()->RevokeFileSystem(mount_name_);
 }
 
 }  // namespace fileapi