Files.app: Reset hidden state of default commands when input elements are focused.

Files.app: Reset hidden state of default commands when input elements are focused.

paste command's hidden property is set true when one directory is focused.
However, it will not be reset when an input is focused just after on directory is focused,
because different handler (forceDefaultHandler) handles canExecute event when inputs are focused.
All cut/copy/paste/delete commands should be shown in input element,
so this CL reset the hidden property of them.

BUG=528843
TEST=manually tested on rename input, search input, and filename input on save-as dialog.

Committed: https://crrev.com/b181a7749b8ec5eaa7aad6e42db9bdf0fc450bda
Cr-Commit-Position: refs/heads/master@{#347870}

[fukino, 2015-09-09 06:33:02]

fukino@chromium.org changed reviewers:
+ yawano@chromium.org

[fukino, 2015-09-09 06:33:03]

PTAL. Thanks!

[yawano, 2015-09-09 07:25:58]

lgtm with nit.

https://codereview.chromium.org/1312583009/diff/1/ui/file_manager/file_manage...
File ui/file_manager/file_manager/foreground/js/file_manager_commands.js
(right):

https://codereview.chromium.org/1312583009/diff/1/ui/file_manager/file_manage...
ui/file_manager/file_manager/foreground/js/file_manager_commands.js:222: if
(event.command.id === commandId) {
nit: You can return earlier as we are doing around line 216.

[fukino, 2015-09-09 07:31:05]

Thank you!

https://codereview.chromium.org/1312583009/diff/1/ui/file_manager/file_manage...
File ui/file_manager/file_manager/foreground/js/file_manager_commands.js
(right):

https://codereview.chromium.org/1312583009/diff/1/ui/file_manager/file_manage...
ui/file_manager/file_manager/foreground/js/file_manager_commands.js:222: if
(event.command.id === commandId) {
On 2015/09/09 07:25:58, yawano wrote:
> nit: You can return earlier as we are doing around line 216.

Done.

[fukino, 2015-09-09 07:31:10]

The CQ bit was checked by fukino@chromium.org

[fukino, 2015-09-09 07:31:10]

The patchset sent to the CQ was uploaded after l-g-t-m from yawano@chromium.org
Link to the patchset: https://codereview.chromium.org/1312583009/#ps20001
(title: "Use early return.")

[commit-bot: I haz the power, 2015-09-09 07:32:10]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1312583009/20001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1312583009/20001

[commit-bot: I haz the power, 2015-09-09 09:07:11]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2015-09-09 09:07:56]

Patchset 2 (id:??) landed as
https://crrev.com/b181a7749b8ec5eaa7aad6e42db9bdf0fc450bda
Cr-Commit-Position: refs/heads/master@{#347870}