SkRecord: make culling work if SkRecordAnnotateCullingPairs is called.

src/record/SkRecorder.cpp

 #include "SkRecorder.h"
 #include "SkPicture.h"
 
 // SkCanvas will fail in mysterious ways if it doesn't know the real width and height.
 SkRecorder::SkRecorder(SkRecord* record, int width, int height)
     : SkCanvas(width, height), fRecord(record) {}
 
 // To make appending to fRecord a little less verbose.
 #define APPEND(T, ...) \
         SkNEW_PLACEMENT_ARGS(fRecord->append<SkRecords::T>(), SkRecords::T, (__VA_ARGS__))
@@ skipping 178 matching lines @@
                                                       SkCanvas::SaveFlags flags) {
     APPEND(SaveLayer, this->copy(bounds), this->copy(paint), flags);
     return SkCanvas::kNoLayer_SaveLayerStrategy;
 }
 
 void SkRecorder::willRestore() {
     APPEND(Restore);
 }
 
 void SkRecorder::onPushCull(const SkRect& rect) {
-    APPEND(PushCull, rect);
+    static const unsigned kImpossiblePopOffset = 0;
+    APPEND(PushCull, rect, kImpossiblePopOffset);

[f(malita), 2014/04/08 23:08:31]

If using a constant here, maybe we should promote it and also use it for the
check in Draw::operator()

[mtklein, 2014/04/08 23:11:56]

Done.

 }
 
 void SkRecorder::onPopCull() {
     APPEND(PopCull);
 }
 
 void SkRecorder::didConcat(const SkMatrix& matrix) {
     APPEND(Concat, matrix);
 }
 
@@ skipping 13 matching lines @@
     APPEND(ClipRRect, rrect, op, edgeStyle == kSoft_ClipEdgeStyle);
 }
 
 void SkRecorder::onClipPath(const SkPath& path, SkRegion::Op op, ClipEdgeStyle edgeStyle) {
     APPEND(ClipPath, delay_copy(path), op, edgeStyle == kSoft_ClipEdgeStyle);
 }
 
 void SkRecorder::onClipRegion(const SkRegion& deviceRgn, SkRegion::Op op) {
     APPEND(ClipRegion, delay_copy(deviceRgn), op);
 }