Viewer: balance save/restores and draw offscreen to window in src mode

Change-Id: I9a9bff1c950aaeda095ee49b4860c6fee04ea731
Reviewed-on: https://skia-review.googlesource.com/8887
Reviewed-by: Brian Osman <brianosman@google.com>
Commit-Queue: Brian Salomon <bsalomon@google.com>

diff --git a/tools/viewer/Viewer.cpp b/tools/viewer/Viewer.cpp
index 092910a..c7517cc 100644 (file)
--- a/tools/viewer/Viewer.cpp
+++ b/tools/viewer/Viewer.cpp
@@ -596,7 +596,8 @@ void Viewer::setColorMode(SkColorType colorType, sk_sp<SkColorSpace> colorSpace)
 }
 
 void Viewer::drawSlide(SkCanvas* canvas) {
-    int count = canvas->save();
+    SkAutoCanvasRestore autorestore(canvas, false);
+
     if (fWindow->supportsContentRect()) {
         SkRect contentRect = fWindow->getContentRect();
         canvas->clipRect(contentRect);
@@ -617,14 +618,15 @@ void Viewer::drawSlide(SkCanvas* canvas) {
         slideCanvas = offscreenSurface->getCanvas();
     }
 
+    int count = slideCanvas->save();
     slideCanvas->clear(SK_ColorWHITE);
     slideCanvas->concat(fDefaultMatrix);
     slideCanvas->concat(computeMatrix());
-
     // Time the painting logic of the slide
     double startTime = SkTime::GetMSecs();
     fSlides[fCurrentSlide]->draw(slideCanvas);
     fPaintTimes[fCurrentMeasurement] = SkTime::GetMSecs() - startTime;
+    slideCanvas->restoreToCount(count);
 
     // Force a flush so we can time that, too
     startTime = SkTime::GetMSecs();
@@ -639,10 +641,10 @@ void Viewer::drawSlide(SkCanvas* canvas) {
         sk_sp<SkColorSpace> cs = (kRGBA_F16_SkColorType == fColorType)
             ? SkColorSpace::MakeSRGBLinear() : SkColorSpace::MakeSRGB();
         auto retaggedImage = SkImageMakeRasterCopyAndAssignColorSpace(fLastImage.get(), cs.get());
-        canvas->drawImage(retaggedImage, 0, 0);
+        SkPaint paint;
+        paint.setBlendMode(SkBlendMode::kSrc);
+        canvas->drawImage(retaggedImage, 0, 0, &paint);
     }
-
-    canvas->restoreToCount(count);
 }
 
 void Viewer::onPaint(SkCanvas* canvas) {