Create ListContainer and SpinnerController class.

Create ListContainer and SpinnerController class.

BUG=267281
TEST=None

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

[hirono, 2014-10-31 14:31:30]

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

[hirono, 2014-10-31 14:31:37]

Patchset #1 (id:1) has been deleted

[hirono, 2014-10-31 14:31:48]

PTAL the CL? Thanks!

[fukino, 2014-10-31 15:14:23]

Thanks! lgtm with nits:

https://codereview.chromium.org/695883002/diff/20001/ui/file_manager/file_man...
File ui/file_manager/file_manager/foreground/js/spinner_controller.js (right):

https://codereview.chromium.org/695883002/diff/20001/ui/file_manager/file_man...
ui/file_manager/file_manager/foreground/js/spinner_controller.js:8: * @param
{DirectoryModel} directoryModel
!DirectoryModel should be preferable

https://codereview.chromium.org/695883002/diff/20001/ui/file_manager/file_man...
ui/file_manager/file_manager/foreground/js/spinner_controller.js:22: * @type
{DirectoryModel}
!DirectoryModel

https://codereview.chromium.org/695883002/diff/20001/ui/file_manager/file_man...
File ui/file_manager/file_manager/foreground/js/ui/list_container.js (right):

https://codereview.chromium.org/695883002/diff/20001/ui/file_manager/file_man...
ui/file_manager/file_manager/foreground/js/ui/list_container.js:16: * @type
{Date}
!Date

https://codereview.chromium.org/695883002/diff/20001/ui/file_manager/file_man...
ui/file_manager/file_manager/foreground/js/ui/list_container.js:80: * @param
{Event} event Key event.
!Event

https://codereview.chromium.org/695883002/diff/20001/ui/file_manager/file_man...
ui/file_manager/file_manager/foreground/js/ui/list_container.js:106: * @param
{Event} event Key event.
!Event

https://codereview.chromium.org/695883002/diff/20001/ui/file_manager/file_man...
ui/file_manager/file_manager/foreground/js/ui/list_container.js:115: return;
null check for this.textSearchState

[hirono, 2014-10-31 18:07:14]

The CQ bit was checked by hirono@chromium.org

[hirono, 2014-10-31 18:07:24]

Thanks!

https://codereview.chromium.org/695883002/diff/20001/ui/file_manager/file_man...
File ui/file_manager/file_manager/foreground/js/spinner_controller.js (right):

https://codereview.chromium.org/695883002/diff/20001/ui/file_manager/file_man...
ui/file_manager/file_manager/foreground/js/spinner_controller.js:8: * @param
{DirectoryModel} directoryModel
On 2014/10/31 15:14:23, fukino wrote:
> !DirectoryModel should be preferable

Done.

https://codereview.chromium.org/695883002/diff/20001/ui/file_manager/file_man...
ui/file_manager/file_manager/foreground/js/spinner_controller.js:22: * @type
{DirectoryModel}
On 2014/10/31 15:14:23, fukino wrote:
> !DirectoryModel

Done.

https://codereview.chromium.org/695883002/diff/20001/ui/file_manager/file_man...
File ui/file_manager/file_manager/foreground/js/ui/list_container.js (right):

https://codereview.chromium.org/695883002/diff/20001/ui/file_manager/file_man...
ui/file_manager/file_manager/foreground/js/ui/list_container.js:16: * @type
{Date}
On 2014/10/31 15:14:23, fukino wrote:
> !Date

Done.

https://codereview.chromium.org/695883002/diff/20001/ui/file_manager/file_man...
ui/file_manager/file_manager/foreground/js/ui/list_container.js:80: * @param
{Event} event Key event.
On 2014/10/31 15:14:23, fukino wrote:
> !Event

Done.

https://codereview.chromium.org/695883002/diff/20001/ui/file_manager/file_man...
ui/file_manager/file_manager/foreground/js/ui/list_container.js:106: * @param
{Event} event Key event.
On 2014/10/31 15:14:23, fukino wrote:
> !Event

Done.

https://codereview.chromium.org/695883002/diff/20001/ui/file_manager/file_man...
ui/file_manager/file_manager/foreground/js/ui/list_container.js:115: return;
On 2014/10/31 15:14:23, fukino wrote:
> null check for this.textSearchState

I changed it non-nullable.

[commit-bot: I haz the power, 2014-10-31 18:08:44]

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

[hirono, 2014-10-31 18:09:35]

The CQ bit was unchecked by hirono@chromium.org

[hirono, 2014-10-31 18:11:05]

The CQ bit was checked by hirono@chromium.org

[commit-bot: I haz the power, 2014-10-31 18:13:03]

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

[hirono, 2014-10-31 18:18:56]

The CQ bit was unchecked by hirono@chromium.org

[hirono, 2014-10-31 18:19:50]

The CQ bit was checked by hirono@chromium.org

[commit-bot: I haz the power, 2014-10-31 18:21:48]

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

[commit-bot: I haz the power, 2014-10-31 19:26:32]

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

[commit-bot: I haz the power, 2014-10-31 19:26:34]

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

[hirono, 2014-11-04 00:55:16]

@fukino - I fixed a test. The event was bubbled to the parent in UI tree after
it is re-dispatched to LinkContainer. Because the event target was changed to
LinkContainer when it was re-dispatched, the handler of parent element raises an
error.

Please take another look? Thanks!

[hirono, 2014-11-04 04:10:14]

It looks tests are still broken. Let me fix again.

[hirono, 2014-11-04 05:16:27]

Done. PTAL

[fukino, 2014-11-04 06:03:43]

On 2014/11/04 05:16:27, hirono wrote:
> Done. PTAL

still lgtm.

[hirono, 2014-11-04 06:06:20]

The CQ bit was checked by hirono@chromium.org

[commit-bot: I haz the power, 2014-11-04 06:07:19]

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

[commit-bot: I haz the power, 2014-11-04 08:56:16]

Committed patchset #6 (id:120001)

[commit-bot: I haz the power, 2014-11-04 08:56:50]

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