Bug#743440: iceweasel not accessible (to screen readers, atk, spi)

[Samuel Thibault]

Hello,

Is iceweasel 32 or 33 planned to be uploaded for Jessie?

If not, it would be really important to at least apply the attached
patch from upstream in Jessie, to fix the complete inaccessibility of
iceweasel...

FTR, the removed code was used for old gnome versions, which are no more
in Jessie.

Samuel
-------------- next part --------------

# HG changeset patch
# User Trevor Saunders <trev.saunders at gmail.com>
# Date 1399052780 14400
# Node ID 6b3e73a12bb603a184cf3c945eb14a4d43ed5048
# Parent  f4693085c5070693ecbbf1827c7e3f37b0d23311
bug 743580 - don't remove atk-bridge from GTK_MODULES during startup r=ginn, karlt

diff --git a/toolkit/xre/nsAppRunner.cpp b/toolkit/xre/nsAppRunner.cpp
--- a/toolkit/xre/nsAppRunner.cpp
+++ b/toolkit/xre/nsAppRunner.cpp
@@ -2912,29 +2912,18 @@ XREMain::XRE_mainInit(bool* aExitFlag)
       Output(true, "Could not determine HOME directory");
       return 1;
     }
     SaveWordToEnv("HOME", nsDependentCString(pw->pw_dir));
   }
 #endif
 
 #ifdef MOZ_ACCESSIBILITY_ATK
-  // Reset GTK_MODULES, strip atk-bridge if exists
-  // Mozilla will load libatk-bridge.so later if necessary
-  const char* gtkModules = PR_GetEnv("GTK_MODULES");
-  if (gtkModules && *gtkModules) {
-    nsCString gtkModulesStr(gtkModules);
-    gtkModulesStr.ReplaceSubstring("atk-bridge", "");
-    char* expr = PR_smprintf("GTK_MODULES=%s", gtkModulesStr.get());
-    if (expr)
-      PR_SetEnv(expr);
-    // We intentionally leak |expr| here since it is required by PR_SetEnv.
-  }
-
-  // Suppress atk-bridge init at startup, it works after GNOME 2.24.2
+  // Suppress atk-bridge init at startup, until mozilla accessibility is
+  // initialized.  This works after gnome 2.24.2.
   SaveToEnv("NO_AT_BRIDGE=1");
 #endif
 
   // Check for application.ini overrides
   const char* override = nullptr;
   ar = CheckArg("override", true, &override);
   if (ar == ARG_BAD) {
     Output(true, "Incorrect number of arguments passed to -override");