Add SkTileGridPicture SampleApp playback support.

samplecode/SamplePictFile.cpp

 
 /*
  * Copyright 2011 Google Inc.
  *
  * Use of this source code is governed by a BSD-style license that can be
  * found in the LICENSE file.
  */
 #include "SampleCode.h"
 #include "SkDumpCanvas.h"
 #include "SkView.h"
 #include "SkCanvas.h"
 #include "Sk64.h"
 #include "SkGradientShader.h"
 #include "SkGraphics.h"
 #include "SkImageDecoder.h"
 #include "SkOSFile.h"
 #include "SkPath.h"
 #include "SkPicture.h"
 #include "SkRandom.h"
 #include "SkRegion.h"
 #include "SkShader.h"
+#include "SkTileGridPicture.h"
 #include "SkUtils.h"
 #include "SkColorPriv.h"
 #include "SkColorFilter.h"
 #include "SkTime.h"
 #include "SkTypeface.h"
 #include "SkXfermode.h"
 
 #include "SkStream.h"
 #include "SkSurface.h"
 #include "SkXMLParser.h"
 
 class PictFileView : public SampleView {
+public:
+    PictFileView(const char name[] = NULL)
+        : fFilename(name)
+        , fBBox(kNo_BBox)
+        , fTileSize(SkSize::Make(0, 0)) {
+        for (unsigned i = 0; i < kLast_BBox_enum; ++i) {
+            fPictures[i] = NULL;
+        }
+    }
+
+    virtual ~PictFileView() {
+        for (unsigned i = 0; i < kLast_BBox_enum; ++i) {
+            SkSafeUnref(fPictures[i]);
+        }
+    }
+

[robertphillips, 2013/12/10 18:44:28]

override?

[fmalita_google_do_not_use, 2013/12/10 19:48:17]

Done.

+    virtual void onTileSizeChanged(const SkSize &tileSize) {
+        if (tileSize != fTileSize) {
+            fTileSize = tileSize;

[robertphillips, 2013/12/10 18:44:28]

SkSafeSetNull?

[fmalita_google_do_not_use, 2013/12/10 19:48:17]

Done.

+            SkSafeUnref(fPictures[kTileGrid_BBox]);
+            fPictures[kTileGrid_BBox] = NULL;
+        }
+    }
+
+protected:
+    // overrides from SkEventSink

[robertphillips, 2013/12/10 18:44:28]

override keyword?

[fmalita_google_do_not_use, 2013/12/10 19:48:17]

Done.

+    virtual bool onQuery(SkEvent* evt) {
+        if (SampleCode::TitleQ(*evt)) {
+            SkString name("P:");
+            const char* basename = strrchr(fFilename.c_str(), SkPATH_SEPARATOR);
+            name.append(basename ? basename+1: fFilename.c_str());
+            if (fBBox != kNo_BBox) {
+                name.append(fBBox == kRTree_BBox ? " <bbox: R>" : " <bbox: T>");
+            }
+            SampleCode::TitleR(evt, name.c_str());
+            return true;
+        }
+        return this->INHERITED::onQuery(evt);
+    }
+
+    virtual bool onEvent(const SkEvent& evt) {
+        if (evt.isType("PictFileView::toggleBBox")) {
+            fBBox = (BBoxType)((fBBox + 1) % kLast_BBox_enum);
+            return true;
+        }
+        return this->INHERITED::onEvent(evt);
+    }
+
+    virtual void onDrawContent(SkCanvas* canvas) {
+        SkASSERT(fBBox < kLast_BBox_enum);
+        SkPicture** picture = fPictures + fBBox;
+
+        if (!*picture) {
+            *picture = LoadPicture(fFilename.c_str(), fBBox);
+        }
+        if (*picture) {
+            canvas->drawPicture(**picture);
+        }
+    }
+
+private:
+    enum BBoxType {

[robertphillips, 2013/12/10 18:44:28]

kNo_BBoxType? same for others?

[fmalita_google_do_not_use, 2013/12/10 19:48:17]

Done.

+        kNo_BBox,
+        kRTree_BBox,
+        kTileGrid_BBox,
+

[robertphillips, 2013/12/10 18:44:28]

kLast_BBoxType? (no _enum)?
Also shouldn't this be:

kLast_BBoxType = kTileGrid_BBoxType
};
static const BBoxType kBBoxTypeCount = kLast_BBoxType + 1;

[fmalita_google_do_not_use, 2013/12/10 19:48:17]

Oh yeah, the style guide is quite specific about this. Done.

+        kLast_BBox_enum
+    };
+
     SkString    fFilename;
-    SkPicture*  fPicture;
-    SkPicture*  fBBoxPicture;
-    bool        fUseBBox;
+    SkPicture*  fPictures[kLast_BBox_enum];
+    BBoxType    fBBox;
+    SkSize      fTileSize;
 
-    static SkPicture* LoadPicture(const char path[], bool useBBox) {
+    SkPicture* LoadPicture(const char path[], BBoxType bbox) {
         SkPicture* pic = NULL;
 
         SkBitmap bm;
         if (SkImageDecoder::DecodeFile(path, &bm)) {
             bm.setImmutable();
             pic = SkNEW(SkPicture);
             SkCanvas* can = pic->beginRecording(bm.width(), bm.height());
             can->drawBitmap(bm, 0, 0, NULL);
             pic->endRecording();
         } else {
@@ skipping 13 matching lines @@
                 SkPicture p2;
                 pic->draw(p2.beginRecording(pic->width(), pic->height()));
                 p2.endRecording();
 
                 SkString path2(path);
                 path2.append(".new.skp");
                 SkFILEWStream writer(path2.c_str());
                 p2.serialize(&writer);
             }
         }
 

[robertphillips, 2013/12/10 18:44:28]

put const on the left hand side (e.g., kNo_BBox != bbox)?

[fmalita_google_do_not_use, 2013/12/10 19:48:17]

Done.

-        if (useBBox) {
-            SkPicture* bboxPicture = SkNEW(SkPicture);
+        if (bbox != kNo_BBox) {

[robertphillips, 2013/12/10 18:44:28]

Doesn't the next 5 lines only need to be done if we are creating a tile grid?

[fmalita_google_do_not_use, 2013/12/10 19:48:17]

That is correct, I was trading sloppiness for brevity. There's probably a better
way to handle it... done.

+            SkASSERT(!fTileSize.isEmpty());
+            SkTileGridPicture::TileGridInfo gridInfo;
+            gridInfo.fMargin = SkISize::Make(0, 0);
+            gridInfo.fOffset = SkIPoint::Make(0, 0);
+            gridInfo.fTileInterval = fTileSize.toRound();
+
+            SkPicture* bboxPicture = (bbox == kRTree_BBox)
+                    ? SkNEW(SkPicture)
+                    : SkNEW_ARGS(SkTileGridPicture, (pic->width(), pic->height(), gridInfo));
             pic->draw(bboxPicture->beginRecording(pic->width(), pic->height(),
                     SkPicture::kOptimizeForClippedPlayback_RecordingFlag));
             bboxPicture->endRecording();
             SkDELETE(pic);
             return bboxPicture;
 
         } else {
             return pic;
         }
     }
 
-public:
-    PictFileView(const char name[] = NULL) : fFilename(name) {
-        fPicture = NULL;
-        fBBoxPicture = NULL;
-        fUseBBox = false;
-    }
-
-    virtual ~PictFileView() {
-        SkSafeUnref(fPicture);
-        SkSafeUnref(fBBoxPicture);
-    }
-
-protected:
-    // overrides from SkEventSink
-    virtual bool onQuery(SkEvent* evt) {
-        if (SampleCode::TitleQ(*evt)) {
-            SkString name("P:");
-            const char* basename = strrchr(fFilename.c_str(), SkPATH_SEPARATOR);
-            name.append(basename ? basename+1: fFilename.c_str());
-            if (fUseBBox) {
-                name.append(" <bbox>");
-            }
-            SampleCode::TitleR(evt, name.c_str());
-            return true;
-        }
-        return this->INHERITED::onQuery(evt);
-    }
-
-    virtual bool onEvent(const SkEvent& evt) {
-        if (evt.isType("PictFileView::toggleBBox")) {
-            fUseBBox = !fUseBBox;
-            return true;
-        }
-        return this->INHERITED::onEvent(evt);
-    }
-
-    virtual void onDrawContent(SkCanvas* canvas) {
-        SkPicture** picture = fUseBBox ? &fBBoxPicture : &fPicture;
-
-        if (!*picture) {
-            *picture = LoadPicture(fFilename.c_str(), fUseBBox);
-        }
-        if (*picture) {
-            canvas->drawPicture(**picture);
-        }
-    }
-
-private:
     typedef SampleView INHERITED;
 };
 
 SampleView* CreateSamplePictFileView(const char filename[]);
 SampleView* CreateSamplePictFileView(const char filename[]) {
     return new PictFileView(filename);
 }
 
 //////////////////////////////////////////////////////////////////////////////
 
 #if 0
 static SkView* MyFactory() { return new PictFileView; }
 static SkViewRegister reg(MyFactory);
 #endif