1748749 - Cache ActionCount/ActionNameAt

Status: RESOLVED FIXED

(Core :: Disability Access APIs, task)

Description

[Eitan Isaacson [:eeejay]]

This is needed in Android/ctw for parent-size pivot navigation.

Comment 1

[Eitan Isaacson [:eeejay]]

Attachment: Bug 1748749 - P1: Push action name to cache. r=Jamie

We use an atom instead of a string because it a limited corpus of
strings so we don't need unique instances.

Comment 2

[Eitan Isaacson [:eeejay]]

Attachment: Bug 1748749 - P2: Cache has long desc. r=Jamie

We only need to cache the existance of a longdesc in an image.
This increments the action count.

Depends on D135905

Comment 3

[Eitan Isaacson [:eeejay]]

Attachment: Bug 1748749 - P3: Move TranslateString to Accessible. r=Jamie

Small refactor to allow action descriptions to be implemented in
Accessible.

Depends on D135906

Comment 4

[Eitan Isaacson [:eeejay]]

Attachment: Bug 1748749 - P4: Support cached actions. r=Jamie

Move ActionCount/ActionNameAt/ActionDescriptionAt to Accessible,
use cached values in RemoteAccessible, and tweak platforms.

Also introduce browser test.

Depends on D135907

Comment 5

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/f1be79d75c5d
P1: Push action name to cache. r=Jamie
https://hg.mozilla.org/integration/autoland/rev/9dccfcc00718
P2: Cache has long desc. r=Jamie
https://hg.mozilla.org/integration/autoland/rev/9d405f9475b3
P3: Move TranslateString to Accessible. r=Jamie
https://hg.mozilla.org/integration/autoland/rev/fce9d3a2500d
P4: Support cached actions. r=Jamie

Comment 6

[Atila Butkovits]

Backed out for causing failures at browser_caching_actions.js.

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

Push with failures: https://treeherder.mozilla.org/jobs?repo=autoland&selectedTaskRun=ctTzDisKS1uVJoVGqct9oA.0&resultStatus=testfailed%2Cbusted%2Cexception%2Cretry%2Cusercancel&revision=6f15d287ecd884d588104dc4789fc80c19b43da4

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

Comment 7

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/439a3322ec38
P1: Push action name to cache. r=Jamie
https://hg.mozilla.org/integration/autoland/rev/35dbb5c0c9bc
P2: Cache has long desc. r=Jamie
https://hg.mozilla.org/integration/autoland/rev/720cf9840dbf
P3: Move TranslateString to Accessible. r=Jamie
https://hg.mozilla.org/integration/autoland/rev/8e45fa284a74
P4: Support cached actions. r=Jamie

Comment 8

[Iulian Moraru]

Backed out (bug 1748749, bug 1743967, bug 1749828) for causing build bustages on RotorRules.mm.

Push with failures

Failure log

Backout link

[task 2022-01-28T22:28:05.335Z] 22:28:05     INFO -  gmake[4]: Entering directory '/builds/worker/workspace/obj-build/accessible/mac'
[task 2022-01-28T22:28:05.335Z] 22:28:05     INFO -  accessible/mac/Unified_mm_accessible_mac0.o
[task 2022-01-28T22:28:05.338Z] 22:28:05     INFO -  /builds/worker/fetches/sccache/sccache /builds/worker/fetches/clang/bin/clang++ --sysroot /builds/worker/fetches/MacOSX10.12.sdk -mmacosx-version-min=10.12 -stdlib=libc++ -std=gnu++17 --target=x86_64-apple-darwin -o Unified_mm_accessible_mac0.o -c  -fvisibility=hidden -fvisibility-inlines-hidden -U_FORTIFY_SOURCE -D_FORTIFY_SOURCE=2 -fstack-protector-strong -DNDEBUG=1 -DTRIMMED=1 -DOS_POSIX=1 -DOS_MACOSX=1 -DMOZ_HAS_MOZGLUE -DMOZILLA_INTERNAL_API -DIMPL_LIBXUL -DSTATIC_EXPORTABLE_JS_API -I/builds/worker/checkouts/gecko/accessible/mac -I/builds/worker/workspace/obj-build/accessible/mac -I/builds/worker/checkouts/gecko/accessible/base -I/builds/worker/checkouts/gecko/accessible/generic -I/builds/worker/checkouts/gecko/accessible/html -I/builds/worker/checkouts/gecko/accessible/ipc -I/builds/worker/checkouts/gecko/accessible/ipc/other -I/builds/worker/checkouts/gecko/accessible/xul -I/builds/worker/checkouts/gecko/layout/generic -I/builds/worker/checkouts/gecko/layout/xul -I/builds/worker/checkouts/gecko/widget -I/builds/worker/checkouts/gecko/widget/cocoa -I/builds/worker/workspace/obj-build/ipc/ipdl/_ipdlheaders -I/builds/worker/checkouts/gecko/ipc/chromium/src -I/builds/worker/workspace/obj-build/dist/include -I/builds/worker/workspace/obj-build/dist/include/nspr -I/builds/worker/workspace/obj-build/dist/include/nss -DMOZILLA_CLIENT -include /builds/worker/workspace/obj-build/mozilla-config.h -Qunused-arguments -Qunused-arguments -Wall -Wbitfield-enum-conversion -Wdeprecated-this-capture -Wempty-body -Wformat-type-confusion -Wignored-qualifiers -Wpointer-arith -Wshadow-field-in-constructor-modified -Wsign-compare -Wtype-limits -Wno-error=tautological-type-limit-compare -Wunreachable-code -Wunreachable-code-return -Wunused-but-set-parameter -Wno-invalid-offsetof -Wclass-varargs -Wempty-init-stmt -Wfloat-overflow-conversion -Wfloat-zero-conversion -Wloop-analysis -Wno-range-loop-analysis -Wc++2a-compat -Wcomma -Wenum-compare-conditional -Wimplicit-fallthrough -Werror=non-literal-null-conversion -Wstring-conversion -Wno-inline-new-delete -Wno-error=deprecated-declarations -Wno-error=array-bounds -Wno-error=backend-plugin -Wno-error=free-nonheap-object -Wno-error=return-std-move -Wno-error=atomic-alignment -Wno-error=deprecated-copy -Wformat -Wformat-security -Wno-gnu-zero-variadic-macro-arguments -Wno-psabi -Wno-unknown-warning-option -fno-sized-deallocation -fno-aligned-new -fcrash-diagnostics-dir=/builds/worker/artifacts -fno-exceptions -fPIC -fno-rtti -ffunction-sections -fdata-sections -fno-exceptions -fno-math-errno -pthread -pipe -g -Xclang -load -Xclang /builds/worker/workspace/obj-build/build/clang-plugin/libclang-plugin.so -Xclang -add-plugin -Xclang moz-check -O3 -fno-omit-frame-pointer -funwind-tables -Werror -fno-strict-aliasing  -MD -MP -MF .deps/Unified_mm_accessible_mac0.o.pp   -x objective-c++ -fobjc-exceptions   Unified_mm_accessible_mac0.mm
[task 2022-01-28T22:28:05.338Z] 22:28:05     INFO -  In file included from Unified_mm_accessible_mac0.mm:119:
[task 2022-01-28T22:28:05.339Z] 22:28:05    ERROR -  /builds/worker/checkouts/gecko/accessible/mac/RotorRules.mm:317:14: error: unused variable 'groupPos' [-Werror,-Wunused-variable]
[task 2022-01-28T22:28:05.339Z] 22:28:05     INFO -      GroupPos groupPos = aAcc->GroupPosition();
[task 2022-01-28T22:28:05.339Z] 22:28:05     INFO -               ^
[task 2022-01-28T22:28:05.339Z] 22:28:05     INFO -  1 error generated.
[task 2022-01-28T22:28:05.339Z] 22:28:05    ERROR -  gmake[4]: *** [/builds/worker/checkouts/gecko/config/rules.mk:668: Unified_mm_accessible_mac0.o] Error 1
[task 2022-01-28T22:28:05.340Z] 22:28:05     INFO -  gmake[4]: Leaving directory '/builds/worker/workspace/obj-build/accessible/mac'
[task 2022-01-28T22:28:05.340Z] 22:28:05    ERROR -  gmake[3]: *** [/builds/worker/checkouts/gecko/config/recurse.mk:72: accessible/mac/target-objects] Error 2
[task 2022-01-28T22:28:05.340Z] 22:28:05     INFO -  gmake[3]: *** Waiting for unfinished jobs....
[task 2022-01-28T22:28:05.340Z] 22:28:05     INFO -  gmake[4]: Entering directory '/builds/worker/workspace/obj-build/config/external/icu/i18n'

Comment 9

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/8e23acc969d2
P1: Push action name to cache. r=Jamie
https://hg.mozilla.org/integration/autoland/rev/4190c5e44b32
P2: Cache has long desc. r=Jamie
https://hg.mozilla.org/integration/autoland/rev/cd2e76de3d18
P3: Move TranslateString to Accessible. r=Jamie
https://hg.mozilla.org/integration/autoland/rev/04f178d71ae6
P4: Support cached actions. r=Jamie

Comment 10

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/18eebd367184
P1: Push action name to cache. r=Jamie
https://hg.mozilla.org/integration/autoland/rev/1e7eec5576e5
P2: Cache has long desc. r=Jamie
https://hg.mozilla.org/integration/autoland/rev/84936bb27997
P3: Move TranslateString to Accessible. r=Jamie
https://hg.mozilla.org/integration/autoland/rev/8306a663c330
P4: Support cached actions. r=Jamie

Comment 11

[Norisz Fay [:noriszfay]]

https://hg.mozilla.org/mozilla-central/rev/18eebd367184
https://hg.mozilla.org/mozilla-central/rev/1e7eec5576e5
https://hg.mozilla.org/mozilla-central/rev/84936bb27997
https://hg.mozilla.org/mozilla-central/rev/8306a663c330