Port suitableForGpuRasterization to SkRecord

src/core/SkRecord.h

Index: src/core/SkRecord.h
diff --git a/src/core/SkRecord.h b/src/core/SkRecord.h
index 96da69b12ef8c150729c07c8e846fab5452cb1dc..e7ff37c6eb69fae93d19dc2d291de6236f1028c0 100644
--- a/src/core/SkRecord.h
+++ b/src/core/SkRecord.h
@@ -13,6 +13,15 @@
 #include "SkTLogic.h"
 #include "SkTemplates.h"
 
+struct SkRecordAnalysis {
+    SkRecordAnalysis()
+        : fWillPlaybackBitmaps(false)
+        , fSuitableForGpuRasterization(false) { }
+
+    bool fWillPlaybackBitmaps;
+    bool fSuitableForGpuRasterization;
+};
+
 // SkRecord (REC-ord) represents a sequence of SkCanvas calls, saved for future use.
 // These future uses may include: replay, optimization, serialization, or combinations of those.
 //
@@ -110,6 +119,8 @@ public:
         return fRecords[i].set(this->allocCommand<T>());
     }
 
+    SkRecordAnalysis fAccelerationInfo;

[mtklein, 2014/08/18 16:34:58]

Let's keep this parallel to SkRecord like fWillPlaybackBitmaps used to be?  I'm
suspicious of having a public member, especially one that's not actually used by
any code in SkRecord.

[tomhudson, 2014/08/18 16:58:47]

(This was intended to be a private member with accessors; that's part of the
cleanup noted.)

Ideally this struct would be a const member of SkPicture, but as I tried to
explain last week I was reluctant to add yet another header to include/core/.
I'll throw up a version of that so you can see what it looks like.

+
 private:
     // Implementation notes!
     //