[Files.app] Use generateTopLevelEntries() method to retrieve the list on the top level

chrome/browser/resources/file_manager/js/directory_tree.js

 // Copyright (c) 2013 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 'use strict';
 
 ////////////////////////////////////////////////////////////////////////////////
 // DirectoryTreeUtil
 
 /**
@@ skipping 577 matching lines @@
       completionCallback);
 };
 
 /**
  * Retrieves the latest subdirectories and update them on the tree.
  * @param {boolean} recursive True if the update is recursively.
  * @param {function()=} opt_successCallback Callback called on success.
  * @param {function()=} opt_errorCallback Callback called on error.
  */
 DirectoryTree.prototype.updateSubDirectories = function(
     recursive, opt_successCallback, opt_errorCallback) {

[hirono, 2013/09/10 00:51:13]

opt_errorCallback will not be called.
Can we remove it?

[yoshiki, 2013/09/10 01:01:01]

I want to keep the same interface with
DirectoryItem.prototype.updateSubDirectories() since they has the same name.

-  var myDriveItem = this.items[0];
-  DirectoryTreeUtil.updateSubDirectories(
-      myDriveItem,
-      function(entries) {
-        this.entries_ = entries;
-        this.redraw(recursive);
-        if (opt_successCallback)
-          opt_successCallback();
-      }.bind(this),
-      opt_errorCallback);
+  this.entries_ = DirectoryTreeUtil.generateTopLevelEntries();
+  this.redraw(recursive);
+  if (opt_successCallback)
+    opt_successCallback();
 };
 
 /**
  * Redraw the list.
  * @param {boolean} recursive True if the update is recursively. False if the
  *     only root items are updated.
  */
 DirectoryTree.prototype.redraw = function(recursive) {
   DirectoryTreeUtil.updateSubElementsFromList(
       this,
@@ skipping 47 matching lines @@
   this.style.paddingBottom = margin + 'px';
   this.scrollBar_.setBottomMarginForPanel(margin);
 };
 
 /**
  * Updates the UI after the layout has changed.
  */
 DirectoryTree.prototype.relayout = function() {
   cr.dispatchSimpleEvent(this, 'relayout');
 };