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

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

Issue 18024002: Add the new window item to the Files.app's context menu on the launcher. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 7 years, 6 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/js/file_manager.js
diff --git a/chrome/browser/resources/file_manager/js/file_manager.js b/chrome/browser/resources/file_manager/js/file_manager.js
index 08e892eaf10f70fc9f44e15fd700d93ed74fced3..10428fb3e4e1df09baf22d19951b51b3a72ce9e5 100644
--- a/chrome/browser/resources/file_manager/js/file_manager.js
+++ b/chrome/browser/resources/file_manager/js/file_manager.js
@@ -625,7 +625,7 @@ var BOTTOM_MARGIN_FOR_PREVIEW_PANEL_PX = 52;
Commands.newFolderCommand, this, this.directoryModel_);
CommandUtil.registerCommand(doc, 'newwindow',
- Commands.newWindowCommand, this);
+ Commands.newWindowCommand, this, this.directoryModel_);
CommandUtil.registerCommand(doc, 'change-default-app',
Commands.changeDefaultAppCommand, this);
@@ -790,9 +790,8 @@ var BOTTOM_MARGIN_FOR_PREVIEW_PANEL_PX = 52;
* @private
*/
FileManager.prototype.initStrings_ = function(callback) {
- chrome.fileBrowserPrivate.getStrings(function(strings) {
- loadTimeData.data = strings;
- this.loadTimeDataAvailable = true;
+ chrome.storage.local.get('strings', function(items) {
yoshiki 2013/06/27 10:25:40 If Files.app is launched via chrome://files (it me
mtomasz 2013/06/27 10:26:57 Good catch. I'll fix it. We need the strings here,
mtomasz 2013/06/27 12:03:09 Done.
+ loadTimeData.data = items['strings'];
callback();
});
};

Powered by Google App Engine
This is Rietveld 408576698