Accessible name not calculated properly

Accessible name not calculated properly

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

[ramya.v, 2015-01-02 14:20:43]

ramya.v@samsung.com changed reviewers:
+ aboxhall@chromium.org, dmazzoni@chromium.org

[ramya.v, 2015-01-02 14:20:43]

PTAL!

Thanks

[ramya.v, 2015-01-27 03:06:14]

On 2015/01/02 14:20:43, ramya.v wrote:
> PTAL!
> 
> Thanks

@dominic Could you please review the patch?

Thanks!

[dmazzoni, 2015-01-27 06:40:46]

lgtm

https://codereview.chromium.org/836553002/diff/1/Source/modules/accessibility...
File Source/modules/accessibility/AXNodeObject.cpp (right):

https://codereview.chromium.org/836553002/diff/1/Source/modules/accessibility...
Source/modules/accessibility/AXNodeObject.cpp:96: if (node->isHTMLElement()) {
Just combine this into the block above:

if (node->isHTMLElement()) {
    const AtomicString& alt = toHTMLElement(node)->getAttribute(altAttr);
    if (!alt.isEmpty())
        return alt;

    const AtomicString& title = toHTMLElement(node)->getAttribute(titleAttr);
    if (!title.isEmpty())
        return title;
}

[ramya.v, 2015-01-27 10:16:09]

The CQ bit was checked by ramya.v@samsung.com

[commit-bot: I haz the power, 2015-01-27 10:16:22]

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

[commit-bot: I haz the power, 2015-01-27 12:20:01]

Committed patchset #2 (id:20001) as
https://src.chromium.org/viewvc/blink?view=rev&revision=189028