Fix use of uninitialized memory in skia/ext/analysis_canvas.cc

skia/ext/analysis_canvas.cc

 // Copyright (c) 2013 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "base/logging.h"
 #include "base/trace_event/trace_event.h"
 #include "skia/ext/analysis_canvas.h"
 #include "third_party/skia/include/core/SkDraw.h"
 #include "third_party/skia/include/core/SkRRect.h"
 #include "third_party/skia/include/core/SkShader.h"
@@ skipping 42 matching lines @@
            xfermode == SkXfermode::kSrcOver_Mode));
 }
 
 // Returns true if the specified drawn_rect will cover the entire canvas, and
 // that the canvas is not clipped (i.e. it covers ALL of the canvas).
 bool IsFullQuad(SkCanvas* canvas, const SkRect& drawn_rect) {
   if (!canvas->isClipRect())
     return false;
 
   SkIRect clip_irect;
-  canvas->getClipDeviceBounds(&clip_irect);
+  if (!canvas->getClipDeviceBounds(&clip_irect))
+    return false;
+  
   // if the clip is smaller than the canvas, we're partly clipped, so abort.
   if (!clip_irect.contains(SkIRect::MakeSize(canvas->getDeviceSize())))
     return false;
 
   const SkMatrix& matrix = canvas->getTotalMatrix();
   // If the transform results in a non-axis aligned
   // rect, then be conservative and return false.
   if (!matrix.rectStaysRect())
     return false;
 
@@ skipping 15 matching lines @@
 }
 
 void AnalysisCanvas::SetForceNotTransparent(bool flag) {
   is_forced_not_transparent_ = flag;
   if (is_forced_not_transparent_)
     is_transparent_ = false;
 }
 
 void AnalysisCanvas::onDrawPaint(const SkPaint& paint) {
   SkRect rect;
-  getClipBounds(&rect);
-  drawRect(rect, paint);
+  if (getClipBounds(&rect))
+    drawRect(rect, paint);
 }
 
 void AnalysisCanvas::onDrawPoints(SkCanvas::PointMode mode,
                                   size_t count,
                                   const SkPoint points[],
                                   const SkPaint& paint) {
   is_solid_color_ = false;
   is_transparent_ = false;
   ++draw_op_count_;
 }
@@ skipping 333 matching lines @@
       force_not_transparent_stack_level_ = kNoLayer;
     }
   }
 
   INHERITED::willRestore();
 }
 
 }  // namespace skia