Add AX role ,'AXDescriptionList', for dl element(re-land).

Add AX role ,'AXDescriptionList', for dl element(re-land).

There are related patches from WebKit, the patch written by
James Craig<james@cookiecrook.com>; wkrev.com/147520 and the
patch written by Mario Sanchez Prada  <mario.prada@samsung.com>;
wkrev.com/161492.
And I modified them based on the current blink.

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

[je_julie(Not used), 2014-09-26 12:03:35]

je_julie.kim@samsung.com changed reviewers:
+ dmazzoni@chromium.org

[je_julie(Not used), 2014-09-26 12:04:44]

Dominic, PTAL.

[dmazzoni, 2014-09-26 16:01:47]

The CQ bit was checked by dmazzoni@chromium.org

[dmazzoni, 2014-09-26 16:01:49]

lgtm

[commit-bot: I haz the power, 2014-09-26 16:02:35]

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

[commit-bot: I haz the power, 2014-09-26 17:06:22]

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

[commit-bot: I haz the power, 2014-09-26 17:06:23]

Try jobs failed on following builders:
  linux_blink_rel on tryserver.blink
(http://build.chromium.org/p/tryserver.blink/builders/linux_blink_rel/builds/2...)

[je_julie(Not used), 2014-09-27 04:18:38]

On 2014/09/26 17:06:23, I haz the power (commit-bot) wrote:
> Try jobs failed on following builders:
>   linux_blink_rel on tryserver.blink
>
(http://build.chromium.org/p/tryserver.blink/builders/linux_blink_rel/builds/2...)

Hi Dominic,
Thanks for reviewing.
In order to pass layout test for dl-role, we need patch for chromium,
https://codereview.chromium.org/607053002/,
because there is conversion for test-runner.
Normally, when we add some rule or attribute for AX,
we apply patch for blink and then apply patch for chromium.

For this case, my idea is below.
I'll separate layout test from this patch.
First, I'll apply implementation from blink here.
Second, I'll apply implementation from chromium,
https://codereview.chromium.org/607053002/.
And then I'll apply layout test for dl role.

How do you think?

[dmazzoni, 2014-09-29 05:47:11]

Sounds good. Please upload a new change without the test, and split the test
into a new changelist.

When Blink and Chromium are merged - hopefully in the next couple of months -
this should get easier.

[je_julie(Not used), 2014-09-29 06:18:06]

On 2014/09/29 05:47:11, dmazzoni wrote:
> Sounds good. Please upload a new change without the test, and split the test
> into a new changelist.
I uploaded a new change. PTAL.

> 
> When Blink and Chromium are merged - hopefully in the next couple of months -
> this should get easier.
Sounds great!

[dmazzoni, 2014-09-29 07:01:02]

The CQ bit was checked by dmazzoni@chromium.org

[dmazzoni, 2014-09-29 07:01:03]

lgtm

[commit-bot: I haz the power, 2014-09-29 07:02:28]

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

[commit-bot: I haz the power, 2014-09-29 09:26:19]

Committed patchset #2 (id:20001) as 182813

[eustas, 2014-09-29 15:29:33]

A revert of this CL (patchset #2 id:20001) has been created in
https://codereview.chromium.org/616553002/ by eustas@chromium.org.

The reason for reverting is: This patch, probably, broke AccesibilityDl test on
Mac
http://build.chromium.org/p/chromium.webkit/builders/Win7%20Tests/builds/5898.

[je_julie(Not used), 2014-09-30 00:07:41]

On 2014/09/29 15:29:33, eustas wrote:
> A revert of this CL (patchset #2 id:20001) has been created in
> https://codereview.chromium.org/616553002/ by mailto:eustas@chromium.org.
> 
> The reason for reverting is: This patch, probably, broke AccesibilityDl test
on
> Mac
> http://build.chromium.org/p/chromium.webkit/builders/Win7%20Tests/builds/5898.

Hi eustas,
I saw the log for content_browsertests at your revert CL.
But I have the related patch,https://codereview.chromium.org/607053002/, to
this.
When we add AX role, patch for blink is first and then patch for chromium.
There could be gap between blink and chromium when patch for blink is applied.
So far, I think it's avoidable because repository is different.
I checked already All Accesibility Tests are passed with blink patch and
chromium patch.

For this situation, is there any other suggestion?

[shreeramk, 2014-09-30 11:43:26]

On 2014/09/30 00:07:41, je_julie.kim wrote:
> On 2014/09/29 15:29:33, eustas wrote:
> > A revert of this CL (patchset #2 id:20001) has been created in
> > https://codereview.chromium.org/616553002/ by mailto:eustas@chromium.org.
> > 
> > The reason for reverting is: This patch, probably, broke AccesibilityDl test
> on
> > Mac
> >
http://build.chromium.org/p/chromium.webkit/builders/Win7%20Tests/builds/5898.
> 
> Hi eustas,
> I saw the log for content_browsertests at your revert CL.
> But I have the related patch,https://codereview.chromium.org/607053002/, to
> this.
> When we add AX role, patch for blink is first and then patch for chromium.
> There could be gap between blink and chromium when patch for blink is applied.
> So far, I think it's avoidable because repository is different.
> I checked already All Accesibility Tests are passed with blink patch and
> chromium patch.
> 
> For this situation, is there any other suggestion?

Hi jeongeun,
You could follow below process:

Steps:
1. Submit a CL with skipping that test for mac.(Let it land first then go to
step2)
2. Submit Blink Side Patch.
3. Submit chromium side patch and update the test that you skipped in step 1.

[je_julie(Not used), 2014-09-30 12:43:34]

On 2014/09/30 11:43:26, shreeram.k wrote:
> Hi jeongeun,
> You could follow below process:
> 
> Steps:
> 1. Submit a CL with skipping that test for mac.(Let it land first then go to
> step2)
> 2. Submit Blink Side Patch.
> 3. Submit chromium side patch and update the test that you skipped in step 1.

Hi Shreeram,
Thanks for your information.
Now, I'm preparing a CL for skipping TC.

[je_julie(Not used), 2014-10-01 13:14:29]

This patch is ready to be re-landed.
It's rebased on the latest cod without any actual change from the previous
patch.
The related TC is disabled at
https://code.google.com/p/chromium/issues/detail?id=418600.
According to reviewer's comment, I don't need any additional confirm and I'll
re-land it.

[je_julie(Not used), 2014-10-01 13:15:39]

The CQ bit was checked by je_julie.kim@samsung.com

[commit-bot: I haz the power, 2014-10-01 13:16:18]

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

[commit-bot: I haz the power, 2014-10-01 14:16:38]

Committed patchset #3 (id:40001) as 183062