Use ExtractDrivePath(path()) instead of virtual_path() for getting Drive path.

chrome/browser/chromeos/drive/drive_file_system_util_unittest.cc

 // 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.
 
 #include "chrome/browser/chromeos/drive/drive_file_system_util.h"
 
 #include "base/files/file_path.h"
+#include "base/message_loop.h"
 #include "base/utf_string_conversions.h"
 #include "testing/gtest/include/gtest/gtest.h"
+#include "webkit/fileapi/external_mount_points.h"
+#include "webkit/fileapi/file_system_context.h"
+#include "webkit/fileapi/file_system_task_runners.h"
+#include "webkit/fileapi/file_system_url.h"
+#include "webkit/fileapi/isolated_context.h"
+#include "webkit/fileapi/mock_file_system_options.h"
 
 namespace drive {
 namespace util {
 
 TEST(DriveFileSystemUtilTest, FilePathToDriveURL) {
   base::FilePath path;
 
   // Path with alphabets and numbers.
   path = GetDriveMyDriveRootPath().AppendASCII("foo/bar012.txt");
   EXPECT_EQ(path, DriveURLToFilePath(FilePathToDriveURL(path)));
@@ skipping 49 matching lines @@
             ExtractDrivePath(
                 base::FilePath::FromUTF8Unsafe("/special/drive")));
   EXPECT_EQ(base::FilePath::FromUTF8Unsafe("drive/foo.txt"),
             ExtractDrivePath(
                 base::FilePath::FromUTF8Unsafe("/special/drive/foo.txt")));
   EXPECT_EQ(base::FilePath::FromUTF8Unsafe("drive/subdir/foo.txt"),
             ExtractDrivePath(base::FilePath::FromUTF8Unsafe(
                 "/special/drive/subdir/foo.txt")));
 }
 
+TEST(DriveFileSystemUtilTest, ExtractDrivePathFromFileSystemUrl) {
+  // Set up file system context for testing.
+  MessageLoop message_loop;
+  scoped_refptr<fileapi::ExternalMountPoints> mount_points =
+      fileapi::ExternalMountPoints::CreateRefCounted();
+  scoped_refptr<fileapi::FileSystemContext> context(
+      new fileapi::FileSystemContext(
+          fileapi::FileSystemTaskRunners::CreateMockTaskRunners(),
+          mount_points,
+          NULL,  // special_storage_policy
+          NULL,  // quota_manager_proxy,
+          base::FilePath::FromUTF8Unsafe("/tmp"),  // partition_path

[hashimoto, 2013/04/08 06:03:49]

Using "/tmp" seems not multi platform friendly.
How about using base::ScopedTempDir?
The same goes for other "/tmp".

[kinaba, 2013/04/08 06:43:41]

Done.

+          fileapi::CreateAllowFileAccessOptions()));
+
+  // Type:"external" + virtual_path:"drive/foo/bar" resolves to "drive/foo/bar".
+  mount_points->RegisterRemoteFileSystem(
+      "drive",
+      fileapi::kFileSystemTypeDrive,
+      NULL,  // RemoteFileSystemProxyInterface
+      base::FilePath::FromUTF8Unsafe("/special/drive"));

[hashimoto, 2013/04/08 06:03:49]

How about using drive::util::GetDriveMountPointPath() like
DriveSystemService::AddDriveMountPoint()?
The same goes for "drive".

[kinaba, 2013/04/08 06:43:41]

Done (and added a test for the case "drive" is not used as the mount name).

+  EXPECT_EQ(
+      base::FilePath::FromUTF8Unsafe("drive/foo/bar"),
+      ExtractDrivePathFromFileSystemUrl(context->CrackURL(GURL(
+          "filesystem:chrome-extension://dummy-id/external/drive/foo/bar"))));
+
+  // Type:"external" + virtual_path:"Downloads/foo" is not a Drive path.
+  mount_points->RegisterFileSystem(
+      "Downloads",
+      fileapi::kFileSystemTypeNativeLocal,
+      base::FilePath::FromUTF8Unsafe("/tmp"));
+  EXPECT_EQ(
+      base::FilePath(),
+      ExtractDrivePathFromFileSystemUrl(context->CrackURL(GURL(
+          "filesystem:chrome-extension://dummy-id/external/Downloads/foo"))));
+
+  // Type:"isolated" + virtual_path:"isolated_id/name" mapped on a Drive path.
+  std::string isolated_name;
+  std::string isolated_id =
+      fileapi::IsolatedContext::GetInstance()->RegisterFileSystemForPath(
+          fileapi::kFileSystemTypeNativeForPlatformApp,
+          base::FilePath::FromUTF8Unsafe("/special/drive/bar/buz"),

[hashimoto, 2013/04/08 06:03:49]

How about using GetDriveMountPointPath()?

[kinaba, 2013/04/08 06:43:41]

Done.

+          &isolated_name);
+  EXPECT_EQ(
+      base::FilePath::FromUTF8Unsafe("drive/bar/buz"),
+      ExtractDrivePathFromFileSystemUrl(context->CrackURL(GURL(
+          "filesystem:chrome-extension://dummy-id/isolated/" +
+          isolated_id + "/" + isolated_name))));
+}
+
 TEST(DriveFileSystemUtilTest, EscapeUnescapeCacheFileName) {
   const std::string kUnescapedFileName(
       "tmp:`~!@#$%^&*()-_=+[{|]}\\\\;\',<.>/?");
   const std::string kEscapedFileName(
       "tmp:`~!@#$%25^&*()-_=+[{|]}\\\\;\',<%2E>%2F?");
   EXPECT_EQ(kEscapedFileName, EscapeCacheFileName(kUnescapedFileName));
   EXPECT_EQ(kUnescapedFileName, UnescapeCacheFileName(kEscapedFileName));
 }
 
 TEST(DriveFileSystemUtilTest, EscapeUtf8FileName) {
@@ skipping 97 matching lines @@
   EXPECT_FALSE(util::IsSpecialResourceId("file:123"));
   EXPECT_FALSE(util::IsSpecialResourceId("folder:root"));
   EXPECT_FALSE(util::IsSpecialResourceId("folder:xyz"));
 
   EXPECT_TRUE(util::IsSpecialResourceId("<drive>"));
   EXPECT_TRUE(util::IsSpecialResourceId("<other>"));
 }
 
 }  // namespace util
 }  // namespace drive