[Files.app] Refile a timing of doing selectBestMatchItem().

chrome/browser/resources/file_manager/js/navigation_list.js

 // Copyright (c) 2012 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';
 
 /**
  * Entry of NavigationListModel. This construtor should be called only from
  * the helper methods (NavigationModelItem.createWithPath/createWithEntry).
  *
@@ skipping 428 matching lines @@
 function NavigationList() {
 }
 
 /**
  * NavigationList inherits cr.ui.List.
  */
 NavigationList.prototype = {
   __proto__: cr.ui.List.prototype,
 
   set dataModel(dataModel) {
-    if (!this.boundHandleListChanged_)
-      this.boundHandleListChanged_ = this.onListContentChanged_.bind(this);
-
-    if (this.dataModel_) {
-      dataModel.removeEventListener('change', this.boundHandleListChanged_);
-      dataModel.removeEventListener('permuted', this.boundHandleListChanged_);
-    }
-
-    dataModel.addEventListener('change', this.boundHandleListChanged_);
-    dataModel.addEventListener('permuted', this.boundHandleListChanged_);
-
     var parentSetter = cr.ui.List.prototype.__lookupSetter__('dataModel');
     return parentSetter.call(this, dataModel);
   },
 
   get dataModel() {
     return this.dataModel_;
   },
 
   // TODO(yoshiki): Add a setter of 'directoryModel'.
 };
@@ skipping 215 matching lines @@
 /**
  * Invoked when the current directory is changed.
  * @param {Event} event The event.
  * @private
  */
 NavigationList.prototype.onCurrentDirectoryChanged_ = function(event) {
   this.selectBestMatchItem_();
 };
 
 /**
- * Invoked when the content in the data model is changed.
- * @param {Event} event The event.
+ * Invoked when the batch update is finished.
+ * @override
+ */
+NavigationList.prototype.endBatchUpdates = function() {

[hirono, 2013/09/03 08:20:30]

It seems that the endBatchUpdates will not be called when the dataModel#change
event triggered. Is it OK?

[yoshiki, 2013/09/03 09:33:32]

Good catch. Fixed.

+  // Calls the parent class method.
+  cr.ui.List.prototype.endBatchUpdates.call(this);
+
+  this.onListContentChanged_();
+};
+
+/**
+ * Invoked when the elements in the list is updated.
  * @private
  */
-NavigationList.prototype.onListContentChanged_ = function(event) {
+NavigationList.prototype.onListContentChanged_ = function() {
   this.selectBestMatchItem_();
 };
 
 /**
  * Synchronizes the volume list selection with the current directory, after
  * it is changed outside of the volume list.
  * @private
  */
 NavigationList.prototype.selectBestMatchItem_ = function() {
   var entry = this.directoryModel_.getCurrentDirEntry();
@@ skipping 27 matching lines @@
     var itemPath = this.dataModel.item(i).path;
     if (PathUtil.getRootPath(itemPath) == newRootPath) {
       // Not to invoke the handler of this instance, sets the guard.
       this.dontHandleSelectionEvent_ = true;
       this.selectionModel.selectedIndex = i;
       this.dontHandleSelectionEvent_ = false;
       return;
     }
   }
 };