Closed Bug 756969 Opened 12 years ago Closed 12 years ago

don't calculate rendering text when calculating visibility state

Categories

(Core :: Disability Access APIs, defect)

defect
Not set
normal

Tracking

()

RESOLVED WONTFIX

People

(Reporter: surkov, Assigned: surkov)

References

Details

(Keywords: access)

Attachments

(1 file, 1 obsolete file)

Attached patch patch (obsolete) — Splinter Review
we spend about 1% for that per test case of bug 732872 but this doesn't make sense at all since we don't create accessible having no rendered text and we keep its rendered text cached (so in the case when frame doesn't have rendered text anymore but we didn't destroyed an accessible yet then it's considered that accessible still has a rendered text).
Attachment #625566 - Flags: review?(trev.saunders)
Attached patch patch2Splinter Review
correct patch
Attachment #625566 - Attachment is obsolete: true
Attachment #625566 - Flags: review?(trev.saunders)
Attachment #625567 - Flags: review?(trev.saunders)
Comment on attachment 625567 [details] [diff] [review]
patch2

on the another hand it makes us out of sync with accessible boundaries which are empty in this case so we should expose invisible state for correctness
Attachment #625567 - Flags: review?(trev.saunders)
wontfix per comment #2
Status: ASSIGNED → RESOLVED
Closed: 12 years ago
Resolution: --- → WONTFIX
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: