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/utf_string_conversions.h"
 #include "testing/gtest/include/gtest/gtest.h"
 
@@ skipping 58 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) {
+  const GURL dummy_origin("chrome-extension://extension-id");
+  EXPECT_EQ(base::FilePath::FromUTF8Unsafe("drive/foo/bar"),
+            ExtractDrivePathFromFileSystemUrl(
+                fileapi::FileSystemURL::CreateForTest(
+                    dummy_origin,
+                    fileapi::kFileSystemTypeDrive,
+                    base::FilePath::FromUTF8Unsafe("/special/drive/foo/bar"))));
+  EXPECT_EQ(base::FilePath(),
+            ExtractDrivePathFromFileSystemUrl(
+                fileapi::FileSystemURL::CreateForTest(
+                    dummy_origin,
+                    fileapi::kFileSystemTypeDrive,
+                    base::FilePath::FromUTF8Unsafe("/special/drivex/foo"))));
+  EXPECT_EQ(base::FilePath(),

[hashimoto, 2013/04/08 03:51:00]

How about testing a case where |virtual_path| is different from |path|, if it's
not difficult?

[kinaba, 2013/04/08 05:47:50]

Done. I hope the things became clearer.

+            ExtractDrivePathFromFileSystemUrl(
+                fileapi::FileSystemURL::CreateForTest(
+                    dummy_origin,
+                    fileapi::kFileSystemTypeNativeLocal,
+                    base::FilePath::FromUTF8Unsafe("/Downloads/foo.txt"))));
+}
+
 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