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

Unified Diff: ui/file_manager/file_manager/foreground/js/file_manager.js

Issue 2828633002: Compile file_selection and some other Files app files in gyp v2 (Closed)
Patch Set: Add TODO to depend on externs/ instead of background/. Created 3 years, 8 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: ui/file_manager/file_manager/foreground/js/file_manager.js
diff --git a/ui/file_manager/file_manager/foreground/js/file_manager.js b/ui/file_manager/file_manager/foreground/js/file_manager.js
index f17131c58da0750a8d28c3435b81f402a454f547..16041cf126adbd056ec6ea5360948c97610d8c31 100644
--- a/ui/file_manager/file_manager/foreground/js/file_manager.js
+++ b/ui/file_manager/file_manager/foreground/js/file_manager.js
@@ -636,7 +636,8 @@ FileManager.prototype = /** @struct */ {
FileManager.prototype.initCommands_ = function() {
assert(this.ui_.textContextMenu);
- this.commandHandler_ = new CommandHandler(this);
+ this.commandHandler_ =
+ new CommandHandler(this, assert(this.selectionHandler_));
// TODO(hirono): Move the following block to the UI part.
var commandButtons = this.dialogDom_.querySelectorAll('button[command]');
@@ -1004,7 +1005,10 @@ FileManager.prototype = /** @struct */ {
this.folderShortcutsModel_ = new FolderShortcutsDataModel(
this.volumeManager_);
- this.selectionHandler_ = new FileSelectionHandler(this);
+ this.selectionHandler_ = new FileSelectionHandler(
+ assert(this.directoryModel_), assert(this.fileOperationManager_),
+ assert(this.ui_.listContainer), assert(this.metadataModel_),
+ assert(this.volumeManager_));
this.directoryModel_.getFileListSelection().addEventListener('change',
this.selectionHandler_.onFileSelectionChanged.bind(

Powered by Google App Engine
This is Rietveld 408576698