1743967 - Implement DoAction for cached RemoteAccessible on Windows

Status: RESOLVED FIXED

(Core :: Disability Access APIs, task)

Description

[James Teh [:Jamie]]

Similar to TakeFocus and ScrollTo. We'll also need to support the other action methods, but the most useful is DoAction, so let's deal with that first.

Comment 1

[Eitan Isaacson [:eeejay]]

Attachment: Bug 1743967 - Support async DoAction. r=Jamie

Now that we cache ActionCount, we can check for the absence of actions
and return false, or send an async message and return true.

Depends on D135908

Comment 2

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/6f15d287ecd8
Support async DoAction. r=Jamie

Comment 3

[Pulsebot]

Backout by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/2db711522a1f
Backed out 5 changesets (bug 1743967, bug 1748749) for causing failures at browser_caching_actions.js. CLOSED TREE

Comment 4

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/0748b3df321c
Support async DoAction. r=Jamie

Comment 5

[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 6

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/8b5356f3d50d
Support async DoAction. r=Jamie

Comment 7

[Pulsebot]

Backout by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/2e89ef6864e8
Backed out 6 changesets (bug 1743967, bug 1748749, bug 1749828) for causing bc failures on browser_PBMCookieBehavior.js.

Comment 8

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/24cdb9a96bc7
Support async DoAction. r=Jamie

Comment 9

[Norisz Fay [:noriszfay]]

https://hg.mozilla.org/mozilla-central/rev/24cdb9a96bc7