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

diff --git a/include/core/SkPathRef.h b/include/core/SkPathRef.h
index ed7c31765323dd63169e99f927654270ad87fee9..ba68fcba3c60f77a1e2dae760d71d0b3bbd21c02 100644 (file)
--- a/include/core/SkPathRef.h
+++ b/include/core/SkPathRef.h
@@ -291,7 +291,9 @@ private:
     // 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;