Makes AXNodeObject test if it's a table cell or row before calling isPresentationalInTable.

Makes AXNodeObject test if it's a table cell or row before calling isPresentationalInTable.

The API, isPresentationalInTable(), is to see whether the current node is
a table cell or a table row and its parent has a presentational role.
If the current node is not a table cell or a table row, we don't need to
call it.

This patch adds conditions at inheritsPresentationalRoleFrom() to test
whether it is a table-cell or a table-row.

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

[je_julie(Not used), 2015-05-04 15:52:41]

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

[je_julie(Not used), 2015-05-04 15:52:42]

Hi Dominic,

This patch makes AXNodeObject test if it's a cell or row of table before calling
isPresentationalInTable() because it's  for checking presentational Role only in
Table.

PTAL.

[dmazzoni, 2015-05-04 16:00:05]

lgtm

I'm assuming this is a speculative fix for the isPresentationalInTable crash?
Please add the bug number to this changelist even if you're not 100% sure it's
related, or file a new bug.

[je_julie(Not used), 2015-05-05 02:45:45]

On 2015/05/04 16:00:05, dmazzoni wrote:
> lgtm
> 
> I'm assuming this is a speculative fix for the isPresentationalInTable crash?
> Please add the bug number to this changelist even if you're not 100% sure it's
> related, or file a new bug.

You're right. I filed a new bug, 484501.
If the crash is fixed with this patch, I'll add relationship to the crash issue.
Thanks,

[je_julie(Not used), 2015-05-05 02:47:07]

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

[commit-bot: I haz the power, 2015-05-05 02:47:45]

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

[commit-bot: I haz the power, 2015-05-05 05:09:12]

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