New issue
Advanced search Search tips
Note: Color blocks (like or ) mean that a user may not be available. Tooltip shows the reason.

Issue 833751 link

Starred by 1 user

Issue metadata

Status: Available
Owner: ----
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 3
Type: Bug



Sign in to add a comment

Change file_manager_commands.js to use Polymer

Project Member Reported by sashab@chromium.org, Apr 17 2018

Issue description

Change file_manager_commands.js to use Polymer.

It currently uses cr.ui.commands, which can't be called asynchronously (canExecute has to be synchronous). This means we can't check things against the API before enabling them (e.g. for permissions checks).

Look at making something similar to command_manager.js in bookmarks:
https://cs.chromium.org/chromium/src/chrome/browser/resources/md_bookmarks/command_manager.js?sq=package:chromium&dr
 

Comment 1 by sashab@chromium.org, Apr 17 2018

Components: Platform>Apps>FileManager

Sign in to add a comment