Backed out changeset 68de70a3d9f8 (bug 969591) for assertions.
authorRyan VanderMeulen <ryanvm@gmail.com>
Mon, 10 Feb 2014 11:04:20 -0500
changeset 167870 67ea2ad5b39ef99ba9aaaee99ca92a2e8bc9be5d
parent 167869 21226a58f8c60f1149e95eae5c60a1724f7ad159
child 167871 e788872f644fc2ab8b4cc7c5e7f52c44f84cdd8f
push id26190
push userryanvm@gmail.com
push dateMon, 10 Feb 2014 20:37:53 +0000
treeherdermozilla-central@07739c5c874f [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
bugs969591
milestone30.0a1
backs out68de70a3d9f8c2a5295708356ea8672dafa5dcd5
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
Backed out changeset 68de70a3d9f8 (bug 969591) for assertions.
gfx/layers/RotatedBuffer.cpp
--- a/gfx/layers/RotatedBuffer.cpp
+++ b/gfx/layers/RotatedBuffer.cpp
@@ -498,26 +498,18 @@ RotatedContentBuffer::BeginPaint(ThebesL
 
   result.mRegionToDraw.Sub(neededRegion, validRegion);
 
   // Do not modify result.mRegionToDraw or result.mContentType after this call.
   // Do not modify mBufferRect, mBufferRotation, or mDidSelfCopy,
   // or call CreateBuffer before this call.
   FinalizeFrame(result.mRegionToDraw);
 
-  // Have some cases where the region to draw is empty,
-  // but the needed region is not empty, so we still need to allocate
-  // a clean buffer for those cases. If everything is empty, then clear out
-  // the buffers.
-  if (result.mRegionToDraw.IsEmpty() && canReuseBuffer) {
-    if (neededRegion.IsEmpty()) {
-      Clear();
-    }
+  if (result.mRegionToDraw.IsEmpty())
     return result;
-  }
 
   nsIntRect drawBounds = result.mRegionToDraw.GetBounds();
   RefPtr<DrawTarget> destDTBuffer;
   RefPtr<DrawTarget> destDTBufferOnWhite;
   uint32_t bufferFlags = canHaveRotation ? ALLOW_REPEAT : 0;
   if (mode == SurfaceMode::SURFACE_COMPONENT_ALPHA) {
     bufferFlags |= BUFFER_COMPONENT_ALPHA;
   }