Files.app: Fix TaskController so that it calls correct default task.

Files.app: Fix TaskController so that it calls correct default task.

Previously the TaskController calls the private version of executeDefaultTask
method that does not wait for initialization completion of FileTasks.  Thus it
tries to call default task before the default task is determined.

BUG=443942
TEST=manually

Committed: https://crrev.com/697b5dc34dfe7c4cf4e11e71b79e18cefaf7282b
Cr-Commit-Position: refs/heads/master@{#309616}

[hirono, 2014-12-24 01:36:10]

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

[hirono, 2014-12-24 01:36:11]

PTAL? Thanks!

[yawano, 2014-12-24 02:31:44]

https://codereview.chromium.org/821003002/diff/40001/ui/file_manager/file_man...
File ui/file_manager/file_manager/foreground/js/task_controller_unittest.js
(right):

https://codereview.chromium.org/821003002/diff/40001/ui/file_manager/file_man...
ui/file_manager/file_manager/foreground/js/task_controller_unittest.js:7: ]),
100);
Please change time to 0 if the intention of this code is just to make this call
async.

[hirono, 2014-12-24 02:39:11]

Thanks!

https://codereview.chromium.org/821003002/diff/40001/ui/file_manager/file_man...
File ui/file_manager/file_manager/foreground/js/task_controller_unittest.js
(right):

https://codereview.chromium.org/821003002/diff/40001/ui/file_manager/file_man...
ui/file_manager/file_manager/foreground/js/task_controller_unittest.js:7: ]),
100);
On 2014/12/24 02:31:44, yawano wrote:
> Please change time to 0 if the intention of this code is just to make this
call
> async.

Done.

[yawano, 2014-12-24 02:40:46]

lgtm. Thank you.

[hirono, 2014-12-24 03:34:23]

The CQ bit was checked by hirono@chromium.org

[commit-bot: I haz the power, 2014-12-24 03:34:56]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/821003002/60001

[commit-bot: I haz the power, 2014-12-24 03:40:35]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2014-12-24 03:40:36]

Try jobs failed on following builders:
  chromium_presubmit on tryserver.chromium.linux
(http://build.chromium.org/p/tryserver.chromium.linux/builders/chromium_presub...)

[hirono, 2014-12-24 03:43:35]

The CQ bit was checked by hirono@chromium.org

[commit-bot: I haz the power, 2014-12-24 03:44:07]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/821003002/80001

[commit-bot: I haz the power, 2014-12-24 04:38:31]

Committed patchset #5 (id:80001)

[commit-bot: I haz the power, 2014-12-24 04:39:53]

Patchset 5 (id:??) landed as
https://crrev.com/697b5dc34dfe7c4cf4e11e71b79e18cefaf7282b
Cr-Commit-Position: refs/heads/master@{#309616}