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

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

Issue 883973002: Move cloud import button to toolbar, make material designey. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Add test coverage for new command update logic. Fix a type, and undo some early debug code. Created 5 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: 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 1e8d3e8b99110f70cd93d621578bba3faa3af00a..a7aed7df8448bc12002b3b089f4ede02a9e3d115 100644
--- a/ui/file_manager/file_manager/foreground/js/file_manager.js
+++ b/ui/file_manager/file_manager/foreground/js/file_manager.js
@@ -485,14 +485,14 @@ FileManager.prototype = /** @struct */ {
importer.importEnabled().then(
function(enabled) {
if (enabled) {
+ var commandAdapter = new importer.ButtonCommandAdapter(this);
this.importController_ = new importer.ImportController(
new importer.RuntimeControllerEnvironment(this),
/** @type {!importer.MediaScanner} */ (
this.mediaScanner_),
/** @type {!importer.ImportRunner} */ (
this.mediaImportHandler_),
- this.commandHandler.updateAvailability.bind(
- this.commandHandler));
+ commandAdapter.update.bind(commandAdapter));
}
}.bind(this));

Powered by Google App Engine
This is Rietveld 408576698