Bug 920429. Fix busted boolean expression. r=mattwoodrow
authorRobert O'Callahan <robert@ocallahan.org>
Wed, 25 Sep 2013 23:11:23 +1200
changeset 148648 10205aaf44cddfe8811e481057e10b14c2d0e326
parent 148647 722a17aaa05b211358b40c89a70c34e6b560cd2e
child 148649 de77a9248c0481cb809adee38962c21342b73c3c
push id25349
push userryanvm@gmail.com
push dateWed, 25 Sep 2013 18:52:12 +0000
treeherdermozilla-central@39f30376058c [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
reviewersmattwoodrow
bugs920429
milestone27.0a1
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 920429. Fix busted boolean expression. r=mattwoodrow
gfx/layers/ThebesLayerBuffer.cpp
--- a/gfx/layers/ThebesLayerBuffer.cpp
+++ b/gfx/layers/ThebesLayerBuffer.cpp
@@ -594,17 +594,17 @@ ThebesLayerBuffer::BeginPaint(ThebesLaye
       // pixels are sampled
       neededRegion = destBufferRect;
     }
 
     // If we have an existing buffer, but the content type has changed or we
     // have transitioned into/out of component alpha, then we need to recreate it.
     if (HaveBuffer() &&
         (contentType != BufferContentType() ||
-         mode == Layer::SURFACE_COMPONENT_ALPHA) != (HaveBufferOnWhite())) {
+        (mode == Layer::SURFACE_COMPONENT_ALPHA) != HaveBufferOnWhite())) {
 
       // We're effectively clearing the valid region, so we need to draw
       // the entire needed region now.
       result.mRegionToInvalidate = aLayer->GetValidRegion();
       validRegion.SetEmpty();
       Clear();
       // Restart decision process with the cleared buffer. We can only go
       // around the loop one more iteration, since mBuffer is null now.