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

Unified Diff: chrome/browser/resources/file_manager/background/js/background.js

Issue 132453007: Migrate fullPaths to URLs in appState. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: 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/background/js/background.js
diff --git a/chrome/browser/resources/file_manager/background/js/background.js b/chrome/browser/resources/file_manager/background/js/background.js
index edd3620feb42131f241e0c66079347db9d6ee296..98085c074c4948f0b6ccfd7587c5da46daec3ffe 100644
--- a/chrome/browser/resources/file_manager/background/js/background.js
+++ b/chrome/browser/resources/file_manager/background/js/background.js
@@ -538,9 +538,9 @@ function launchFileManager(opt_appState, opt_id, opt_type, opt_callback) {
}
// Selection path specified, and it is different.
- if (opt_appState.selectionPath &&
- opt_appState.selectionPath !==
- contentWindow.appState.selectionPath) {
+ if (opt_appState.selectionURL &&
+ opt_appState.selectionURL !==
+ contentWindow.appState.selectionURL) {
continue;
}
@@ -669,9 +669,9 @@ Background.prototype.onExecute_ = function(action, details) {
},
// It is not allowed to call getParent() here, since there may be
// no permissions to access it at this stage. Therefore we are passing
- // the selectionPath only, and the currentDirectory will be resolved
+ // the selectionURL only, and the currentDirectory will be resolved
// later.
- selectionPath: details.entries[0].fullPath
+ selectionURL: details.entries[0].toURL()
};
// For mounted devices just focus any Files.app window. The mounted
// volume will appear on the navigation list.

Powered by Google App Engine
This is Rietveld 408576698