Fix SkPicture path bound optimization to handle inverse filled paths.

src/core/SkPictureRecord.cpp

Index: src/core/SkPictureRecord.cpp
===================================================================
--- src/core/SkPictureRecord.cpp	(revision 9020)
+++ src/core/SkPictureRecord.cpp	(working copy)
@@ -783,7 +783,11 @@
     validate(initialOffset, size);
 
     if (fRecordFlags & SkPicture::kUsePathBoundsForClip_RecordingFlag) {
-        return this->INHERITED::clipRect(path.getBounds(), op, doAA);
+        if (!path.isInverseFillType()) {

[reed1, 2013/05/06 21:27:19]

this seems a little double negatively. Is it correct to say?

if (path.isInverseFillType()) {
    return kEmpty_ClipType != this->getClipType();
} else {
    return this->INHERITED::...
}

That said, is that even correct? I wonder if we need to check if the current
clip is contained inside the path in the inverse case. If it is, then we should
return false...

+            return this->INHERITED::clipRect(path.getBounds(), op, doAA);
+        } else {
+            return this->getClipDeviceBounds(NULL);
+        }
     } else {
         return this->INHERITED::clipPath(path, op, doAA);
     }