Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(5101)

Unified Diff: chrome/browser/resources/file_manager/js/file_manager.js

Issue 7038024: Move sorting logic from table to list. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Fixed autofill phones. Created 9 years, 7 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « no previous file | chrome/browser/resources/file_manager/main.html » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/browser/resources/file_manager/js/file_manager.js
diff --git a/chrome/browser/resources/file_manager/js/file_manager.js b/chrome/browser/resources/file_manager/js/file_manager.js
index 58f073b438f73afce8ad6d815d8c15475e59ac37..b96ecd91640e434e5c6181cbc809e5c3ad844952 100644
--- a/chrome/browser/resources/file_manager/js/file_manager.js
+++ b/chrome/browser/resources/file_manager/js/file_manager.js
@@ -518,20 +518,18 @@ FileManager.prototype = {
// Always sharing the data model between the detail/thumb views confuses
// them. Instead we maintain this bogus data model, and hook it up to the
// view that is not in use.
- this.emptyDataModel_ = new cr.ui.table.TableDataModel([]);
+ this.emptyDataModel_ = new cr.ui.ArrayDataModel([]);
- this.dataModel_ = new cr.ui.table.TableDataModel([]);
+ this.dataModel_ = new cr.ui.ArrayDataModel([]);
this.dataModel_.sort('name');
- this.dataModel_.addEventListener('sorted',
- this.onDataModelSorted_.bind(this));
this.dataModel_.prepareSort = this.prepareSort_.bind(this);
if (this.dialogType_ == FileManager.DialogType.SELECT_OPEN_FILE ||
this.dialogType_ == FileManager.DialogType.SELECT_OPEN_FOLDER ||
this.dialogType_ == FileManager.DialogType.SELECT_SAVEAS_FILE) {
- this.selectionModelClass_ = cr.ui.table.TableSingleSelectionModel;
+ this.selectionModelClass_ = cr.ui.ListSingleSelectionModel;
} else {
- this.selectionModelClass_ = cr.ui.table.TableSelectionModel;
+ this.selectionModelClass_ = cr.ui.ListSelectionModel;
}
this.initTable_();
@@ -1426,16 +1424,6 @@ FileManager.prototype = {
};
/**
- * Invoked by the table dataModel after a sort completes.
- *
- * We use this hook to make sure selected files stay visible after a sort.
- */
- FileManager.prototype.onDataModelSorted_ = function() {
- var i = this.currentList_.selectionModel.leadIndex;
- this.currentList_.scrollIntoView(i);
- }
-
- /**
* Update the selection summary UI when the selection summarization completes.
*/
FileManager.prototype.onSelectionSummarized_ = function() {
@@ -1568,9 +1556,6 @@ FileManager.prototype = {
function onReadSome(entries) {
if (entries.length == 0) {
- if (self.dataModel_.sortStatus.field != 'name')
- self.dataModel_.updateIndex(0);
-
if (opt_callback)
opt_callback();
return;
@@ -1616,7 +1601,6 @@ FileManager.prototype = {
var spliceArgs = [].slice.call(this.rootEntries_);
spliceArgs.unshift(0, 0); // index, deleteCount
self.dataModel_.splice.apply(self.dataModel_, spliceArgs);
- self.dataModel_.updateIndex(0);
if (opt_callback)
opt_callback();
« no previous file with comments | « no previous file | chrome/browser/resources/file_manager/main.html » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698