Bug 1331951 - Remove plugin async drawing work around that landed in bug 1312242. r=bsmedberg, a=jcristau
authorJim Mathies <jmathies@mozilla.com>
Fri, 20 Jan 2017 13:59:13 -0600
changeset 353697 2d482631e0f14763a95ab18680d154cd267b775b
parent 353696 086c474cfefbe660922ce3d588dea7213559b24a
child 353698 82ca0b8cafe57be09bcede3a195f3b4cf46f9249
push id6795
push userjlund@mozilla.com
push dateMon, 23 Jan 2017 14:19:46 +0000
treeherdermozilla-esr52@76101b503191 [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
reviewersbsmedberg, jcristau
bugs1331951, 1312242
milestone52.0a2
Bug 1331951 - Remove plugin async drawing work around that landed in bug 1312242. r=bsmedberg, a=jcristau
dom/plugins/ipc/PluginModuleParent.cpp
--- a/dom/plugins/ipc/PluginModuleParent.cpp
+++ b/dom/plugins/ipc/PluginModuleParent.cpp
@@ -2725,27 +2725,16 @@ PluginModuleParent::NPP_NewInternal(NPMI
                if (!values[wmodeAttributeIndex].EqualsLiteral("transparent")) {
                    values[wmodeAttributeIndex].Assign(opaqueAttributeValue);
                }
            } else {
                names.AppendElement(wmodeAttributeName);
                values.AppendElement(opaqueAttributeValue);
            }
         }
-
-      // Update the flashvar bgcolor if it's not set, fixes a rendering problem with
-      // async plugin painting and transparent flash.
-      if (supportsAsyncRender) {
-        NS_NAMED_LITERAL_CSTRING(bgcolorAttributeName, "bgcolor");
-        NS_NAMED_LITERAL_CSTRING(bgcolorAttributeDefault, "#FFFFFF");
-        if (!names.Contains(bgcolorAttributeName)) {
-          names.AppendElement(bgcolorAttributeName);
-          values.AppendElement(bgcolorAttributeDefault);
-        }
-      }
 #endif
     }
 
     // Release the surrogate reference that was in pdata
     RefPtr<PluginAsyncSurrogate> surrogate(
         dont_AddRef(PluginAsyncSurrogate::Cast(instance)));
     // Now replace it with the instance
     instance->pdata = static_cast<PluginDataResolver*>(parentInstance);