The default bug view has changed. See this FAQ.

"Assertion failure in -[mozRootAccessible didReceiveFocus]" "trying to set focus to ignored element"

RESOLVED FIXED in mozilla17

Status

()

Core
Disability Access APIs
RESOLVED FIXED
6 years ago
5 years ago

People

(Reporter: Jesse Ruderman, Assigned: hub)

Tracking

(Blocks: 2 bugs, {testcase})

Trunk
mozilla17
x86_64
Mac OS X
testcase
Points:
---
Dependency tree / graph

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(3 attachments)

(Reporter)

Description

6 years ago
Created attachment 543093 [details]
testcase

1. Build Firefox with --enable-accessibility.
2. Enable accessibility, e.g. by pasting the following into the js console:

Components.classes["@mozilla.org/accessibilityService;1"]
      .getService(Components.interfaces.nsIAccessibleRetrieval);

3. Load the testcase.
4. Click the button.

Result:

2011-06-30 01:59:27.785 firefox-bin[55214:903] *** Assertion failure in -[mozRootAccessible didReceiveFocus], /Users/jruderman/trees/mozilla-central/accessible/src/mac/mozAccessible.mm:582

2011-06-30 01:59:27.786 firefox-bin[55214:903] Mozilla has caught an Obj-C exception [NSInternalInconsistencyException: trying to set focus to ignored element! ((0x129cb4b10) AXUnknown)]

"Regression" from bug 524775.

This assertion is also mentioned in bug 450531, fwiw.

Updated

6 years ago
Blocks: 342989
(Assignee)

Comment 1

5 years ago
I can't reproduce this on my tree....
(Reporter)

Comment 2

5 years ago
Still happens for me, using a debug build from Tinderbox on Mac OS X 10.6.
(Assignee)

Comment 3

5 years ago
(In reply to Jesse Ruderman from comment #2)
> Still happens for me, using a debug build from Tinderbox on Mac OS X 10.6.

Which tinderbox build? One that does enable a11y or the regular one (that has it disabled)?
(Reporter)

Comment 4

5 years ago
I can reproduce with the build in https://ftp.mozilla.org/pub/mozilla.org/firefox/tinderbox-builds/mozilla-central-macosx64-debug/1323730278/, for example. It's built with --enable-accessibility.  (IIRC, the "regular" Mac debug builds have accessibility enabled iff they are 64-bit builds.)
(Assignee)

Comment 5

5 years ago
Even with that built I can't reproduce...
(Assignee)

Comment 6

5 years ago
I have gotten that with DOM Inspector.
(Assignee)

Updated

5 years ago
Blocks: 761574
(Assignee)

Comment 7

5 years ago
Unlike I said in comment 5 I completely can reproduce with the test case.
QA Contact: accessibility-apis
(Assignee)

Updated

5 years ago
Assignee: nobody → hub
(Assignee)

Updated

5 years ago
Duplicate of this bug: 450531
(Assignee)

Comment 9

5 years ago
Created attachment 654200 [details] [diff] [review]
Don't ignore focusable elements. r=
(Assignee)

Comment 10

5 years ago
Comment on attachment 654200 [details] [diff] [review]
Don't ignore focusable elements. r=

This is as proposed in bug 761574
Attachment #654200 - Flags: review?(surkov.alexander)

Comment 11

5 years ago
Comment on attachment 654200 [details] [diff] [review]
Don't ignore focusable elements. r=

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

r=me, thank you

::: accessible/src/mac/mozAccessible.mm
@@ +101,5 @@
>  
>    // unknown (either unimplemented, or irrelevant) elements are marked as ignored
>    // as well as expired elements.
> +  return !mGeckoAccessible || ([[self role] isEqualToString:NSAccessibilityUnknownRole] &&
> +                               !(mGeckoAccessible->State() & states::FOCUSABLE));

use NativeInteractiveState
Attachment #654200 - Flags: review?(surkov.alexander) → review+
(Assignee)

Comment 12

5 years ago
https://hg.mozilla.org/integration/mozilla-inbound/rev/269195e72bf9
Backed out because it landed on a burning tree

https://hg.mozilla.org/integration/mozilla-inbound/rev/46365c62935c
(Assignee)

Comment 14

5 years ago
https://hg.mozilla.org/integration/mozilla-inbound/rev/7055bd788274

Comment 15

5 years ago
Hub, sorry to say that, but InteractiveState should be used instead, otherwise aria-descendant logic which affects on focusable state is ignored.
(Assignee)

Comment 16

5 years ago
ok, I'll fix it.
(Assignee)

Updated

5 years ago
Whiteboard: [leave open]
https://hg.mozilla.org/mozilla-central/rev/7055bd788274
(Assignee)

Updated

5 years ago
Attachment #654200 - Flags: checkin+
(Assignee)

Comment 18

5 years ago
Created attachment 655261 [details] [diff] [review]
Part 2: use InteractiveState() instead of NativeInteractiveState().
(Assignee)

Updated

5 years ago
Attachment #655261 - Flags: review?(surkov.alexander)

Comment 19

5 years ago
Comment on attachment 655261 [details] [diff] [review]
Part 2: use InteractiveState() instead of NativeInteractiveState().

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

you don't really need review for this
Attachment #655261 - Flags: review?(surkov.alexander) → review+
(Assignee)

Comment 20

5 years ago
https://hg.mozilla.org/integration/mozilla-inbound/rev/0b399d43b570
Whiteboard: [leave open]
(Assignee)

Updated

5 years ago
Attachment #655261 - Flags: checkin+
https://hg.mozilla.org/mozilla-central/rev/0b399d43b570
Status: NEW → RESOLVED
Last Resolved: 5 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla17

Updated

5 years ago
Duplicate of this bug: 761574
You need to log in before you can comment on or make changes to this bug.