xml-roles for input types {radio, range, button}, math

xml-roles for input types {radio, range, button}, math

According to latest editor draft(http://rawgit.com/w3c/html-api-map/master/index.html) above mentioned input type attributes should have xml-roles. Also math tag should have xml-role.

BUG=418486
Committed: https://crrev.com/bbbdcec8301e5ff9b8273e3e27ff7426d9dacfa7
Cr-Commit-Position: refs/heads/master@{#297817}

[shreeramk, 2014-09-29 15:32:03]

shreeram.k@samsung.com changed reviewers:
+ dmazzoni@chromium.org

[shreeramk, 2014-09-29 15:32:04]

Please review.

I ll update the failing win test tomorrow.

Thanks!

[shreeramk, 2014-09-29 15:34:05]

On 2014/09/29 15:32:04, shreeram.k wrote:
> Please review.
> 
> I ll update the failing win test tomorrow.
> 
> Thanks!

Sorry. I forgot to add input-radio.html in dump accessibility.

I ll update that too tomorrow.

[shreeramk, 2014-10-01 05:44:07]

On 2014/09/29 15:34:05, shreeram.k wrote:
> On 2014/09/29 15:32:04, shreeram.k wrote:
> > Please review.
> > 
> > I ll update the failing win test tomorrow.
> > 
> > Thanks!
> 
> Sorry. I forgot to add input-radio.html in dump accessibility.
> 
> I ll update that too tomorrow.

Due to network issue with my linux machine. I am not able to update this CL.
I ll update this as soon as network issue is fixed.

[shreeramk, 2014-10-01 08:10:13]

Updated the CL.

Please review. Thanks!

[shreeramk, 2014-10-01 18:37:33]

shreeram.k@samsung.com changed reviewers:
+ mkwst@chromium.org

[Mike West, 2014-10-01 18:46:28]

1. There's no test for the math role you added.
2. Can you point to the relevant section of the relevant spec in the CL
description?

[dmazzoni, 2014-10-01 19:09:07]

On 2014/10/01 18:46:28, Mike West wrote:
> 1. There's no test for the math role you added.

Sadly it's impossible to add tests for new roles until you first land the new
role in Blink, then add support for it in test_runner (in Chrome code), then go
back and add a test in Blink. It's maddening.

[dmazzoni, 2014-10-01 19:36:39]

The CQ bit was checked by dmazzoni@chromium.org

[dmazzoni, 2014-10-01 19:36:39]

lgtm

[commit-bot: I haz the power, 2014-10-01 19:38:06]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/609343002/80001

[commit-bot: I haz the power, 2014-10-01 21:47:23]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2014-10-01 21:47:24]

Try jobs failed on following builders:
  win_chromium_rel_swarming on tryserver.chromium.win
(http://build.chromium.org/p/tryserver.chromium.win/builders/win_chromium_rel_...)

[shreeramk, 2014-10-02 06:35:08]

The CQ bit was checked by shreeram.k@samsung.com

[commit-bot: I haz the power, 2014-10-02 06:35:23]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/609343002/80001

[commit-bot: I haz the power, 2014-10-02 07:01:45]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2014-10-02 07:01:46]

Try jobs failed on following builders:
  win_chromium_rel_swarming on tryserver.chromium.win
(http://build.chromium.org/p/tryserver.chromium.win/builders/win_chromium_rel_...)

[shreeramk, 2014-10-02 11:43:59]

The CQ bit was checked by shreeram.k@samsung.com

[commit-bot: I haz the power, 2014-10-02 11:45:11]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/609343002/80001

[shreeramk, 2014-10-02 12:26:18]

On 2014/10/01 18:46:28, Mike West wrote:
> 1. There's no test for the math role you added.

Ok. I would add a test for math tag while fixing Adding DumpAccessibilityTree
Tests(4-20) bugs.

> 2. Can you point to the relevant section of the relevant spec in the CL
> description?

This is the latest editor draft -
http://rawgit.com/w3c/html-api-map/master/index.html

[Mike West, 2014-10-02 12:28:58]

On 2014/10/02 12:26:18, shreeram.k wrote:
> On 2014/10/01 18:46:28, Mike West wrote:
> > 1. There's no test for the math role you added.
> 
> Ok. I would add a test for math tag while fixing Adding DumpAccessibilityTree
> Tests(4-20) bugs.

Ok. LGTM.
 
> > 2. Can you point to the relevant section of the relevant spec in the CL
> > description?
> 
> This is the latest editor draft -
> http://rawgit.com/w3c/html-api-map/master/index.html

Yes. Please add that to the CL description instead of "According to specs". :)

[commit-bot: I haz the power, 2014-10-02 12:50:57]

Committed patchset #5 (id:80001) as bfdbfcd83191122231e0e7b8d5a7de56c32dbdde

[commit-bot: I haz the power, 2014-10-02 12:51:35]

Patchset 5 (id:??) landed as
https://crrev.com/bbbdcec8301e5ff9b8273e3e27ff7426d9dacfa7
Cr-Commit-Position: refs/heads/master@{#297817}