Chromium Code Reviews| 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( |