Closed Bug 178393 Opened 21 years ago Closed 12 years ago

nsAccessibilityInterface module load should also work when GTK_MODULES is set

Categories

(Core :: XUL, defect)

x86
Linux
defect
Not set
normal

Tracking

()

RESOLVED FIXED

People

(Reporter: yinbolian, Assigned: yinbolian)

References

Details

(Keywords: access)

Attachments

(1 file)

GTK_MODULES is not a good way to enable accessibility feature on Gnome2
platform. But some pure gtk programs still use it to enable accessibility
feature. Mozilla should work when GTK_MODULES is set to other accessibility
modules, such as "GTK_MODULES=gail:atk-bridge".
Blocks: 145863
Status: NEW → ASSIGNED
QA Contact: jrgm → mindy.liu
Summary: MAI module load should also work when GTK_MODULES is set → nsAccessibilityInterface module load should also work when GTK_MODULES is set
Attached patch patchSplinter Review
Attachment #105166 - Flags: review?(blizzard)
Comment on attachment 105166 [details] [diff] [review]
patch

104 **	 The caller must ensure that the string passed
105 **	 to PR_SetEnv() is persistent. That is: The string should
106 **	 not be on the stack, where it can be overwritten
107 **	 on return from the function calling PR_SetEnv().
108 **	 Similarly, the string passed to PR_SetEnv() must not be
109 **	 overwritten by other actions of the process. ... Some
110 **	 platforms use the string by reference rather than copying
111 **	 it into the environment space. ... You have been warned!
Attachment #105166 - Flags: review?(blizzard) → review-
QA Contact: mindy.liu → xptoolkit.widgets
Trevor, is this still relevant at all?
Keywords: access
(In reply to Marco Zehe (:MarcoZ) from comment #3)
> Trevor, is this still relevant at all?

well, none of the patch, the subject, or the description make it clear to me what problem needs solved, so I say its useless, and so close it as invalid
Status: ASSIGNED → RESOLVED
Closed: 12 years ago
Resolution: --- → INVALID
I believe this bug is fixed.
Resolution: INVALID → FIXED
You need to log in before you can comment on or make changes to this bug.