Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(8)

Unified Diff: chrome/browser/resources/file_manager/foreground/js/file_tasks.js

Issue 148193002: Removed most of path utility functions in Files app. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Rebased. Created 6 years, 11 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: chrome/browser/resources/file_manager/foreground/js/file_tasks.js
diff --git a/chrome/browser/resources/file_manager/foreground/js/file_tasks.js b/chrome/browser/resources/file_manager/foreground/js/file_tasks.js
index 474e9d9813f80a4900dbf493245940dffc4f7008..f9e37b48a86c1f4d8de1a82255c0b6be1f70ee50 100644
--- a/chrome/browser/resources/file_manager/foreground/js/file_tasks.js
+++ b/chrome/browser/resources/file_manager/foreground/js/file_tasks.js
@@ -211,9 +211,10 @@ FileTasks.prototype.processTasks_ = function(tasks) {
this.tasks_ = [];
var id = chrome.runtime.id;
var isOnDrive = false;
+ var fm = this.fileManager_;
for (var index = 0; index < this.entries_.length; ++index) {
- // TODO(mtomasz): Use Entry instead of paths.
- if (PathUtil.isDriveBasedPath(this.entries_[index].fullPath)) {
+ var locationInfo = fm.volumeManager.getLocationInfo(this.entries_[index]);
+ if (locationInfo && locationInfo.isDriveBased) {
isOnDrive = true;
break;
}

Powered by Google App Engine
This is Rietveld 408576698