Index: chrome/browser/chromeos/drive/drive_file_system_proxy.cc |
diff --git a/chrome/browser/chromeos/drive/drive_file_system_proxy.cc b/chrome/browser/chromeos/drive/drive_file_system_proxy.cc |
index b3286af15160b414d7b239227e835adcf2e1ef7d..02aa3a7556212ba1506a958a2b2a7d5347c547db 100644 |
--- a/chrome/browser/chromeos/drive/drive_file_system_proxy.cc |
+++ b/chrome/browser/chromeos/drive/drive_file_system_proxy.cc |
@@ -696,8 +696,15 @@ bool DriveFileSystemProxy::ValidateUrl( |
if (!url.is_valid() || url.type() != fileapi::kFileSystemTypeDrive) { |
return false; |
} |
kinuko
2013/03/19 20:34:50
nit: can you drop { } for this one line body while
kinaba
2013/03/21 07:35:13
Done.
|
- *file_path = url.virtual_path(); |
- return true; |
+ |
+ // |url.virtual_path()| cannot be used directly because in the case the url is |
+ // isolated file system url, the virtual path will be formatted as |
+ // <isolated_file_system_id>/<file_name> and thus unusable by drive file |
+ // system. |
+ // TODO(kinaba): fix other uses of virtual_path() as in |
+ // https://codereview.chromium.org/12483010/ |
+ *file_path = util::ExtractDrivePath(url.path()); |
+ return !file_path->empty(); |
} |
void DriveFileSystemProxy::CallDriveFileSystemMethodOnUIThread( |