Skip accessibility focus on tab in favor of close button

Skip accessibility focus on tab in favor of close button

The tab will always be checked first in the accessibility
order since its x coordinate is "left" of the close button.
However, visually the x button should be considered to be in
the foreground when using Explore by Touch.

With this change, the tab will return false when being checked
to be hit by explore by touch when we know the close button
will also be hit.

BUG=641141
Committed: https://crrev.com/24759a90d46ce86537be0b84241964f3ffaf6cc9
Cr-Commit-Position: refs/heads/master@{#416292}

[kraush, 2016-09-02 14:39:24]

kraush@amazon.com changed reviewers:
+ twellington@chromium.org

[kraush, 2016-09-02 14:39:25]

Hi Theresa

[kraush, 2016-09-02 14:40:08]

Argh, hit the wrong button with my first message!
Full message:

Hi Theresa,

Can you take a look at this change to restore the tab close button for Explore
by Touch?

Thanks! :)
Holger

[Theresa, 2016-09-02 16:43:32]

LTGM - thank you!

[Theresa, 2016-09-02 16:43:41]

*LGTM

[Theresa, 2016-09-02 16:43:55]

The CQ bit was checked by twellington@chromium.org

[commit-bot: I haz the power, 2016-09-02 16:44:11]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-09-02 17:31:07]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-09-02 17:34:51]

Description was changed from

==========
Skip accessibility focus on tab in favor of close button

The tab will always be checked first in the accessibility
order since its x coordinate is "left" of the close button.
However, visually the x button should be considered to be in
the foreground when using Explore by Touch.

With this change, the tab will return false when being checked
to be hit by explore by touch when we know the close button
will also be hit.

BUG=641141
==========

to

==========
Skip accessibility focus on tab in favor of close button

The tab will always be checked first in the accessibility
order since its x coordinate is "left" of the close button.
However, visually the x button should be considered to be in
the foreground when using Explore by Touch.

With this change, the tab will return false when being checked
to be hit by explore by touch when we know the close button
will also be hit.

BUG=641141

Committed: https://crrev.com/24759a90d46ce86537be0b84241964f3ffaf6cc9
Cr-Commit-Position: refs/heads/master@{#416292}
==========

[commit-bot: I haz the power, 2016-09-02 17:34:52]

Patchset 1 (id:??) landed as
https://crrev.com/24759a90d46ce86537be0b84241964f3ffaf6cc9
Cr-Commit-Position: refs/heads/master@{#416292}