1792964 - Hittesting table rows is incorrect when caching

Status: RESOLVED FIXED

(Core :: Disability Access APIs, defect)

Description

[Morgan Reschenberg [:morgan]]

See summary at bug 1792603
Filing a separate bug here because it's possible we can work around this in the meantime with a hack similar to what we're doing for states::OFFSCREEN

Comment 1

[James Teh [:Jamie]]

Morgan, am I correct that your latest offscreen patch deals with this, since it adds the row to the viewport cache? Can we close this as a dup?

Comment 2

[Morgan Reschenberg [:morgan]]

Yes, but I think it's worth adding a test (we have a offscreen test for tables but not a hittesting one). I can do that.

Comment 3

[Morgan Reschenberg [:morgan]]

Attachment: Bug 1792964: Add hittesting test for tables, table rows, table cells r?eeejay

Comment 4

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/f942a92c1666
Add hittesting test for tables, table rows, table cells r=eeejay

Comment 5

[Cosmin Sabou [:CosminS]]

Backed out for causing mochitest failures on browser_test_visibility.js.

Push with failures: https://treeherder.mozilla.org/jobs?repo=autoland&group_state=expanded&resultStatus=pending%2Crunning%2Csuccess%2Ctestfailed%2Cbusted%2Cexception%2Cretry%2Cusercancel&revision=f9ec55a441c4554b04137a5420cbf288a920a9f7&test_paths=accessible%2Ftests%2Fbrowser%2Fstates%2F&selectedTaskRun=dqTLx2vwSMCMfWK5-bX4XQ.0

Failure log: https://treeherder.mozilla.org/logviewer?job_id=393558771&repo=autoland

Backout link: https://hg.mozilla.org/integration/autoland/rev/a99882b97fbc5154a3bbf7c06d59fc159922f7c5

Comment 6

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/94e6e52575c6
Add hittesting test for tables, table rows, table cells r=eeejay

Comment 7

[Sandor Molnar[:smolnar]]

https://hg.mozilla.org/mozilla-central/rev/94e6e52575c6