commit dc97fdc23571ab8126f8c2a8b0991cf2280ef023
author Robert Phillips <robertphillips@google.com> Wed Jul 18 08:27:19 2018 -0400
committer Skia Commit-Bot <skia-commit-bot@chromium.org> Wed Jul 18 14:11:01 2018 +0000
tree 187ee72b969765aa3ddb8501c67f21968dec743b
parent c3c1c319d22f34583c5994d9b0326596820f8f3d

Fix Valgrind skinning bug

This should take care of all the issues like:

Conditional jump or move depends on uninitialised value(s)
   SkRect::join(float, float, float, float) (SkRect.cpp:119)
   join (SkRect.h:1370)
   GrDrawVerticesOp::GrDrawVerticesOp(...) (GrDrawVerticesOp.cpp:105)

Change-Id: I520ea55c9e310017bd8d5e2a7c9c343e3ac8eb40
Reviewed-on: https://skia-review.googlesource.com/142101
Commit-Queue: Ben Wagner <bungeman@google.com>
Auto-Submit: Robert Phillips <robertphillips@google.com>
Reviewed-by: Ben Wagner <bungeman@google.com>

src/gpu/ops/GrDrawVerticesOp.cpp

diff --git a/src/gpu/ops/GrDrawVerticesOp.cpp b/src/gpu/ops/GrDrawVerticesOp.cpp
index b5d16ea..d227920 100644
--- a/src/gpu/ops/GrDrawVerticesOp.cpp
+++ b/src/gpu/ops/GrDrawVerticesOp.cpp
@@ -98,7 +98,7 @@
     if (this->hasBones()) {
         // We don't know the bounds if there are deformations involved, so attempt to calculate
         // the maximum possible.
-        SkRect bounds;
+        SkRect bounds = SkRect::MakeEmpty();
         const SkRect originalBounds = bones[0].mapRect(mesh.fVertices->bounds());
         for (int i = 1; i < boneCount; i++) {
             const SkMatrix& matrix = bones[i];