576931 - "ASSERTION: Wrong children repair!" upon enabling accessibility

Status: RESOLVED FIXED

(Core :: Disability Access APIs, defect)

Description

[Jesse Ruderman]

1. In the JS console, paste
Components.classes["@mozilla.org/accessibilityService;1"]     
.getService(Components.interfaces.nsIAccessibleRetrieval);

Result:

###!!! ASSERTION: Wrong children repair!: 'parent == mParent', file /home/jruderman/mozilla-central/accessible/src/base/nsAccessible.cpp, line 2830

This assertion is new to me, so I guess this is a recent regression.

Comment 1

[David Bolter [:davidb] (NeedInfo me for attention)]

The assertion was added July 1st.

Comment 2

[Serge Gautherie (:sgautherie)]

http://tinderbox.mozilla.org/showlog.cgi?log=SeaMonkey/1282913047.1282915896.13545.gz&fulltext=1
Linux comm-central-trunk debug test mochitest-other on 2010/08/27 05:44:07

Lots of
{
WARNING: Bad accessible tree!: file /builds/slave/comm-central-trunk-linux-debug/build/mozilla/accessible/src/base/nsAccessible.cpp, line 2849
###!!! ASSERTION: Wrong children repair!: 'parent == mParent', file /builds/slave/comm-central-trunk-linux-debug/build/mozilla/accessible/src/base/nsAccessible.cpp, line 2864
}

For example:
{
JavaScript strict warning: chrome://mochikit/content/a11y/accessible/events/docload_wnd.xul, line 55: assignment to undeclared variable gFindBarInitialized

JavaScript error: chrome://navigator/content/tabbrowser.xml, line 919: nsIInterfaceRequestor is not defined

...

1886 ERROR TEST-UNEXPECTED-FAIL | chrome://mochikit/content/a11y/accessible/events/test_docload.xul | Test timed out.
}

Comment 3

[[e-mail]]

I don't know whether the JS warnings/errors relate to the assertions.

(In reply to comment #2)
> JavaScript strict warning:
> chrome://mochikit/content/a11y/accessible/events/docload_wnd.xul, line 55:
> assignment to undeclared variable gFindBarInitialized
We don't have that variable, which presumably explains why it's undeclared.

> JavaScript error: chrome://navigator/content/tabbrowser.xml, line 919:
> nsIInterfaceRequestor is not defined
This is a bug; it should be Components.interfaces.nsIInterfaceRequestor but I obviously accidentally omitted the qualifiers when I fixed bug 457548. (It probably finds a shortcut declaration somewhere in the global scope.)

Comment 4

[Serge Gautherie (:sgautherie)]

(In reply to comment #3)
> This is a bug; it should be Components.interfaces.nsIInterfaceRequestor

I filed bug 591550.

Comment 5

[Serge Gautherie (:sgautherie)]

(In reply to comment #3)
> We don't have that variable, which presumably explains why it's undeclared.

I filed bug 591639.

Comment 6

[alexander :surkov (:asurkov)]

one case of this bug is described by bug 574588, perhaps most frequent case

Comment 7

[Serge Gautherie (:sgautherie)]

Ftr, status after bug 574588 check-in:
http://tinderbox.mozilla.org/showlog.cgi?log=SeaMonkey/1283743505.1283745870.31878.gz&fulltext=1
Linux comm-central-trunk debug test mochitest-other on 2010/09/05 20:25:05
http://tinderbox.mozilla.org/showlog.cgi?log=SeaMonkey/1283730619.1283734494.24060.gz&fulltext=1
WINNT 5.2 comm-central-trunk debug test mochitest-other on 2010/09/05 16:50:19
still report this assertion.

Comment 8

[Serge Gautherie (:sgautherie)]

Ftr, status after bug 592189 "check-in":
http://tinderbox.mozilla.org/showlog.cgi?log=Firefox/1288148075.1288150456.21883.gz&fulltext=1
Rev3 Fedora 12 mozilla-central debug test mochitest-other on 2010/10/26
still reports this assertion.

Comment 9

[Serge Gautherie (:sgautherie)]

Ftr, status after bug 545465 check-in: (for example)
http://tinderbox.mozilla.org/showlog.cgi?log=Firefox/1289676623.1289679096.25424.gz&fulltext=1
Rev3 Fedora 12 mozilla-central debug test mochitest-other on 2010/11/13 11:30:23
http://tinderbox.mozilla.org/showlog.cgi?log=SeaMonkey/1289677070.1289680244.29452.gz&fulltext=1
WINNT 5.2 comm-central-trunk debug test mochitest-other on 2010/11/13 11:37:50
still report this assertion.

Comment 10

[alexander :surkov (:asurkov)]

it's fixed by patch1 of bug 626667 (Gecko 2.0 beta11).