Closed Bug 1748749 Opened 2 years ago Closed 2 years ago

Cache ActionCount/ActionNameAt

Categories

(Core :: Disability Access APIs, task)

task

Tracking

()

RESOLVED FIXED
98 Branch
Tracking Status
firefox98 --- fixed

People

(Reporter: eeejay, Assigned: eeejay)

References

Details

(Whiteboard: [ctw-m1] )

Attachments

(4 files)

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

Type: defect → task

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

Assignee: nobody → eitan
Status: NEW → ASSIGNED

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

Depends on D135905

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

Depends on D135906

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

Also introduce browser test.

Depends on D135907

Depends on: 1752380
Pushed by eisaacson@mozilla.com:
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
Attachment #9258957 - Attachment description: Bug 1748749 - P1: Push action name to cache. r?Jamie → Bug 1748749 - P1: Push action name to cache. r=Jamie
Attachment #9258958 - Attachment description: Bug 1748749 - P2: Cache has long desc. r?Jamie → Bug 1748749 - P2: Cache has long desc. r=Jamie
Attachment #9258959 - Attachment description: Bug 1748749 - P3: Move TranslateString to Accessible. r?Jamie → Bug 1748749 - P3: Move TranslateString to Accessible. r=Jamie
Attachment #9258960 - Attachment description: Bug 1748749 - P4: Support cached actions. r?Jamie → Bug 1748749 - P4: Support cached actions. r=Jamie
Pushed by eisaacson@mozilla.com:
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

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'
Pushed by eisaacson@mozilla.com:
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
Pushed by csabou@mozilla.com:
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
Flags: needinfo?(eitan)
Regressions: 1756755
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: