388608 - Accessible Hierarchy broken effective 17th July trunk build

Status: RESOLVED FIXED

(Core :: Disability Access APIs, defect, P2)

Description

[Joanmarie Diggs]

Steps to reproduce:

1. Launch either Firefox or Thunderbird from 17th July or later
2. Launch Accerciser and use it to examine the hierarchy of accessibles

Expected results:  The hierarchy would be exposed.

Actual results:  The application's frame has a single child of role "redundant object" which claims 0 children.

This problem does not occur with the 16th July and earlier builds.

Comment 1

[Joanmarie Diggs]

Sorry for the spam.  I forgot to update the priority.  P2 because this is making accessibility testing and development impossible.  Thanks!

Comment 2

[alexander :surkov (:asurkov)]

Ginn, is it possible bug 387347 brake this?

Comment 3

[Ginn Chen]

I don't think so.

I think our a11y lib is not well loaded.

I saw symbol unresolved issue on Solaris. may related.

Comment 4

[Ginn Chen]

I believe it's caused by Bug 93789.

Comment 5

[Ginn Chen]

Attachment: patch

I can't use PR_SetEnv to unsetenv on Unix.
Anyway, it's for Unix only, I think it's safe to use getenv, setenv, unsetenv directly.

Comment 6

[alexander :surkov (:asurkov)]

+                    aaa:labelledby="onloadRadio onloadProvider onloadAfter"/>

IIRC we don't support IDREFS for the value of @labelledby. We always assume that value is IDREF. Am I wrong?

Comment 7

[Ginn Chen]

FWIW: surkov's last comment is not for this bug.

Comment 8

[Ginn Chen]

Firefox / Thunderbird is totally inaccessible on Linux / Solaris because of this bug.

Comment 9

[Benjamin Smedberg]

Comment on attachment 272909 [details] [diff] [review]
patch

I was told at one point that there were unix-like OSes that don't support unsetenv, and that the XP way to do this was PR_SetEnv("NAME=") which would set the value to empty if there was not an unsetenv implementation. But you're welcome to try it and see whether my memory is incorrect.

Comment 10

[Ginn Chen]

In nsprpub/pr/include/prenv.h

A call to PR_SetEnv() where the parameter is
of the form "name=" may un-set the envrionment variable on
some platforms; on others it may set the value of the
environment variable to the null-string.

Linux and Solaris is not the "some platform".

Checkin and see if any platform is broken.

Comment 11

[Firefox Bug Husbandry Bot]

https://bugzilla.mozilla.org/show_bug.cgi?id=1527381