Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(5)

Unified Diff: tools/PictureRenderer.h

Issue 845623002: Remove SkTileGrid (except for TileGridInfo). (Closed) Base URL: https://skia.googlesource.com/skia.git@master
Patch Set: kill gridSupported Created 5 years, 11 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « tests/TileGridTest.cpp ('k') | tools/PictureRenderer.cpp » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: tools/PictureRenderer.h
diff --git a/tools/PictureRenderer.h b/tools/PictureRenderer.h
index c8462d9f037c213d1b9bc718c6ac524100be5f69..bfa09e397d58e5993d3ffaab3a844e73f56d066c 100644
--- a/tools/PictureRenderer.h
+++ b/tools/PictureRenderer.h
@@ -57,9 +57,8 @@ public:
enum BBoxHierarchyType {
kNone_BBoxHierarchyType = 0,
kRTree_BBoxHierarchyType,
- kTileGrid_BBoxHierarchyType,
- kLast_BBoxHierarchyType = kTileGrid_BBoxHierarchyType,
+ kLast_BBoxHierarchyType = kRTree_BBoxHierarchyType,
};
// this uses SkPaint::Flags as a base and adds additional flags
@@ -237,10 +236,6 @@ public:
BBoxHierarchyType getBBoxHierarchyType() { return fBBoxHierarchyType; }
- void setGridSize(int width, int height) {
- fGridInfo.fTileInterval.set(width, height);
- }
-
void setJsonSummaryPtr(ImageResultsAndExpectations* jsonSummaryPtr) {
fJsonSummaryPtr = jsonSummaryPtr;
}
@@ -266,12 +261,6 @@ public:
}
if (kRTree_BBoxHierarchyType == fBBoxHierarchyType) {
config.append("_rtree");
- } else if (kTileGrid_BBoxHierarchyType == fBBoxHierarchyType) {
- config.append("_grid");
- config.append("_");
- config.appendS32(fGridInfo.fTileInterval.width());
- config.append("x");
- config.appendS32(fGridInfo.fTileInterval.height());
}
#if SK_SUPPORT_GPU
switch (fDeviceType) {
@@ -316,12 +305,6 @@ public:
}
if (kRTree_BBoxHierarchyType == fBBoxHierarchyType) {
result["bbh"] = "rtree";
- } else if (kTileGrid_BBoxHierarchyType == fBBoxHierarchyType) {
- SkString tmp("grid_");
- tmp.appendS32(fGridInfo.fTileInterval.width());
- tmp.append("x");
- tmp.appendS32(fGridInfo.fTileInterval.height());
- result["bbh"] = tmp.c_str();
}
#if SK_SUPPORT_GPU
SkString tmp;
@@ -439,9 +422,6 @@ public:
, fUseDFText(false)
#endif
{
- fGridInfo.fMargin.setEmpty();
- fGridInfo.fOffset.setZero();
- fGridInfo.fTileInterval.set(1, 1);
sk_bzero(fDrawFilters, sizeof(fDrawFilters));
fViewport.set(0, 0);
}
@@ -466,7 +446,6 @@ protected:
SkString fWritePath;
SkString fMismatchPath;
SkString fInputFilename;
- SkTileGridFactory::TileGridInfo fGridInfo; // used when fBBoxHierarchyType is TileGrid
void buildBBoxHierarchy();
@@ -696,7 +675,7 @@ private:
void setupTiles();
void setupPowerOf2Tiles();
- bool postRender(SkCanvas*, const SkIRect& tileRect,
+ bool postRender(SkCanvas*, const SkIRect& tileRect,
SkBitmap* tempBM, SkBitmap** out,
int tileNumber);
« no previous file with comments | « tests/TileGridTest.cpp ('k') | tools/PictureRenderer.cpp » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698