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

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

Issue 101243003: Remove getCurrentDirectoryURL and getCurrentDirPath from directory_model.js (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 7 years 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_manager_commands.js
diff --git a/chrome/browser/resources/file_manager/foreground/js/file_manager_commands.js b/chrome/browser/resources/file_manager/foreground/js/file_manager_commands.js
index 7360cd69d57b15304dae4574717561eacb54af6c..7cd972249ddd2a0e85f9ad3c40f1293912dd195f 100644
--- a/chrome/browser/resources/file_manager/foreground/js/file_manager_commands.js
+++ b/chrome/browser/resources/file_manager/foreground/js/file_manager_commands.js
@@ -451,8 +451,10 @@ CommandHandler.COMMANDS_['new-folder'] = {
*/
CommandHandler.COMMANDS_['new-window'] = {
execute: function(event, fileManager) {
+ // TODO(mtomasz): Use Entry.toURL() instead of fullPath.
fileManager.backgroundPage.launchFileManager({
- defaultPath: fileManager.getCurrentDirectory()
+ defaultPath: fileManager.getCurrentDirectoryEntry() &&
+ fileManager.getCurrentDirectoryEntry().fullPath
});
},
canExecute: function(event, fileManager) {

Powered by Google App Engine
This is Rietveld 408576698