commit 0b4cc892cb9edd4167133a465b01baa476ecc244
author Mike Klein <mtklein@google.com> Wed Jul 16 17:18:20 2014 -0400
committer Mike Klein <mtklein@google.com> Wed Jul 16 17:18:20 2014 -0400
tree 410b3f339f900a9510005ce270db248f87a8728c
parent e194c495e95cbe92dfb9ed04515904a417fe7e9b

Disable assertion of fBoundsIsDirty.

We allow this to be raced on, and it may have already become not-dirty by the
time we get to this function if computed by another thread.

BUG=skia:

Review URL: https://codereview.chromium.org/398913002

include/core/SkPathRef.h

diff --git a/include/core/SkPathRef.h b/include/core/SkPathRef.h
index ed7c317..ba68fcb 100644
--- a/include/core/SkPathRef.h
+++ b/include/core/SkPathRef.h
@@ -291,7 +291,9 @@
     // called, if dirty, by getBounds()
     void computeBounds() const {
         SkDEBUGCODE(this->validate();)
-        SkASSERT(fBoundsIsDirty);
+        // TODO(mtklein): remove fBoundsIsDirty and fIsFinite,
+        // using an inverted rect instead of fBoundsIsDirty and always recalculating fIsFinite.
+        //SkASSERT(fBoundsIsDirty);
 
         fIsFinite = ComputePtBounds(fBounds.get(), *this);
         fBoundsIsDirty = false;