Bug 612840. Part 2: Remove unnecessary opacity check. r=cjones
authorRobert O'Callahan <robert@ocallahan.org>
Mon, 20 Dec 2010 14:26:14 +1300
changeset 59480 53bc550efbccbf583419f1027de08c9b4e56b9ba
parent 59479 ca873b14eb719c0bfed13b81a6b71007208f9278
child 59481 bc423fb0aa13fb9c2d261c9993ccf6021fe85ff0
push id17638
push userrocallahan@mozilla.com
push dateMon, 20 Dec 2010 01:39:18 +0000
treeherdermozilla-central@302d1d3e2817 [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
reviewerscjones
bugs612840
milestone2.0b9pre
first release with
nightly linux32
nightly linux64
nightly mac
nightly win32
nightly win64
last release without
nightly linux32
nightly linux64
nightly mac
nightly win32
nightly win64
Bug 612840. Part 2: Remove unnecessary opacity check. r=cjones
gfx/layers/basic/BasicLayers.cpp
--- a/gfx/layers/basic/BasicLayers.cpp
+++ b/gfx/layers/basic/BasicLayers.cpp
@@ -463,17 +463,17 @@ BasicThebesLayer::Paint(gfxContext* aCon
 
   PRBool canUseOpaqueSurface = CanUseOpaqueSurface();
   Buffer::ContentType contentType =
     canUseOpaqueSurface ? gfxASurface::CONTENT_COLOR :
                           gfxASurface::CONTENT_COLOR_ALPHA;
   float opacity = GetEffectiveOpacity();
 
   if (!BasicManager()->IsRetained() ||
-      (opacity == 1.0 && !canUseOpaqueSurface &&
+      (!canUseOpaqueSurface &&
        !(mContentFlags & CONTENT_NO_TEXT_OVER_TRANSPARENT) &&
        !MustRetainContent())) {
     mValidRegion.SetEmpty();
     mBuffer.Clear();
 
     nsIntRegion toDraw = IntersectWithClip(mVisibleRegion, target);
     if (!toDraw.IsEmpty()) {
       target->Save();