Bug 119817

Summary:

AX: Integrate layout tests from overflow bounding boxes are way too big; prevents VoiceOver taps from activating the right elements

Product:

WebKit

Reporter:

chris fleizach <cfleizach>

Component:

Accessibility

Assignee:

chris fleizach <cfleizach>

Status:

RESOLVED FIXED

Severity:

Normal

CC:

aboxhall, apinheiro, commit-queue, dmazzoni, jdiggs, mario, webkit-bug-importer

Priority:

Keywords:

InRadar

Version:

528+ (Nightly build)

Hardware:

All

OS:

All

Attachments:

Description	Flags
patch	simon.fraser: review+

Description chris fleizach 2013-08-14 14:29:35 PDT

When there's a serious of transforms applied, the bounding box of elements can be wrong for accessibility.

This bug is to integrate layouts tests to ensure this does not happen

<rdar://problem/14700616>

Comment 1 chris fleizach 2013-08-14 14:33:28 PDT

Created attachment 208761 [details]
patch

Comment 2 chris fleizach 2013-08-14 14:34:42 PDT

This problem was introduced by
http://trac.webkit.org/changeset/144350

and partially rolled out with
http://trac.webkit.org/changeset/153598

Comment 3 Simon Fraser (smfr) 2013-08-14 15:39:34 PDT

Comment on attachment 208761 [details]
patch

View in context: https://bugs.webkit.org/attachment.cgi?id=208761&action=review

> LayoutTests/accessibility/transformed-bounds.html:1
> +<!DOCTYPE HTML PUBLIC "-//IETF//DTD HTML//EN">

<!DOCTYPE html> will do

> LayoutTests/accessibility/transformed-bounds.html:15
> +        
> +        .pills .button{-webkit-appearance:none;display:inline-block;vertical-align:top;text-align:center;width:100%}
> +        .pills .label{display:block;font-weight:bold;white-space:nowrap;text-overflow:ellipsis;overflow:hidden}
> +        .ipad .top-section{text-align:center}
> +        .pills{display:inline-block;height:29px;margin:0 auto 10px;padding:0 15px;min-width:292px}
> +        .pills div{display:inline-block;text-align:center}
> +        .swoosh-overlay .contents .row .item:not(.software) .card-container{-webkit-transform-origin-x:315px}
> +        .swoosh-overlay .contents .row .item .card-container{-webkit-transform-style:preserve-3d;position:absolute}
> +        .swoosh-overlay .item .card-back{position:absolute;z-index:200;width:630px;height:630px;-webkit-backface-visibility:hidden;-webkit-transform:rotatey(180deg)}

Please expand these. I doubt that all the styles are necessary for the testcase; can you simplify this more?

Comment 4 chris fleizach 2013-08-14 16:58:22 PDT

http://trac.webkit.org/changeset/154082