Fixed issue with miscalculated aria-posinset attribute's value.

Fixed issue with miscalculated aria-posinset attribute's value.

This bug was caused by returning a string as an index from list_selection_model.js in an Event argument of a change event. In the list.js, such index stored as string was incremented, which caused concatenation instead of incrementation. This patch converts these values to number to avoid such problems.

TEST=Enable ChromeVox, enter Files.app (CTRL-M), navigate among files. Correct item number should be spoken, eg. 2/x, 3/x.
BUG=164061
Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=171056

[mtomasz, 2012-12-04 01:56:01]

@arv: PTAL.

[arv (Not doing code reviews), 2012-12-04 15:33:36]

LGTM

https://codereview.chromium.org/11421214/diff/1/chrome/browser/resources/shar...
File chrome/browser/resources/shared/js/cr/ui/list_selection_model.js (right):

https://codereview.chromium.org/11421214/diff/1/chrome/browser/resources/shar...
chrome/browser/resources/shared/js/cr/ui/list_selection_model.js:230: index:
index,
or just do the conversion here

index: Number(index),

That way can skip one array allocation

[mtomasz, 2012-12-04 18:53:09]

https://codereview.chromium.org/11421214/diff/1/chrome/browser/resources/shar...
File chrome/browser/resources/shared/js/cr/ui/list_selection_model.js (right):

https://codereview.chromium.org/11421214/diff/1/chrome/browser/resources/shar...
chrome/browser/resources/shared/js/cr/ui/list_selection_model.js:230: index:
index,
On 2012/12/04 15:33:36, arv wrote:
> or just do the conversion here
> 
> index: Number(index),
> 
> That way can skip one array allocation

Done.

[commit-bot: I haz the power, 2012-12-04 18:53:22]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/mtomasz@chromium.org/11421214/1002

[commit-bot: I haz the power, 2012-12-04 21:51:04]

Change committed as 171056