Files.app: Make all columns resizable in table view.

Files.app: Make all columns resizable in table view.

BUG=458356
TEST=manually tested by dragging each separator on the column header.

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

[fukino, 2015-02-13 06:39:46]

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

[fukino, 2015-02-13 06:39:55]

PTAL. Thanks!

[hirono, 2015-02-13 06:46:22]

Thanks! lgtm

https://codereview.chromium.org/926483003/diff/1/ui/file_manager/file_manager...
File ui/file_manager/file_manager/foreground/js/ui/file_table.js (right):

https://codereview.chromium.org/926483003/diff/1/ui/file_manager/file_manager...
ui/file_manager/file_manager/foreground/js/ui/file_table.js:126: if (columnIndex
< 0 ||
nit: We can remove the check, or assert?

[fukino, 2015-02-13 06:54:42]

Thank!

https://codereview.chromium.org/926483003/diff/1/ui/file_manager/file_manager...
File ui/file_manager/file_manager/foreground/js/ui/file_table.js (right):

https://codereview.chromium.org/926483003/diff/1/ui/file_manager/file_manager...
ui/file_manager/file_manager/foreground/js/ui/file_table.js:126: if (columnIndex
< 0 ||
On 2015/02/13 06:46:22, hirono wrote:
> nit: We can remove the check, or assert?

Maybe we can use assert, but some investigation is needed to prove it.
Let me this range-check as is for now.

[fukino, 2015-02-13 11:25:33]

The CQ bit was checked by fukino@chromium.org

[commit-bot: I haz the power, 2015-02-13 11:26:33]

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

[commit-bot: I haz the power, 2015-02-13 12:31:10]

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

[commit-bot: I haz the power, 2015-02-13 12:31:11]

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

[fukino, 2015-02-13 12:38:36]

The CQ bit was checked by fukino@chromium.org

[commit-bot: I haz the power, 2015-02-13 12:39:40]

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

[commit-bot: I haz the power, 2015-02-13 13:33:28]

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

[commit-bot: I haz the power, 2015-02-13 13:33:30]

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

[fukino, 2015-02-13 14:54:43]

The CQ bit was checked by fukino@chromium.org

[commit-bot: I haz the power, 2015-02-13 14:55:45]

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

[commit-bot: I haz the power, 2015-02-13 15:52:15]

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

[commit-bot: I haz the power, 2015-02-13 15:52:16]

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

[fukino, 2015-02-14 09:26:37]

The CQ bit was checked by fukino@chromium.org

[commit-bot: I haz the power, 2015-02-14 09:27:27]

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

[commit-bot: I haz the power, 2015-02-14 09:28:10]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2015-02-14 09:29:02]

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