DOMUI Prefs: Replace search engine edit overlay with inline editing.

chrome/browser/resources/shared/js/cr/ui/list_single_selection_model.js

Index: chrome/browser/resources/shared/js/cr/ui/list_single_selection_model.js
diff --git a/chrome/browser/resources/shared/js/cr/ui/list_single_selection_model.js b/chrome/browser/resources/shared/js/cr/ui/list_single_selection_model.js
index f68818c140b47e69fef80f77a5630532d665538f..ff3c8c64a5616cc40c3092206b853c16983adc06 100644
--- a/chrome/browser/resources/shared/js/cr/ui/list_single_selection_model.js
+++ b/chrome/browser/resources/shared/js/cr/ui/list_single_selection_model.js
@@ -140,7 +140,6 @@ cr.define('cr.ui', function() {
         if (this.selectedIndexBefore_ != this.selectedIndex_) {
           var e = new Event('change');
           var indexes = [this.selectedIndexBefore_, this.selectedIndex_];
-          indexes.sort();

[arv (Not doing code reviews), 2011/01/11 21:52:45]

Why was this removed?

[stuartmorgan, 2011/01/11 23:18:41]

If the indexes are sorted, then the new row can be told that it's selected
before the old row is told that it's no longer selected, which goes against the
idea of having a selection model that enforces only one selection at a time.

It's unclear to me why it was there in the first place.

[arv (Not doing code reviews), 2011/01/11 23:29:29]

Probably just a remnant from the multiple selection model.

           e.changes = indexes.filter(function(index) {
             return index != -1;
           }).map(function(index) {