[SkDebugger] Handle path empty-bounds gracefully

src/utils/debugger/SkDrawCommand.cpp

 
 /*
  * Copyright 2012 Google Inc.
  *
  * Use of this source code is governed by a BSD-style license that can be
  * found in the LICENSE file.
  */
 
 
 #include "SkDrawCommand.h"
@@ skipping 80 matching lines @@
         canvas->scale(SkDoubleToScalar((kInsetFrac*size.fWidth)/bounds.width()),
                       SkDoubleToScalar((kInsetFrac*size.fHeight)/bounds.width()));
     } else {
         canvas->scale(SkDoubleToScalar((kInsetFrac*size.fWidth)/bounds.height()),
                       SkDoubleToScalar((kInsetFrac*size.fHeight)/bounds.height()));
     }
     canvas->translate(-bounds.centerX(), -bounds.centerY());
 }
 
 
 void render_path(SkCanvas* canvas, const SkPath& path) {

[robertphillips, 2015/10/06 14:12:55]

Maybe put this after the clear so we get a blank screen ?

[f(malita), 2015/10/06 14:17:21]

Done.

+    const SkRect& bounds = path.getBounds();
+    if (bounds.isEmpty()) {
+        return;
+    }
+
+    SkAutoCanvasRestore acr(canvas, true);
     canvas->clear(0xFFFFFFFF);
-    canvas->save();
-
-    const SkRect& bounds = path.getBounds();
 
     xlate_and_scale_to_bounds(canvas, bounds);
 
     SkPaint p;
     p.setColor(SK_ColorBLACK);
     p.setStyle(SkPaint::kStroke_Style);
 
     canvas->drawPath(path, p);
-    canvas->restore();
 }
 
 void render_bitmap(SkCanvas* canvas, const SkBitmap& input, const SkRect* srcRect = nullptr) {
     const SkISize& size = canvas->getDeviceSize();
 
     SkScalar xScale = SkIntToScalar(size.fWidth-2) / input.width();
     SkScalar yScale = SkIntToScalar(size.fHeight-2) / input.height();
 
     if (input.width() > input.height()) {
         yScale *= input.height() / (float) input.width();
@@ skipping 850 matching lines @@
 
 void SkSetMatrixCommand::setUserMatrix(const SkMatrix& userMatrix) {
     fUserMatrix = userMatrix;
 }
 
 void SkSetMatrixCommand::execute(SkCanvas* canvas) const {
     SkMatrix temp = SkMatrix::Concat(fUserMatrix, fMatrix);
     canvas->setMatrix(temp);
 }