AXRoleDescription is not exposed correctly for aside, main, nav element

AXRoleDescription is not exposed correctly for aside, main, nav element

According to latest editor's draft http://rawgit.com/w3c/html-api-map/master/index.html

    HTML Element        AXRoleDescription
    aside               complementary
    main                main
    nav                 navigation

    BUG=281895
Committed: https://crrev.com/e55306defdc23e95b7d3cf91c8d344dbc31c9f61
Cr-Commit-Position: refs/heads/master@{#302991}

[shreeramk, 2014-11-05 16:05:37]

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

[shreeramk, 2014-11-05 16:05:38]

@Dominic: PTAL. Thanks

@Avi & @Dominic: Please give your review points for the changes in
content_strings.grd

[dmazzoni, 2014-11-05 16:20:28]

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

https://codereview.chromium.org/682833003/diff/60001/content/app/strings/cont...
content/app/strings/content_strings.grd:342: complementary
The text of the string to be translated should match its description

https://codereview.chromium.org/682833003/diff/60001/content/app/strings/cont...
content/app/strings/content_strings.grd:382: <message
name="IDS_AX_ROLE_NAVIGATIONAL_LINK" desc="accessibility role description for
group of navigational links.">
This is a good description - it provides helpful context for the translator

[Avi (use Gerrit), 2014-11-05 16:42:25]

LGTM for when dmazzoni is happy

[shreeramk, 2014-11-05 19:10:44]

On 2014/11/05 16:20:28, dmazzoni wrote:
>
https://codereview.chromium.org/682833003/diff/60001/content/app/strings/cont...
> File content/app/strings/content_strings.grd (right):
> 
>
https://codereview.chromium.org/682833003/diff/60001/content/app/strings/cont...
> content/app/strings/content_strings.grd:342: complementary
> The text of the string to be translated should match its description

@Dominic
Fixed in patchset 5.

PTAL. Thanks

[dmazzoni, 2014-11-06 07:11:52]

lgtm

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

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

[commit-bot: I haz the power, 2014-11-06 07:47:34]

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

[commit-bot: I haz the power, 2014-11-06 09:03:09]

Committed patchset #5 (id:80001)

[commit-bot: I haz the power, 2014-11-06 09:03:44]

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