Bug 1449641 - Remove extraneous "if (mRenderTarget->mDrawTarget != mDrawTarget)". r?sotaro
The same condition is checked just before this if, so it's always true.
MozReview-Commit-ID: 9Vscnkz7AoY
--- a/gfx/layers/basic/BasicCompositor.cpp
+++ b/gfx/layers/basic/BasicCompositor.cpp
@@ -1033,22 +1033,17 @@ BasicCompositor::TryToEndRemoteDrawing(b
[self]() { self->TryToEndRemoteDrawing(); });
MessageLoop::current()->PostDelayedTask(runnable.forget(), retryMs);
return;
}
if (mRenderTarget->mDrawTarget != mDrawTarget) {
// Note: Most platforms require us to buffer drawing to the widget surface.
// That's why we don't draw to mDrawTarget directly.
- RefPtr<SourceSurface> source;
- if (mRenderTarget->mDrawTarget != mDrawTarget) {
- source = mWidget->EndBackBufferDrawing();
- } else {
- source = mRenderTarget->mDrawTarget->Snapshot();
- }
+ RefPtr<SourceSurface> source = mWidget->EndBackBufferDrawing();
RefPtr<DrawTarget> dest(mTarget ? mTarget : mDrawTarget);
nsIntPoint offset = mTarget ? mTargetBounds.TopLeft() : nsIntPoint();
// The source DrawTarget is clipped to the invalidation region, so we have
// to copy the individual rectangles in the region or else we'll draw blank
// pixels.
for (auto iter = mInvalidRegion.RectIter(); !iter.Done(); iter.Next()) {