Populate volume items without resolving their display roots.

Populate volume items without resolving their display roots.

* Populate volumes without resolving their display root.
The display root will be resolved after decorate() of the volume item.
If failed, the display root will be tried to be resolved when the volume item is activated (by click/Ctrl+N/etc...).
* Remove DirectoryTree.models_, which was a temporary variable to keep resolved display roots.

BUG=413317
TEST=manually tested the case when the first resolveDisplayRoot() fails. run browser_tests.

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

[fukino, 2014-09-25 06:15:41]

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

[fukino, 2014-09-25 06:15:42]

Tomasz, could you take a look? Thanks!

https://codereview.chromium.org/593573003/diff/40001/chrome/test/data/extensi...
File
chrome/test/data/extensions/api_test/file_manager_browsertest/file_manager/folder_shortcuts.js
(right):

https://codereview.chromium.org/593573003/diff/40001/chrome/test/data/extensi...
chrome/test/data/extensions/api_test/file_manager_browsertest/file_manager/folder_shortcuts.js:45:
navItem: '#tree-item-autogen-id-2',
Auto-generated id for the drive volume is slid by the asynchronous display-root
resolution, but still deterministic.

[mtomasz, 2014-09-25 09:28:30]

Basically looks good, but I'll take a deeper look tomorrow. Thanks for fixing!

[mtomasz, 2014-09-26 02:35:41]

lgtm

[fukino, 2014-09-26 02:38:41]

On 2014/09/26 02:35:41, mtomasz wrote:
> lgtm

Thanks!

[fukino, 2014-09-26 02:38:45]

The CQ bit was checked by fukino@chromium.org

[commit-bot: I haz the power, 2014-09-26 02:39:43]

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

[commit-bot: I haz the power, 2014-09-26 04:46:45]

Committed patchset #3 (id:40001) as 4496259415bf236e11780054c33001701db66fcb

[commit-bot: I haz the power, 2014-09-26 04:47:24]

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