Yura Zenevich [:yzen] (please use "needinfo") Assignee
	Description • 9 years ago

In some cases documentElement is not available and causes an error.
Caused by bug 1214398

	Yura Zenevich [:yzen] (please use "needinfo") Assignee
	Comment 1 • 9 years ago

What this also results in is that Pointer Adapter is not shut down and thus even though the AccessFu is stopped, the pointer events are still blocked and the device is inoperable.

	Yura Zenevich [:yzen] (please use "needinfo") Assignee
	Comment 3 • 9 years ago

Comment on attachment 8682195 [details] [diff] [review]
1220860 patch

Review of attachment 8682195 [details] [diff] [review]:
-----------------------------------------------------------------

::: accessible/jsat/Presentation.jsm
@@ +697,5 @@
>      aPosition, aOldPosition, aReason, aStartOffset, aEndOffset, aIsUserInput) {
>      let context = new PivotContext(
>        aPosition, aOldPosition, aStartOffset, aEndOffset);
> +    if (context.accessible) {
> +      this.displayedAccessibles.set(context.accessible.document.window, context);

This seems to fail, because sometimes context.accessible is null

	Yura Zenevich [:yzen] (please use "needinfo") Assignee
	Comment 4 • 9 years ago

Updated

	Yura Zenevich [:yzen] (please use "needinfo") Assignee
	Comment 5 • 9 years ago

https://hg.mozilla.org/integration/mozilla-inbound/rev/0aea92b111d7

	Carsten Book [:Tomcat]
	Comment 6 • 9 years ago
bugherder

https://hg.mozilla.org/mozilla-central/rev/0aea92b111d7

	Carsten Book [:Tomcat]
	Comment 7 • 9 years ago
bugherder uplift

https://hg.mozilla.org/releases/mozilla-b2g44_v2_5/rev/0aea92b111d7

	Yura Zenevich [:yzen] (please use "needinfo") Assignee
	Comment 9 • 9 years ago

For some reason the patch looks missing in v2.5 branch:
https://hg.mozilla.org/releases/mozilla-b2g44_v2_5/file/tip/accessible/jsat/AccessFu.jsm#l526

If it didn't stick, is it possible to re-land it on v2.5?