Fixing bug in updating file list in save-as dialog.

Fixing bug in updating file list in save-as dialog.

Problem occuired because file manager code uses the selectedIndexes property of selectionModel. But setter for this methos is implemented only for multi-selection list, but not fo single selection (which is used in the save-as dialog). It fails to restore selection and consequently fails to restore lead item. Resetting lead item stops renaming.

BUG=138975
TEST=Manual test


Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=151200

[Oleg Eterevsky, 2012-08-07 14:55:25]

lgtm

[arv (Not doing code reviews), 2012-08-07 17:10:30]

lgtm

https://chromiumcodereview.appspot.com/10823201/diff/1/chrome/browser/resourc...
File chrome/browser/resources/shared/js/cr/ui/list_single_selection_model.js
(right):

https://chromiumcodereview.appspot.com/10823201/diff/1/chrome/browser/resourc...
chrome/browser/resources/shared/js/cr/ui/list_single_selection_model.js:44: set
selectedIndexes(value) {
s/value/indexes/

[SeRya, 2012-08-09 17:37:02]

http://codereview.chromium.org/10823201/diff/1/chrome/browser/resources/share...
File chrome/browser/resources/shared/js/cr/ui/list_single_selection_model.js
(right):

http://codereview.chromium.org/10823201/diff/1/chrome/browser/resources/share...
chrome/browser/resources/shared/js/cr/ui/list_single_selection_model.js:44: set
selectedIndexes(value) {
On 2012/08/07 17:10:30, arv wrote:
> s/value/indexes/

Done.

[commit-bot: I haz the power, 2012-08-09 17:37:13]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/serya@chromium.org/10823201/6001

[commit-bot: I haz the power, 2012-08-09 18:46:55]

Try job failure for 10823201-6001 (retry) on mac_rel for step "browser_tests".
It's a second try, previously, step "browser_tests" failed.
http://build.chromium.org/p/tryserver.chromium/buildstatus?builder=mac_rel&nu...

[commit-bot: I haz the power, 2012-08-12 10:33:00]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/serya@chromium.org/10823201/6001

[commit-bot: I haz the power, 2012-08-12 11:50:14]

Try job failure for 10823201-6001 (retry) on linux_rel for step
"interactive_ui_tests".
It's a second try, previously, step "interactive_ui_tests" failed.
http://build.chromium.org/p/tryserver.chromium/buildstatus?builder=linux_rel&...

[commit-bot: I haz the power, 2012-08-12 13:38:23]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/serya@chromium.org/10823201/6001

[commit-bot: I haz the power, 2012-08-12 14:53:29]

Change committed as 151200