AXRoleDescriptions are not correctly exposed for dl, dd, dt tags

AXRoleDescriptions are not correctly exposed for dl, dd, dt tags

This is in accordance with latest editor's draft http://rawgit.com/w3c/html-api-map/master/index.html

dd:
MSAA+IA2 Role: IA2_ROLE_PARAGRAPH
AX:
AXRole: AXGroup
AXSubrole: AXDefinition
AXRoleDescription: "definition"

dl:
MSAA+IA2:
Role: ROLE_SYSTEM_LIST
States: STATE_SYSTEM_READONLY

AX:
AXRole: AXList
AXSubrole: AXDefinitionList
AXRoleDescription: "definition list"

dt:
MSAA+IA2:
Role: ROLE_SYSTEM_LISTITEM
States: STATE_SYSTEM_READONLY

AX:
AXRole: AXGroup
AXSubrole: AXTerm
AXRoleDescription: "term"

BUG=430795
Committed: https://crrev.com/16cf3bb5bddbe2bd18f4c13039504e3af6d6d8d6
Cr-Commit-Position: refs/heads/master@{#303601}

[shreeramk, 2014-11-06 07:14:49]

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

[shreeramk, 2014-11-06 07:14:50]

@Dominic & @Avi: PTAL. I need inputs on content_strings.grd changes. Thanks.

[dmazzoni, 2014-11-06 07:20:34]

lgtm

https://codereview.chromium.org/703123002/diff/20001/content/app/strings/cont...
File content/app/strings/content_strings.grd (right):

https://codereview.chromium.org/703123002/diff/20001/content/app/strings/cont...
content/app/strings/content_strings.grd:344: <message
name="IDS_AX_ROLE_DESCRIPTION_DETAIL" desc="accessibility role description for
description, definition or value (as in a description list)">
I would just say "accessibility role description for a definition"

https://codereview.chromium.org/703123002/diff/20001/content/app/strings/cont...
content/app/strings/content_strings.grd:348: defintion list
This should be description list?

[Avi (use Gerrit), 2014-11-06 07:21:42]

LGTM since dmazzoni is happy

[shreeramk, 2014-11-06 07:21:46]

https://codereview.chromium.org/703123002/diff/20001/content/app/strings/cont...
File content/app/strings/content_strings.grd (right):

https://codereview.chromium.org/703123002/diff/20001/content/app/strings/cont...
content/app/strings/content_strings.grd:348: defintion list
On 2014/11/06 07:20:34, dmazzoni wrote:
> This should be description list?

Spec says definition list.

[dmazzoni, 2014-11-06 07:28:29]

https://codereview.chromium.org/703123002/diff/20001/content/app/strings/cont...
File content/app/strings/content_strings.grd (right):

https://codereview.chromium.org/703123002/diff/20001/content/app/strings/cont...
content/app/strings/content_strings.grd:348: defintion list
On 2014/11/06 07:21:46, shreeramk wrote:
> On 2014/11/06 07:20:34, dmazzoni wrote:
> > This should be description list?
> 
> Spec says definition list.

It's misspelled, it says: defintion list

As before, I think the message description should match the text to be
translated, not the purpose of the string. The translator doesn't know the spec,
and having the description say "description list" is confusing.

It should be:

<message name="IDS_AX_ROLE_DESCRIPTION_DETAIL" desc="accessibility role
description for a definition of a term">
  definition
</message>

[shreeramk, 2014-11-06 07:43:01]

On 2014/11/06 07:28:29, dmazzoni wrote:
>
https://codereview.chromium.org/703123002/diff/20001/content/app/strings/cont...
> File content/app/strings/content_strings.grd (right):
> 
>
https://codereview.chromium.org/703123002/diff/20001/content/app/strings/cont...
> content/app/strings/content_strings.grd:348: defintion list
> On 2014/11/06 07:21:46, shreeramk wrote:
> > On 2014/11/06 07:20:34, dmazzoni wrote:
> > > This should be description list?
> > 
> > Spec says definition list.
> 
> It's misspelled, it says: defintion list

Fixed in this patchset 3. Sorry for the typo.

> As before, I think the message description should match the text to be
> translated, not the purpose of the string. The translator doesn't know the
spec,
> and having the description say "description list" is confusing.
> 

How this would be for "defintion list"?

> It should be:
> 
> <message name="IDS_AX_ROLE_DESCRIPTION_DETAIL" desc="accessibility role
> description for a definition of a term">
>   definition
> </message>

[shreeramk, 2014-11-06 07:45:36]

On 2014/11/06 07:43:01, shreeramk wrote:
> On 2014/11/06 07:28:29, dmazzoni wrote:
> >
>
https://codereview.chromium.org/703123002/diff/20001/content/app/strings/cont...
> > File content/app/strings/content_strings.grd (right):
> > 
> >
>
https://codereview.chromium.org/703123002/diff/20001/content/app/strings/cont...
> > content/app/strings/content_strings.grd:348: defintion list
> > On 2014/11/06 07:21:46, shreeramk wrote:
> > > On 2014/11/06 07:20:34, dmazzoni wrote:
> > > > This should be description list?
> > > 
> > > Spec says definition list.
> > 
> > It's misspelled, it says: defintion list
> 
> Fixed in this patchset 3. Sorry for the typo.
> 
> > As before, I think the message description should match the text to be
> > translated, not the purpose of the string. The translator doesn't know the
> spec,
> > and having the description say "description list" is confusing.
> > 
> 
> How this would be for "defintion list"?
> 
> > It should be:
> > 
> > <message name="IDS_AX_ROLE_DESCRIPTION_DETAIL" desc="accessibility role
> > description for a definition of a term">
> >   definition
> > </message>

Or you are saying this?

<message name="IDS_AX_ROLE_DESCRIPTION_LIST" desc="accessibility role
description for a definition of a term">
definition list
</message>

[shreeramk, 2014-11-10 18:29:04]

@Dominic: PTAL. Thanks!!

[dmazzoni, 2014-11-11 06:40:33]

The CQ bit was checked by dmazzoni@chromium.org

[dmazzoni, 2014-11-11 06:40:34]

lgtm

[commit-bot: I haz the power, 2014-11-11 06:41:02]

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

[commit-bot: I haz the power, 2014-11-11 07:43:17]

Committed patchset #4 (id:60001)

[commit-bot: I haz the power, 2014-11-11 07:44:30]

Patchset 4 (id:??) landed as
https://crrev.com/16cf3bb5bddbe2bd18f4c13039504e3af6d6d8d6
Cr-Commit-Position: refs/heads/master@{#303601}