Bug 462868 - background-color text attribute being exposed as black when it isn't, r=aaronlev, marcoz

accessible/src/base/nsTextUtils.cpp

@@ -228,7 +228,7 @@ nsBackgroundTextAttr::GetColor(nsIFrame *aFrame)
 
   nsIFrame *parentFrame = aFrame->GetParent();
   if (!parentFrame)
-    return styleBackground->mBackgroundColor;
+    return aFrame->PresContext()->DefaultBackgroundColor();
 
   // Each frame of parents chain for the initially passed 'aFrame' has
   // transparent background color. So background color isn't changed from

accessible/tests/mochitest/test_textattrs.html

@@ -107,7 +107,7 @@
       var defAttrs = {
         "font-style": gComputedStyle.fontStyle,
         "font-size": gComputedStyle.fontSize,
-        "background-color": "rgb(0, 0, 0)", // XXX 455834
+        "background-color": "rgb(255, 255, 255)",
         "font-weight": gComputedStyle.fontWeight,
         "color": gComputedStyle.color,
         "font-family": gComputedStyle.fontFamily,
@@ -135,7 +135,7 @@
       defAttrs = {
         "font-style": gComputedStyle.fontStyle,
         "font-size": gComputedStyle.fontSize,
-        "background-color": "rgb(0, 0, 0)", // XXX bug 455834
+        "background-color": "rgb(255, 255, 255)",
         "font-weight": gComputedStyle.fontWeight,
         "color": gComputedStyle.color,
         "font-family": gComputedStyle.fontFamily,
@@ -212,7 +212,7 @@
       defAttrs = {
         "font-style": gComputedStyle.fontStyle,
         "font-size": gComputedStyle.fontSize,
-        "background-color": "rgb(0, 0, 0)", // XXX bug 455834
+        "background-color": "rgb(255, 255, 255)",
         "font-weight": gComputedStyle.fontWeight,
         "color": gComputedStyle.color,
         "font-family": gComputedStyle.fontFamily,
@@ -244,7 +244,7 @@
       defAttrs = {
         "font-style": gComputedStyle.fontStyle,
         "font-size": gComputedStyle.fontSize,
-        "background-color": "rgb(0, 0, 0)", // XXX bug 455834
+        "background-color": "rgb(255, 255, 255)",
         "font-weight": gComputedStyle.fontWeight,
         "color": gComputedStyle.color,
         "font-family": gComputedStyle.fontFamily,
@@ -277,7 +277,7 @@
       defAttrs = {
         "font-style": gComputedStyle.fontStyle,
         "font-size": gComputedStyle.fontSize,
-        "background-color": "rgb(0, 0, 0)", // XXX bug 455834
+        "background-color": "rgb(255, 255, 255)",
         "font-weight": gComputedStyle.fontWeight,
         "color": gComputedStyle.color,
         "font-family": gComputedStyle.fontFamily,
@@ -328,7 +328,7 @@
       defAttrs = {
         "font-style": gComputedStyle.fontStyle,
         "font-size": gComputedStyle.fontSize,
-        "background-color": "rgb(0, 0, 0)", // XXX 455834
+        "background-color": "rgb(255, 255, 255)",
         "font-weight": gComputedStyle.fontWeight,
         "color": gComputedStyle.color,
         "font-family": gComputedStyle.fontFamily,