Files.app: Omit first draw without metadata in file list

Files.app: Omit first draw without metadata in file list

Before this patch, Files.app was draw the filelist twice.
1) After fetching list of file entry, draw the list without metadata.
2) After fetching the metadata, draw the list with metadata.

This way was because the metadata fetch had been slow, so we drew the list without metadata at the first time. But this took long time in total since drawing was twice (). And sometimes #2 was not happen and caused the issue since #2 was triggered by sort.

Now fetching metadata is enough fast, and we can combine them. This patch omit the first draw #1. This improves the performance, and fixes crbug.com/375575.

BUG=375575, 367123
TEST=filelist was drawn correctly
R=mtomasz@chromium.org

Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=275808

[yoshiki, 2014-05-30 04:24:18]

@mtomasz: PTAL. Thanks.

[mtomasz, 2014-05-30 05:27:47]

lgtm

[yoshiki, 2014-05-30 05:32:31]

Thanks!

[yoshiki, 2014-05-30 05:32:36]

The CQ bit was checked by yoshiki@chromium.org

[commit-bot: I haz the power, 2014-05-30 05:34:33]

CQ is trying da patch. Follow status at
 https://chromium-status.appspot.com/cq/yoshiki@chromium.org/301943002/20001

[commit-bot: I haz the power, 2014-05-30 10:27:10]

FYI, CQ is re-trying this CL (attempt #1).
The failing builders are:
  linux_chromium_chromeos_rel on tryserver.chromium
(http://build.chromium.org/p/tryserver.chromium/builders/linux_chromium_chrome...)

[commit-bot: I haz the power, 2014-05-30 11:56:52]

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

[commit-bot: I haz the power, 2014-05-30 11:56:52]

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

[yoshiki, 2014-06-09 11:33:43]

The CQ bit was checked by yoshiki@chromium.org

[commit-bot: I haz the power, 2014-06-09 11:34:47]

CQ is trying da patch. Follow status at
 https://chromium-status.appspot.com/cq/yoshiki@chromium.org/301943002/40001

[commit-bot: I haz the power, 2014-06-09 12:58:23]

FYI, CQ is re-trying this CL (attempt #1).
The failing builders are:
  ios_dbg_simulator on tryserver.chromium
(http://build.chromium.org/p/tryserver.chromium/builders/ios_dbg_simulator/bui...)

[yoshiki, 2014-06-09 15:01:03]

Committed patchset #2 manually as r275808 (presubmit successful).