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

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

Issue 2633053003: Fix broken quick view in non-local volumes. (Closed)
Patch Set: Do not query unnecessary externalFileUrl. Created 3 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
« no previous file with comments | « no previous file | ui/file_manager/file_manager/foreground/js/quick_view_controller.js » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
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 667e70aa0a7de025659bc6bd6b8a88b01a982992..f779b039a62ecd9bd3ec93e476c0bafe9ed6db04 100644
--- a/ui/file_manager/file_manager/foreground/js/file_manager.js
+++ b/ui/file_manager/file_manager/foreground/js/file_manager.js
@@ -548,7 +548,8 @@ FileManager.prototype = /** @struct */ {
assert(this.metadataModel_), assert(this.selectionHandler_),
assert(this.ui_.listContainer), assert(this.quickViewModel_),
assert(this.taskController_), fileListSelectionModel,
- assert(this.quickViewUma_), metadataBoxController, this.dialogType);
+ assert(this.quickViewUma_), metadataBoxController, this.dialogType,
+ assert(this.volumeManager_));
if (this.dialogType === DialogType.FULL_PAGE) {
importer.importEnabled().then(
« no previous file with comments | « no previous file | ui/file_manager/file_manager/foreground/js/quick_view_controller.js » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698