AX: Make the calendar role "grid".

AX: Make the calendar role "grid".

Rows have role="row", and cells have role="girdcell".

We apply aria-activedescendant to the calendar because the calendar cell is not
focusable. Thus, each of cells needs to have id=.  Also, cells should have
descriptive text as aria-label.

- getLocale() needs to replace _ with - because Intl.DateTimeFormat doesn't
  accept _.

- This CL doesn't improve the calendar picker for type=month and type=week, and
  won't be harmful for them.

BUG=123896
Committed: https://src.chromium.org/viewvc/blink?view=rev&revision=181462

[tkent, 2014-09-03 09:32:06]

tkent@chromium.org changed reviewers:
+ keishi@chromium.org

[tkent, 2014-09-03 09:32:06]

Please review this.

This CL still has a bug that we don't have a11y events for initial value
"2000-01-02".  I'll address it in another CL.

https://codereview.chromium.org/536903002/diff/1/LayoutTests/fast/forms/resou...
File LayoutTests/fast/forms/resources/picker-common.js (right):

https://codereview.chromium.org/536903002/diff/1/LayoutTests/fast/forms/resou...
LayoutTests/fast/forms/resources/picker-common.js:25:
setTimeout(popupOpenCallback, 0);
This is necessary to avoid too-early call of popupOpenCallback in the new test
though I don't know the exact reason.

[keishi, 2014-09-03 12:09:35]

LGTM

https://codereview.chromium.org/536903002/diff/1/Source/web/resources/calenda...
File Source/web/resources/calendarPicker.js (right):

https://codereview.chromium.org/536903002/diff/1/Source/web/resources/calenda...
Source/web/resources/calendarPicker.js:78: return (global.params.locale ||
"en-us").toLowerCase().replace(/_/, '-');
nit: I'm not sure but could the locale have two underscores like zh_yue_HK?

[tkent, 2014-09-05 04:54:47]

https://codereview.chromium.org/536903002/diff/1/Source/web/resources/calenda...
File Source/web/resources/calendarPicker.js (right):

https://codereview.chromium.org/536903002/diff/1/Source/web/resources/calenda...
Source/web/resources/calendarPicker.js:78: return (global.params.locale ||
"en-us").toLowerCase().replace(/_/, '-');
On 2014/09/03 12:09:35, keishi wrote:
> nit: I'm not sure but could the locale have two underscores like zh_yue_HK?

That's right.  The regex should be /_/g

[tkent, 2014-09-05 08:41:17]

Patchset #2 (id:20001) has been deleted

[tkent, 2014-09-05 10:00:46]

The CQ bit was checked by tkent@chromium.org

[commit-bot: I haz the power, 2014-09-05 10:01:15]

CQ is trying da patch. Follow status at
 https://chromium-status.appspot.com/cq/tkent@chromium.org/536903002/40001

[commit-bot: I haz the power, 2014-09-05 10:05:33]

Committed patchset #2 (id:40001) as 181462