1688730 - Layout table status should be cached, invalidated by attr changes on mac

Status: RESOLVED FIXED

(Core :: Disability Access APIs, defect, P1)

Description

[Morgan Reschenberg [:morgan]]

Spinoff from bug 1680840, I'll do the new event and style change notifications there, but will do the core of the mac caching and attr-changed events here so we can land, uplift with less risk

Comment 1

[Morgan Reschenberg [:morgan]]

Attachment: Bug 1688730: Introduce caching for IsLayoutTable r?eeejay

Comment 2

[Morgan Reschenberg [:morgan]]

Attachment: Bug 1688730: Test table layout caching r?eeejay

Depends on D101512

Comment 3

[Morgan Reschenberg [:morgan]]

Attachment: Bug 1688730: Fire relevant object attr changed events when layout table status may have changed r?eeejay

Comment 4

[Morgan Reschenberg [:morgan]]

Attachment: Bug 1688730: Cache table layout status on mac, invalidate on reorder, attr changed r?eeejay

Depends on D102993

Comment 5

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/dd03939b850e
Fire relevant object attr changed events when layout table status may have changed r=eeejay
https://hg.mozilla.org/integration/autoland/rev/bd6d78d36cee
Cache table layout status on mac, invalidate on reorder, attr changed r=eeejay

Comment 6

[Atila Butkovits]

Backed out for causing build bustage on mozTableAccessible.mm.

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

Push with failures: uhttps://treeherder.mozilla.org/jobs?repo=autoland&selectedTaskRun=HT_mlW5WSvm9lM1mMH45lg.0&searchStr=os%2Cx%2Ccross%2Ccompiled%2Cdebug%2Cbuild-macosx64%2Fdebug%2Cb&revision=bd6d78d36ceeac36b814b3712257fd776931727a

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

Comment 7

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/a7d0770117c8
Fire relevant object attr changed events when layout table status may have changed r=eeejay
https://hg.mozilla.org/integration/autoland/rev/677c17ec48ea
Cache table layout status on mac, invalidate on reorder, attr changed r=eeejay

Comment 8

[Atila Butkovits]

Backed out for causing mochitest failures.

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

Push with failures: https://treeherder.mozilla.org/jobs?repo=autoland&selectedTaskRun=Ke6NfeBLSq-9owjSQPMo_Q.0&searchStr=os%2Cx%2C10.14%2Cwebrender%2Cdebug%2Cmochitests%2Cwithout%2Ce10s%2Ctest-macosx1014-64-qr%2Fdebug-mochitest-a11y-1proc%2Ca11y&revision=677c17ec48ea9f3ae2fbff91061632386090a91c

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

Comment 9

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/e4cbaa21e880
Fire relevant object attr changed events when layout table status may have changed r=eeejay
https://hg.mozilla.org/integration/autoland/rev/0705a29070a8
Cache table layout status on mac, invalidate on reorder, attr changed r=eeejay

Comment 10

[Dorel Luca [:dluca]]

https://hg.mozilla.org/mozilla-central/rev/e4cbaa21e880
https://hg.mozilla.org/mozilla-central/rev/0705a29070a8