remove unused GM flags
authormtklein <mtklein@chromium.org>
Fri, 23 Jan 2015 19:07:07 +0000 (11:07 -0800)
committerCommit bot <commit-bot@chromium.org>
Fri, 23 Jan 2015 19:07:08 +0000 (11:07 -0800)
Depends on https://codereview.chromium.org/873753002/

Thumbs up to CLion for refactoring this for me.

BUG=skia:

Review URL: https://codereview.chromium.org/867963004

130 files changed:
bench/GMBench.cpp
gm/aaclip.cpp
gm/aarectmodes.cpp
gm/alphagradients.cpp
gm/arcofzorro.cpp
gm/beziereffects.cpp
gm/beziers.cpp
gm/bigtext.cpp
gm/bitmapcopy.cpp
gm/bitmaprect.cpp
gm/bitmapscroll.cpp
gm/bleed.cpp
gm/blurcircles.cpp
gm/blurrect.cpp
gm/blurroundrect.cpp
gm/blurs.cpp
gm/circles.cpp
gm/circularclips.cpp
gm/clip_strokerect.cpp
gm/colorfilters.cpp
gm/colortype.cpp
gm/colortypexfermode.cpp
gm/complexclip.cpp
gm/complexclip2.cpp
gm/complexclip3.cpp
gm/conicpaths.cpp
gm/convexpaths.cpp
gm/convexpolyclip.cpp
gm/convexpolyeffect.cpp
gm/cubicpaths.cpp
gm/dashcubics.cpp
gm/dashing.cpp
gm/dcshader.cpp
gm/degeneratesegments.cpp
gm/dftext.cpp
gm/discard.cpp
gm/downsamplebitmap.cpp
gm/dropshadowimagefilter.cpp
gm/drrect.cpp
gm/factory.cpp
gm/fatpathfill.cpp
gm/filltypes.cpp
gm/filltypespersp.cpp
gm/filterbitmap.cpp
gm/filterfastbounds.cpp
gm/filterindiabox.cpp
gm/fontcache.cpp
gm/fontmgr.cpp
gm/fontscaler.cpp
gm/getpostextpath.cpp
gm/giantbitmap.cpp
gm/glyph_pos.cpp
gm/glyph_pos_align.cpp
gm/gm.h
gm/gradient_matrix.cpp
gm/gradients.cpp
gm/gradients_no_texture.cpp
gm/gradtext.cpp
gm/hairlines.cpp
gm/hairmodes.cpp
gm/hittestpath.cpp
gm/image.cpp
gm/imagealphathreshold.cpp
gm/imageblur.cpp
gm/imageblur2.cpp
gm/imagefiltersbase.cpp
gm/imagefiltersclipped.cpp
gm/imagefilterscropped.cpp
gm/imagefiltersgraph.cpp
gm/imagefiltersscaled.cpp
gm/imagemagnifier.cpp
gm/imageresizetiled.cpp
gm/inversepaths.cpp
gm/lcdtext.cpp
gm/lighting.cpp
gm/linepaths.cpp
gm/lumafilter.cpp
gm/matrixconvolution.cpp
gm/mixedxfermodes.cpp
gm/multipicturedraw.cpp
gm/nested.cpp
gm/nonclosedpaths.cpp
gm/ovals.cpp
gm/patch.cpp
gm/patchgrid.cpp
gm/patheffects.cpp
gm/pathfill.cpp
gm/pathreverse.cpp
gm/peekpixels.cpp
gm/pictureimagefilter.cpp
gm/pictureshader.cpp
gm/pictureshadertile.cpp
gm/points.cpp
gm/poly2poly.cpp
gm/polygons.cpp
gm/quadpaths.cpp
gm/rects.cpp
gm/resizeimagefilter.cpp
gm/roundrects.cpp
gm/rrect.cpp
gm/rrects.cpp
gm/samplerstress.cpp
gm/selftest.cpp
gm/shaderbounds.cpp
gm/shadertext.cpp
gm/shadertext2.cpp
gm/shadertext3.cpp
gm/shallowgradient.cpp
gm/spritebitmap.cpp
gm/stringart.cpp
gm/strokefill.cpp
gm/strokerect.cpp
gm/strokerects.cpp
gm/strokes.cpp
gm/stroketext.cpp
gm/tallstretchedbitmaps.cpp
gm/techtalk1.cpp
gm/testimagefilters.cpp
gm/texdata.cpp
gm/textblobshader.cpp
gm/texturedomaineffect.cpp
gm/tiledscaledbitmap.cpp
gm/tilemodes.cpp
gm/tilemodes_scaled.cpp
gm/variedtext.cpp
gm/vertices.cpp
gm/verttext.cpp
gm/verttext2.cpp
gm/verylargebitmap.cpp
gm/yuvtorgbeffect.cpp

index bdf3290..bcc83eb 100644 (file)
@@ -18,22 +18,7 @@ const char* GMBench::onGetName() {
 }
 
 bool GMBench::isSuitableFor(Backend backend) {
-    uint32_t flags = fGM->getFlags();
-    switch (backend) {
-        case kGPU_Backend:
-            return !(skiagm::GM::kSkipGPU_Flag & flags);
-        case kPDF_Backend:
-            return !(skiagm::GM::kSkipPDF_Flag & flags);
-        case kRaster_Backend:
-            // GM doesn't have an equivalent flag. If the GM has known issues with 565 then
-            // we skip it for ALL raster configs in bench.
-            return !(skiagm::GM::kSkip565_Flag & flags);
-        case kNonRendering_Backend:
-            return false;
-        default:
-            SkDEBUGFAIL("Unexpected backend type.");
-            return false;
-    }
+    return kNonRendering_Backend != backend;
 }
 
 void GMBench::onDraw(const int loops, SkCanvas* canvas) {
index 6ac9f0b..e977f5f 100644 (file)
@@ -101,8 +101,6 @@ protected:
         draw_rect_tests(canvas);
     }
 
-    uint32_t onGetFlags() const SK_OVERRIDE { return kSkipPipe_Flag; }
-
 private:
     typedef skiagm::GM INHERITED;
 };
@@ -187,8 +185,6 @@ protected:
         }
     }
 
-    uint32_t onGetFlags() const SK_OVERRIDE { return kSkipPipe_Flag; }
-
 private:
     typedef skiagm::GM INHERITED;
 };
index 3c9a683..83456a0 100644 (file)
@@ -133,9 +133,6 @@ namespace skiagm {
         }
 
     protected:
-        uint32_t onGetFlags() const SK_OVERRIDE {
-            return kSkipTiled_Flag;
-        }
 
         SkString onShortName() SK_OVERRIDE {
             return SkString("aarectmodes");
index 4fd11ad..304c523 100644 (file)
@@ -72,8 +72,6 @@ protected:
         }
     }
 
-    uint32_t onGetFlags() const SK_OVERRIDE { return kSkipPipe_Flag | kSkipTiled_Flag; }
-
 private:
     typedef skiagm::GM INHERITED;
 };
index 7748f34..9aa4bb4 100644 (file)
@@ -20,9 +20,6 @@ public:
     }
 
 protected:
-    uint32_t onGetFlags() const SK_OVERRIDE {
-        return kSkipTiled_Flag;
-    }
 
     SkString onShortName() SK_OVERRIDE {
         return SkString("arcofzorro");
index b96952a..e49fc91 100644 (file)
@@ -44,11 +44,6 @@ protected:
         return SkISize::Make(800, 800);
     }
 
-    uint32_t onGetFlags() const SK_OVERRIDE {
-        // This is a GPU-specific GM.
-        return kGPUOnly_Flag;
-    }
-
 
     void onDraw(SkCanvas* canvas) SK_OVERRIDE {
         GrRenderTarget* rt = canvas->internal_private_accessTopLayerRenderTarget();
@@ -204,11 +199,6 @@ protected:
         return SkISize::Make(800, 800);
     }
 
-    uint32_t onGetFlags() const SK_OVERRIDE {
-        // This is a GPU-specific GM.
-        return kGPUOnly_Flag;
-    }
-
 
     void onDraw(SkCanvas* canvas) SK_OVERRIDE {
         GrRenderTarget* rt = canvas->internal_private_accessTopLayerRenderTarget();
@@ -397,11 +387,6 @@ protected:
         return SkISize::Make(800, 800);
     }
 
-    uint32_t onGetFlags() const SK_OVERRIDE {
-        // This is a GPU-specific GM.
-        return kGPUOnly_Flag;
-    }
-
 
     void onDraw(SkCanvas* canvas) SK_OVERRIDE {
         GrRenderTarget* rt = canvas->internal_private_accessTopLayerRenderTarget();
index a35f2a6..5ad34fb 100755 (executable)
@@ -46,9 +46,6 @@ public:
     BeziersGM() {}
 
 protected:
-    uint32_t onGetFlags() const SK_OVERRIDE {
-        return kSkipTiled_Flag;
-    }
 
     SkString onShortName() SK_OVERRIDE {
         return SkString("beziers");
index bdb3176..4fe77cc 100644 (file)
@@ -19,9 +19,6 @@ public:
     BigTextGM() {}
 
 protected:
-    uint32_t onGetFlags() const SK_OVERRIDE {
-        return kSkipTiled_Flag;
-    }
 
     SkString onShortName() SK_OVERRIDE {
         return SkString("bigtext");
index 74505d9..32e91e3 100644 (file)
@@ -114,9 +114,6 @@ protected:
         }
     }
 
-    virtual uint32_t onGetFlags() const { return kSkipPicture_Flag
-                                               | kSkipPipe_Flag; }
-
 private:
     typedef GM INHERITED;
 };
index ae505bf..8fb56f5 100644 (file)
@@ -45,13 +45,6 @@ protected:
         return SkISize::Make(640, 480);
     }
 
-#ifdef SK_CPU_ARM64
-    // Skip tiled drawing on 64-bit ARM until https://skbug.com/2908 is fixed.
-    uint32_t onGetFlags() const SK_OVERRIDE {
-        return kSkipTiled_Flag;
-    }
-#endif
-
     void onDraw(SkCanvas* canvas) SK_OVERRIDE {
         canvas->drawColor(0xFFCCCCCC);
 
@@ -271,7 +264,7 @@ protected:
         canvas->drawRect(r, paint);
         canvas->drawBitmapRect(fBM, NULL, r, NULL);
     }
-    
+
 private:
     typedef skiagm::GM INHERITED;
 };
index d5c3051..9f8aaca 100644 (file)
@@ -63,10 +63,6 @@ protected:
         return SkString("bitmapscroll");
     }
 
-    uint32_t onGetFlags() const SK_OVERRIDE {
-        return kSkipTiled_Flag;
-    }
-
     SkISize onISize() SK_OVERRIDE {
       return SkISize::Make(800, 600);
     }
index e361926..f39f3a1 100644 (file)
@@ -85,9 +85,6 @@ public:
     BleedGM() {}
 
 protected:
-    uint32_t onGetFlags() const SK_OVERRIDE {
-        return kSkipTiled_Flag;
-    }
 
     SkString onShortName() SK_OVERRIDE {
         return SkString("bleed");
index 0e7f916..ee30200 100644 (file)
@@ -19,9 +19,6 @@ public:
     }
 
 protected:
-    uint32_t onGetFlags() const SK_OVERRIDE {
-        return kSkipTiled_Flag;
-    }
 
     SkString onShortName() SK_OVERRIDE {
         return fName;
index a47b9cd..fb3ee37 100644 (file)
@@ -147,8 +147,6 @@ protected:
         }
     }
 
-    uint32_t onGetFlags() const SK_OVERRIDE { return kSkipPipe_Flag | kSkipTiled_Flag; }
-
 private:
     void drawProcs(SkCanvas* canvas, const SkRect& r, const SkPaint& paint,
                    bool doClip, const Proc procs[], size_t procsCount) {
@@ -248,8 +246,6 @@ protected:
         }
     }
 
-    virtual uint32_t onGetFlags() const { return kSkipPipe_Flag; }
-
 private:
     typedef GM INHERITED;
 };
@@ -320,8 +316,6 @@ protected:
         canvas->drawBitmap(bm, SkIntToScalar(center_x), SkIntToScalar(center_y), NULL);
     }
 
-    virtual uint32_t onGetFlags() const { return kSkipPipe_Flag; }
-
 private:
     typedef GM INHERITED;
 };
index c8468fe..45a1103 100644 (file)
@@ -126,9 +126,6 @@ public:
     }
 
 protected:
-    uint32_t onGetFlags() const SK_OVERRIDE {
-        return kSkipTiled_Flag;
-    }
 
     SkString onShortName() SK_OVERRIDE {
         return fName;
index d5bb7d2..96a9e30 100644 (file)
@@ -16,9 +16,6 @@ public:
     }
 
 protected:
-    uint32_t onGetFlags() const SK_OVERRIDE {
-        return kSkipTiled_Flag;
-    }
 
     SkString onShortName() SK_OVERRIDE {
         return SkString("blurs");
index b513ccb..c0503fd 100644 (file)
@@ -25,9 +25,6 @@ public:
     }
 
 protected:
-    uint32_t onGetFlags() const SK_OVERRIDE {
-        return kSkipTiled_Flag;
-    }
 
     SkString onShortName() SK_OVERRIDE {
         return SkString("circles");
index 69e8b69..ec766fb 100644 (file)
@@ -25,9 +25,6 @@ public:
     }
 
 protected:
-    uint32_t onGetFlags() const SK_OVERRIDE {
-        return kSkipTiled_Flag;
-    }
 
     bool runAsBench() const SK_OVERRIDE { return true; }
 
index f7c18e7..a0dc7fd 100644 (file)
@@ -63,8 +63,6 @@ protected:
         canvas->drawRect(rect2, p);
     }
 
-    uint32_t onGetFlags() const SK_OVERRIDE { return kSkipPipe_Flag; }
-
 private:
     typedef skiagm::GM INHERITED;
 };
index 9f278a5..9ecb56c 100644 (file)
@@ -49,10 +49,6 @@ protected:
         return SkISize::Make(640, 480);
     }
 
-    uint32_t onGetFlags() const SK_OVERRIDE {
-        return kSkipTiled_Flag;
-    }
-
     void onDraw(SkCanvas* canvas) SK_OVERRIDE {
         SkPaint paint;
         SkRect r;
index 9b541fa..c75a166 100644 (file)
@@ -60,10 +60,6 @@ protected:
         }
     }
 
-    virtual uint32_t onGetFlags() const {
-        return kSkipPipe_Flag | kSkipPicture_Flag;
-    }
-
 private:
     SkTypeface* fColorType;
 
index cd1b249..51cb7eb 100644 (file)
@@ -156,10 +156,6 @@ protected:
         s->unref();
     }
 
-    virtual uint32_t onGetFlags() const {
-        return kSkipPipe_Flag | kSkipPicture_Flag;
-    }
-
 private:
     SkTypeface* fColorType;
 
index f3d5a33..5603655 100644 (file)
@@ -27,9 +27,6 @@ public:
     }
 
 protected:
-    uint32_t onGetFlags() const SK_OVERRIDE {
-        return kSkipTiled_Flag;
-    }
 
 
     SkString onShortName() {
index 64cf3c8..a55d073 100644 (file)
@@ -111,13 +111,6 @@ protected:
         return "";
     }
 
-    uint32_t onGetFlags() const SK_OVERRIDE {
-        if (kRect_Clip != fClip) {
-            return kSkipTiled_Flag;
-        }
-        return 0;
-    }
-
     virtual SkString onShortName() {
         if (kRect_Clip == fClip && !fAntiAlias) {
             return SkString("complexclip2");
index bae5c5a..afabcf6 100644 (file)
@@ -21,9 +21,6 @@ public:
     }
 
 protected:
-    uint32_t onGetFlags() const SK_OVERRIDE {
-        return kSkipTiled_Flag;
-    }
 
     SkString onShortName() {
         SkString str;
index d04cd26..a5d5ee6 100644 (file)
@@ -106,12 +106,6 @@ protected:
         canvas->restore();
     }
 
-    uint32_t onGetFlags() const SK_OVERRIDE {
-        // tiling w/ non-antialias paths can cause off-by-1-pixels differences which are
-        // unavoidable (chopping in floats -vs- stepping in scan-converter).
-        return kSkipTiled_Flag;
-    }
-
 private:
     SkTArray<SkPath> fPaths;
     typedef skiagm::GM INHERITED;
index 01497b5..889d65b 100644 (file)
@@ -34,9 +34,6 @@ public:
     }
 
 protected:
-    uint32_t onGetFlags() const SK_OVERRIDE {
-        return kSkipTiled_Flag;
-    }
 
     virtual SkString onShortName() {
         return SkString("convexpaths");
index 0ec45b7..8d54b00 100644 (file)
@@ -214,10 +214,6 @@ protected:
         }
     }
 
-    virtual uint32_t onGetFlags() const {
-        return kSkipTiled_Flag;
-    }
-
     bool runAsBench() const SK_OVERRIDE { return true; }
 
 private:
index c92fd93..6e47057 100644 (file)
@@ -42,11 +42,6 @@ protected:
         return SkISize::Make(720, 800);
     }
 
-    uint32_t onGetFlags() const SK_OVERRIDE {
-        // This is a GPU-specific GM.
-        return kGPUOnly_Flag;
-    }
-
     void onOnceBeforeDraw() SK_OVERRIDE {
         SkPath tri;
         tri.moveTo(5.f, 5.f);
index bf05db7..3dd230c 100644 (file)
@@ -16,9 +16,6 @@ public:
     ClippedCubicGM() {}
 
 protected:
-    uint32_t onGetFlags() const SK_OVERRIDE {
-        return kSkipTiled_Flag;
-    }
 
     SkString onShortName() {
         return SkString("clippedcubic");
@@ -59,9 +56,6 @@ public:
     CubicPathGM() {}
 
 protected:
-    uint32_t onGetFlags() const SK_OVERRIDE {
-        return kSkipTiled_Flag;
-    }
 
     SkString onShortName() {
         return SkString("cubicpath");
@@ -208,9 +202,6 @@ public:
     CubicClosePathGM() {}
 
 protected:
-    uint32_t onGetFlags() const SK_OVERRIDE {
-        return kSkipTiled_Flag;
-    }
 
     SkString onShortName() {
         return SkString("cubicclosepath");
index 3d2d8d1..9a7d6d7 100644 (file)
@@ -20,9 +20,6 @@ public:
     DashCubicsGM() {}
 
 protected:
-    uint32_t onGetFlags() const SK_OVERRIDE {
-        return kSkipTiled_Flag;
-    }
 
     virtual SkString onShortName() {
         return SkString("dashcubics");
index a3d6094..ba3ede4 100644 (file)
@@ -49,9 +49,6 @@ public:
     DashingGM() {}
 
 protected:
-    uint32_t onGetFlags() const SK_OVERRIDE {
-        return kSkipTiled_Flag;
-    }
 
     SkString onShortName() {
         return SkString("dashing");
@@ -136,9 +133,6 @@ public:
     Dashing2GM() {}
 
 protected:
-    uint32_t onGetFlags() const SK_OVERRIDE {
-        return kSkipTiled_Flag;
-    }
 
     SkString onShortName() {
         return SkString("dashing2");
@@ -198,9 +192,6 @@ public:
     Dashing3GM() {}
 
 protected:
-    uint32_t onGetFlags() const SK_OVERRIDE {
-        return kSkipTiled_Flag;
-    }
 
     SkString onShortName() {
         return SkString("dashing3");
@@ -331,9 +322,6 @@ public:
     Dashing4GM() {}
 
 protected:
-    uint32_t onGetFlags() const SK_OVERRIDE {
-        return kSkipTiled_Flag;
-    }
 
     SkString onShortName() {
         return SkString("dashing4");
@@ -414,7 +402,6 @@ public:
     Dashing5GM(bool doAA) : fDoAA(doAA) {}
 
 protected:
-    uint32_t onGetFlags() const SK_OVERRIDE { return kSkipTiled_Flag; }
 
     bool runAsBench() const SK_OVERRIDE { return true; }
 
index 07af695..c08923f 100644 (file)
@@ -113,9 +113,6 @@ public:
     }
 
 protected:
-    uint32_t onGetFlags() const SK_OVERRIDE {
-        return kGPUOnly_Flag;
-    }
 
     SkString onShortName() SK_OVERRIDE {
         return SkString("dcshader");
index 09de2ee..e5ca187 100644 (file)
@@ -22,10 +22,6 @@ protected:
         const char* fName2;
     };
 
-    uint32_t onGetFlags() const SK_OVERRIDE {
-        return kSkipTiled_Flag;
-    }
-
     SkString onShortName() {
         return SkString("degeneratesegments");
     }
index 30b82d7..d6e19f6 100755 (executable)
@@ -34,10 +34,6 @@ protected:
         fTypeface = SkTypeface::CreateFromStream(stream.detach());
     }
 
-    uint32_t onGetFlags() const SK_OVERRIDE {
-        return kGPUOnly_Flag;
-    }
-
     SkString onShortName() SK_OVERRIDE {
         return SkString("dftext");
     }
index 19a332f..fc271d2 100644 (file)
@@ -25,8 +25,6 @@ public:
     DiscardGM() {
     }
 
-    uint32_t onGetFlags() const SK_OVERRIDE { return kGPUOnly_Flag; }
-
 protected:
     SkString onShortName() SK_OVERRIDE {
         return SkString("discard");
index 600d2ca..d7be637 100644 (file)
@@ -56,9 +56,6 @@ public:
     }
 
 protected:
-    uint32_t onGetFlags() const SK_OVERRIDE {
-        return kSkipTiled_Flag;
-    }
 
     SkString onShortName() SK_OVERRIDE {
         return fName;
@@ -205,11 +202,6 @@ public:
     }
 
 protected:
-#if 0
-    uint32_t onGetFlags() const SK_OVERRIDE {
-        return kSkipTiled_Flag;
-    }
-#endif
 
     SkString onShortName() SK_OVERRIDE {
         return SkString("showmiplevels");
@@ -241,7 +233,7 @@ protected:
             }
         }
     }
-    
+
 private:
     typedef skiagm::GM INHERITED;
 };
index 5a8c7a2..f60f22c 100644 (file)
@@ -111,14 +111,6 @@ protected:
         canvas->drawRect(r, paint);
     }
 
-    virtual uint32_t onGetFlags() const {
-        // Because of the use of drawSprite, this test is excluded
-        // from scaled replay tests because drawSprite ignores the
-        // reciprocal scale that is applied at record time, which is
-        // the intended behavior of drawSprite.
-        return kSkipScaledReplay_Flag | kSkipTiled_Flag;
-    }
-
     virtual void onDraw(SkCanvas* canvas) {
         void (*drawProc[])(SkCanvas*, const SkRect&, SkImageFilter*) = {
             draw_sprite, draw_bitmap, draw_path, draw_paint, draw_text
index 16c45b7..fd5a936 100644 (file)
@@ -15,9 +15,6 @@ public:
     DRRectGM() {}
 
 protected:
-    uint32_t onGetFlags() const SK_OVERRIDE {
-        return kSkipTiled_Flag;
-    }
 
     SkString onShortName() SK_OVERRIDE {
         return SkString("drrect");
index a2dc4dd..1e89d3d 100644 (file)
@@ -53,11 +53,6 @@ protected:
         canvas->drawBitmap(fBitmap, 0, 0);
     }
 
-    // Skip cross process pipe due to https://code.google.com/p/skia/issues/detail?id=1520
-    virtual uint32_t onGetFlags() const {
-        return INHERITED::onGetFlags() | kSkipPipeCrossProcess_Flag;
-    }
-
 private:
     SkBitmap fBitmap;
 
index e4b8a8d..1d269a0 100644 (file)
@@ -56,9 +56,6 @@ public:
     FatPathFillGM() {}
 
 protected:
-    uint32_t onGetFlags() const SK_OVERRIDE {
-        return kSkipTiled_Flag;
-    }
 
     virtual SkString onShortName() {
         return SkString("fatpathfill");
index 0d7e2c6..19249b2 100644 (file)
@@ -25,9 +25,6 @@ public:
     }
 
 protected:
-    uint32_t onGetFlags() const SK_OVERRIDE {
-        return kSkipTiled_Flag;
-    }
 
     SkString onShortName() SK_OVERRIDE {
         return SkString("filltypes");
index cfdb015..c76a0fa 100644 (file)
@@ -24,9 +24,6 @@ public:
     }
 
 protected:
-    uint32_t onGetFlags() const SK_OVERRIDE {
-        return kSkipTiled_Flag;
-    }
 
     SkString onShortName() SK_OVERRIDE {
         return SkString("filltypespersp");
index 57041fb..66f9d34 100644 (file)
@@ -70,9 +70,6 @@ public:
     }
 
 protected:
-    uint32_t onGetFlags() const SK_OVERRIDE {
-        return kSkipTiled_Flag;
-    }
 
     SkString onShortName() SK_OVERRIDE {
         return fName;
index def7b37..206903f 100644 (file)
@@ -160,15 +160,6 @@ protected:
     static const int kNumVertTiles = 6;
     static const int kNumXtraCols = 2;
 
-    // SkPictureImageFilter doesn't support serialization yet.
-    uint32_t onGetFlags() const SK_OVERRIDE {
-        return kSkipPicture_Flag            |
-               kSkipPipe_Flag               |
-               kSkipPipeCrossProcess_Flag   |
-               kSkipTiled_Flag              |
-               kSkipScaledReplay_Flag;
-    }
-
     SkString onShortName() SK_OVERRIDE{ return SkString("filterfastbounds"); }
 
     SkISize onISize() SK_OVERRIDE{
index 35ccf6e..1ed2a69 100644 (file)
@@ -74,13 +74,6 @@ protected:
         return fName;
     }
 
-#ifdef SK_CPU_ARM64
-    // Skip tiled drawing on 64-bit ARM until https://skbug.com/2908 is fixed.
-    uint32_t onGetFlags() const SK_OVERRIDE {
-        return kSkipTiled_Flag;
-    }
-#endif
-
     SkISize onISize() SK_OVERRIDE {
         return SkISize::Make(1024, 768);
     }
index 3363e24..2aa7011 100644 (file)
@@ -88,11 +88,6 @@ protected:
 
     }
 
-    uint32_t onGetFlags() const SK_OVERRIDE {
-        // this GM is meant only for the GPU
-        return kGPUOnly_Flag;
-    }
-
 private:
     SkTypeface* fTypefaces[TYPEFACE_COUNT];
     typedef GM INHERITED;
index 9d0e350..e89e585 100644 (file)
@@ -115,16 +115,6 @@ protected:
         }
     }
 
-    uint32_t onGetFlags() const SK_OVERRIDE {
-        // fontdescriptors (and therefore serialization) don't yet understand
-        // these new styles, so skip tests that exercise that for now.
-
-        // If certain fonts are picked up (e.g. Microsoft Jhenghei 20MB for Regular, 12MB for Bold),
-        // the resulting pdf can be ~700MB and crashes Chrome's PDF viewer.
-
-        return kSkipPicture_Flag | kSkipPipe_Flag | kSkipPDF_Flag;
-    }
-
 private:
     SkAutoTUnref<SkFontMgr> fFM;
     SkString fName;
@@ -214,12 +204,6 @@ protected:
         this->iterateFamily(canvas, paint, fset);
     }
 
-    uint32_t onGetFlags() const SK_OVERRIDE {
-        // fontdescriptors (and therefore serialization) don't yet understand
-        // these new styles, so skip tests that exercise that for now.
-        return kSkipPicture_Flag | kSkipPipe_Flag;
-    }
-
 private:
     typedef GM INHERITED;
 };
@@ -299,17 +283,7 @@ protected:
             }
         }
     }
-    
-    uint32_t onGetFlags() const SK_OVERRIDE {
-        // fontdescriptors (and therefore serialization) don't yet understand
-        // these new styles, so skip tests that exercise that for now.
-        
-        // If certain fonts are picked up (e.g. Microsoft Jhenghei 20MB for Regular, 12MB for Bold),
-        // the resulting pdf can be ~700MB and crashes Chrome's PDF viewer.
-        
-        return kSkipPicture_Flag | kSkipPipe_Flag | kSkipPDF_Flag;
-    }
-    
+
 private:
     SkAutoTUnref<SkFontMgr> fFM;
     SkString fName;
index 99af04e..3fefd56 100644 (file)
@@ -19,9 +19,6 @@ public:
     }
 
 protected:
-    uint32_t onGetFlags() const SK_OVERRIDE {
-        return kSkipTiled_Flag;
-    }
 
     SkString onShortName() SK_OVERRIDE {
         return SkString("fontscaler");
index c5e01b6..db50969 100644 (file)
@@ -16,9 +16,6 @@ public:
     GetPosTextPathGM() {}
 
 protected:
-    uint32_t onGetFlags() const SK_OVERRIDE {
-        return kSkipTiled_Flag;
-    }
 
     SkString onShortName() SK_OVERRIDE {
         return SkString("getpostextpath");
index d382269..2ac952d 100644 (file)
@@ -71,16 +71,6 @@ public:
     }
 
 protected:
-    uint32_t onGetFlags() const SK_OVERRIDE {
-#ifdef SK_BUILD_FOR_ANDROID
-        return kSkipTiled_Flag;
-#else
-        if (fDoFilter && fDoRotate && fMode != SkShader::kClamp_TileMode) {
-            return kSkipTiled_Flag;
-        }
-        return 0;
-#endif
-    }
 
     SkString onShortName() SK_OVERRIDE {
         SkString str("giantbitmap_");
index 893de13..3f1ea35 100644 (file)
@@ -27,9 +27,6 @@ public:
         }
 
 protected:
-    uint32_t onGetFlags() const SK_OVERRIDE {
-        return kSkipTiled_Flag;
-    }
 
     SkString onShortName() SK_OVERRIDE {
         SkString str("glyph_pos");
index ef2eae2..0314760 100644 (file)
@@ -21,9 +21,6 @@ namespace skiagm {
 
 class GlyphPosAlignGM : public GM {
 protected:
-    uint32_t onGetFlags() const SK_OVERRIDE {
-        return kSkipTiled_Flag;
-    }
 
     SkString onShortName() SK_OVERRIDE {
         return SkString("glyph_pos_align");
diff --git a/gm/gm.h b/gm/gm.h
index 55820ac..b474729 100644 (file)
--- a/gm/gm.h
+++ b/gm/gm.h
@@ -40,22 +40,6 @@ namespace skiagm {
         GM();
         virtual ~GM();
 
-        enum Flags {
-            kSkipPDF_Flag               = 1 << 0,
-            kSkipPicture_Flag           = 1 << 1,
-            kSkipPipe_Flag              = 1 << 2,
-            kSkipPipeCrossProcess_Flag  = 1 << 3,
-            kSkipTiled_Flag             = 1 << 4,
-            kSkip565_Flag               = 1 << 5,
-            kSkipScaledReplay_Flag      = 1 << 6,
-            kSkipGPU_Flag               = 1 << 7,
-            kSkipPDFRasterization_Flag  = 1 << 8,
-
-            kGPUOnly_Flag               = 1 << 9,
-
-            kNoBBH_Flag                 = 1 << 11, // May draw wrong using a bounding-box hierarchy
-        };
-
         enum Mode {
             kGM_Mode,
             kSample_Mode,
@@ -74,10 +58,6 @@ namespace skiagm {
 
         virtual bool runAsBench() const { return false; }
 
-        uint32_t getFlags() const {
-            return this->onGetFlags();
-        }
-
         SkScalar width() {
             return SkIntToScalar(this->getISize().width());
         }
@@ -118,7 +98,7 @@ namespace skiagm {
         virtual void onDrawBackground(SkCanvas*);
         virtual SkISize onISize() = 0;
         virtual SkString onShortName() = 0;
-        virtual uint32_t onGetFlags() const { return 0; }
+
         virtual SkMatrix onGetInitialTransform() const { return SkMatrix::I(); }
 
     private:
index 94f40cb..daeea37 100644 (file)
@@ -120,9 +120,6 @@ public:
     }
 
 protected:
-    uint32_t onGetFlags() const SK_OVERRIDE {
-        return kSkipTiled_Flag;
-    }
 
     SkString onShortName() SK_OVERRIDE {
         return SkString("gradient_matrix");
index 98093ab..43f8c94 100644 (file)
@@ -105,9 +105,6 @@ public:
     }
 
 protected:
-    uint32_t onGetFlags() const SK_OVERRIDE {
-        return kSkipTiled_Flag;
-    }
 
     SkString onShortName() {
         return SkString("gradients");
@@ -162,9 +159,6 @@ public:
     }
 
 protected:
-    uint32_t onGetFlags() const SK_OVERRIDE {
-        return kSkipTiled_Flag;
-    }
 
     SkString onShortName() {
         return SkString("gradients_local_perspective");
@@ -334,9 +328,6 @@ public:
     RadialGradientGM() {}
 
 protected:
-    uint32_t onGetFlags() const SK_OVERRIDE {
-        return kSkipTiled_Flag;
-    }
 
     SkString onShortName() SK_OVERRIDE { return SkString("radial_gradient"); }
     SkISize onISize() SK_OVERRIDE { return SkISize::Make(1280, 1280); }
@@ -377,9 +368,6 @@ public:
     RadialGradient2GM() {}
 
 protected:
-    uint32_t onGetFlags() const SK_OVERRIDE {
-        return kSkipTiled_Flag;
-    }
 
     SkString onShortName() SK_OVERRIDE { return SkString("radial_gradient2"); }
     SkISize onISize() SK_OVERRIDE { return SkISize::Make(800, 400); }
index 789137b..c180209 100644 (file)
@@ -85,9 +85,6 @@ public:
     }
 
 protected:
-    uint32_t onGetFlags() const SK_OVERRIDE {
-        return kSkipTiled_Flag;
-    }
 
     SkString onShortName() SK_OVERRIDE { return SkString("gradients_no_texture"); }
     SkISize onISize() SK_OVERRIDE { return SkISize::Make(640, 615); }
@@ -211,9 +208,6 @@ public:
     GradientsManyColorsGM() {}
 
 protected:
-    uint32_t onGetFlags() const SK_OVERRIDE {
-        return kSkipTiled_Flag;
-    }
 
     SkString onShortName() SK_OVERRIDE { return SkString("gradients_many"); }
     SkISize onISize() SK_OVERRIDE { return SkISize::Make(850, 100); }
index f2123df..87c3b38 100644 (file)
@@ -106,10 +106,6 @@ protected:
         return SkString("gradtext");
     }
 
-    uint32_t onGetFlags() const SK_OVERRIDE {
-        return kSkipTiled_Flag;
-    }
-
     SkISize onISize() SK_OVERRIDE { return SkISize::Make(500, 480); }
 
     static void draw_text(SkCanvas* canvas, const SkPaint& paint) {
index 286cc82..8b0510c 100644 (file)
@@ -13,9 +13,6 @@ namespace skiagm {
 
 class HairlinesGM : public GM {
 protected:
-    uint32_t onGetFlags() const SK_OVERRIDE {
-        return kSkipTiled_Flag;
-    }
 
 
     SkString onShortName() SK_OVERRIDE {
index 46be078..82a55ea 100644 (file)
@@ -121,9 +121,6 @@ namespace skiagm {
             }
         }
 
-        // disable pdf for now, since it crashes on mac
-        virtual uint32_t onGetFlags() const { return kSkipPDF_Flag | kSkipTiled_Flag; }
-
     private:
         typedef GM INHERITED;
     };
index 36a7cb4..11fe834 100644 (file)
@@ -34,9 +34,6 @@ public:
     HitTestPathGM () {}
 
 protected:
-    uint32_t onGetFlags() const SK_OVERRIDE {
-        return kSkipTiled_Flag;
-    }
 
     SkString onShortName() SK_OVERRIDE {
         return SkString("hittestpath");
index d980cb7..ca28dd1 100644 (file)
@@ -188,10 +188,6 @@ protected:
         }
     }
 
-    uint32_t onGetFlags() const SK_OVERRIDE {
-        return GM::kSkipPicture_Flag | GM::kSkipPipe_Flag;
-    }
-
 private:
     typedef skiagm::GM INHERITED;
 };
index b225994..a6b00e4 100644 (file)
@@ -21,13 +21,6 @@ public:
     }
 
 protected:
-    uint32_t onGetFlags() const SK_OVERRIDE {
-        return this->INHERITED::onGetFlags() |
-               GM::kSkipTiled_Flag |
-               GM::kSkipPicture_Flag |
-               GM::kSkipPipe_Flag |
-               GM::kSkipPipeCrossProcess_Flag;
-    }
 
     SkString onShortName() SK_OVERRIDE {
         return SkString("imagealphathreshold");
index df1e8a2..e424fbf 100644 (file)
@@ -23,9 +23,6 @@ public:
     }
 
 protected:
-    uint32_t onGetFlags() const SK_OVERRIDE {
-        return kSkipTiled_Flag;
-    }
 
     SkString onShortName() SK_OVERRIDE {
         return fName;
index aafd01e..978be7a 100644 (file)
@@ -36,9 +36,6 @@ public:
     }
 
 protected:
-    uint32_t onGetFlags() const SK_OVERRIDE {
-        return kSkipTiled_Flag;
-    }
 
     SkString onShortName() SK_OVERRIDE {
         return fName;
index be8d402..ea32769 100644 (file)
@@ -207,14 +207,6 @@ protected:
         canvas->drawRect(r, paint);
     }
 
-    virtual uint32_t onGetFlags() const {
-        // Because of the use of drawSprite, this test is excluded
-        // from scaled replay tests because drawSprite ignores the
-        // reciprocal scale that is applied at record time, which is
-        // the intended behavior of drawSprite.
-        return kSkipScaledReplay_Flag | kSkipTiled_Flag;
-    }
-
     virtual void onDraw(SkCanvas* canvas) {
         void (*drawProc[])(SkCanvas*, const SkRect&, SkImageFilter*) = {
             draw_paint,
index cfe9327..beb2e09 100644 (file)
@@ -31,9 +31,6 @@ public:
     }
 
 protected:
-    uint32_t onGetFlags() const SK_OVERRIDE {
-        return kSkipTiled_Flag;
-    }
 
     SkString onShortName() SK_OVERRIDE {
         return SkString("imagefiltersclipped");
index ad8fd5c..49b0a97 100644 (file)
@@ -107,14 +107,6 @@ protected:
         canvas->drawRect(r, paint);
     }
 
-    virtual uint32_t onGetFlags() const {
-        // Because of the use of drawSprite, this test is excluded
-        // from scaled replay tests because drawSprite ignores the
-        // reciprocal scale that is applied at record time, which is
-        // the intended behavior of drawSprite.
-        return kSkipScaledReplay_Flag | kSkipTiled_Flag;
-    }
-
     virtual void onDraw(SkCanvas* canvas) {
         void (*drawProc[])(SkCanvas*, const SkRect&, SkImageFilter*) = {
             draw_sprite, draw_bitmap, draw_path, draw_paint, draw_text
index c66ee83..4fa59a8 100644 (file)
@@ -102,9 +102,6 @@ public:
     ImageFiltersGraphGM() {}
 
 protected:
-    uint32_t onGetFlags() const SK_OVERRIDE {
-        return kSkipTiled_Flag;
-    }
 
     virtual SkString onShortName() {
         return SkString("imagefiltersgraph");
index d68f8cf..b419187 100644 (file)
@@ -31,9 +31,6 @@ public:
     }
 
 protected:
-    uint32_t onGetFlags() const SK_OVERRIDE {
-        return kSkipTiled_Flag;
-    }
 
     virtual SkString onShortName() {
         return SkString("imagefiltersscaled");
index feac4c1..30be060 100644 (file)
@@ -21,10 +21,6 @@ public:
     }
 
 protected:
-    uint32_t onGetFlags() const SK_OVERRIDE {
-        // Skip tiled drawing until https://code.google.com/p/skia/issues/detail?id=781 is fixed.
-        return this->INHERITED::onGetFlags() | GM::kSkipTiled_Flag;
-    }
 
     SkString onShortName() SK_OVERRIDE {
         return SkString("imagemagnifier");
index e8b46c4..6eb254c 100644 (file)
@@ -22,7 +22,6 @@ public:
     }
 
 protected:
-    uint32_t onGetFlags() const SK_OVERRIDE { return kNoBBH_Flag; }
 
     SkString onShortName() SK_OVERRIDE {
         return SkString("imageresizetiled");
index 75edaaa..87a977a 100644 (file)
@@ -70,9 +70,6 @@ public:
     }
 
 protected:
-    uint32_t onGetFlags() const SK_OVERRIDE {
-        return kSkipTiled_Flag;
-    }
 
     SkString onShortName() SK_OVERRIDE {
         return SkString("inverse_paths");
index b207989..1bcc346 100644 (file)
@@ -158,10 +158,6 @@ protected:
 
     SkISize onISize() SK_OVERRIDE { return SkISize::Make(230, 230); }
 
-    uint32_t onGetFlags() const SK_OVERRIDE {
-        return kSkip565_Flag;
-    }
-
     void onOnceBeforeDraw() SK_OVERRIDE {
         fInfo = SkImageInfo::MakeN32Premul(100, 100);
         SkPictureRecorder recorder;
index 4952256..7abfbce 100644 (file)
@@ -20,9 +20,6 @@ public:
     }
 
 protected:
-    uint32_t onGetFlags() const SK_OVERRIDE {
-        return kSkipTiled_Flag;
-    }
 
     SkString onShortName() SK_OVERRIDE {
         return SkString("lighting");
index 26e7a21..4fb6641 100644 (file)
@@ -16,9 +16,6 @@ public:
     LinePathGM() {}
 
 protected:
-    uint32_t onGetFlags() const SK_OVERRIDE {
-        return kSkipTiled_Flag;
-    }
 
     SkString onShortName() {
         return SkString("linepath");
index 88b2924..e83f327 100644 (file)
@@ -98,9 +98,6 @@ public:
     }
 
 protected:
-    uint32_t onGetFlags() const SK_OVERRIDE {
-        return kSkipTiled_Flag;
-    }
 
     SkString onShortName() SK_OVERRIDE {
         return SkString("lumafilter");
index 56bb0cd..5b1ede3 100644 (file)
@@ -19,9 +19,6 @@ public:
     }
 
 protected:
-    uint32_t onGetFlags() const SK_OVERRIDE {
-        return kSkipTiled_Flag;
-    }
 
     virtual SkString onShortName() {
         return SkString("matrixconvolution");
index 331d1ec..f818719 100644 (file)
@@ -164,11 +164,6 @@ protected:
         canvas->restore();
     }
 
-    virtual uint32_t onGetFlags() const {
-        // Skip PDF rasterization since rendering this PDF takes forever.
-        return kSkipPDFRasterization_Flag | kSkipTiled_Flag;
-    }
-
 private:
     enum {
         kNumShapes = 100,
index 15b57e3..97a5ffc 100644 (file)
@@ -534,7 +534,6 @@ namespace skiagm {
             return name;
         }
 
-        uint32_t onGetFlags() const SK_OVERRIDE { return kSkipTiled_Flag; }
         bool runAsBench() const SK_OVERRIDE { return true; }
 
     private:
index 5c7b09c..025bdce 100644 (file)
@@ -19,9 +19,6 @@ public:
     }
 
 protected:
-    uint32_t onGetFlags() const SK_OVERRIDE {
-        return kSkipTiled_Flag;
-    }
 
     SkString onShortName() SK_OVERRIDE {
         SkString name("nested");
index 189daff..bf49ca0 100644 (file)
@@ -32,9 +32,6 @@ public:
     };
 
 protected:
-    uint32_t onGetFlags() const SK_OVERRIDE {
-        return kSkipTiled_Flag;
-    }
 
     SkString onShortName() SK_OVERRIDE {
         return SkString("nonclosedpaths");
index a4232aa..7af996e 100755 (executable)
@@ -26,9 +26,6 @@ public:
     }
 
 protected:
-    uint32_t onGetFlags() const SK_OVERRIDE {
-        return kSkipTiled_Flag;
-    }
 
     SkString onShortName() SK_OVERRIDE {
         return SkString("ovals");
index 3a09ad8..61a6cad 100644 (file)
@@ -84,10 +84,6 @@ protected:
         return SkISize::Make(800, 800);
     }
 
-    uint32_t onGetFlags() const SK_OVERRIDE {
-        return kSkipTiled_Flag;
-    }
-
     void onDraw(SkCanvas* canvas) SK_OVERRIDE {
 
         SkPaint paint;
index aefa20b..7c0ee9c 100644 (file)
@@ -72,10 +72,6 @@ protected:
         return SkISize::Make(800, 800);
     }
 
-    uint32_t onGetFlags() const SK_OVERRIDE {
-        return kSkipTiled_Flag;
-    }
-
     void onDraw(SkCanvas* canvas) SK_OVERRIDE {
 
         SkPaint paint;
index b24bd90..c51ac12 100644 (file)
@@ -107,9 +107,6 @@ public:
     PathEffectGM() {}
 
 protected:
-    uint32_t onGetFlags() const SK_OVERRIDE {
-        return kSkipTiled_Flag;
-    }
 
     SkString onShortName() SK_OVERRIDE {
         return SkString("patheffect");
index b4e5a02..6b37c66 100644 (file)
@@ -125,9 +125,6 @@ public:
     }
 
 protected:
-    uint32_t onGetFlags() const SK_OVERRIDE {
-        return kSkipTiled_Flag;
-    }
 
     SkString onShortName() SK_OVERRIDE {
         return SkString("pathfill");
@@ -163,9 +160,6 @@ public:
     }
 
 protected:
-    uint32_t onGetFlags() const SK_OVERRIDE {
-        return kSkipTiled_Flag;
-    }
 
     SkString onShortName() SK_OVERRIDE {
         return SkString("pathinvfill");
index dbd58e5..f304467 100644 (file)
@@ -68,9 +68,6 @@ public:
     }
 
 protected:
-    uint32_t onGetFlags() const SK_OVERRIDE {
-        return kSkipTiled_Flag;
-    }
 
     SkString onShortName() SK_OVERRIDE {
         return SkString("path-reverse");
index 24e509c..3a881ff 100644 (file)
@@ -64,12 +64,6 @@ protected:
         }
     }
 
-    uint32_t onGetFlags() const SK_OVERRIDE {
-        // we explicitly test peekPixels and readPixels, neither of which
-        // return something for a picture-backed canvas, so we skip that test.
-        return kSkipPicture_Flag;
-    }
-
 private:
     typedef skiagm::GM INHERITED;
 };
index eae758d..6f05853 100644 (file)
@@ -101,15 +101,6 @@ protected:
         }
     }
 
-    // SkPictureImageFilter doesn't support serialization yet.
-    uint32_t onGetFlags() const SK_OVERRIDE {
-        return kSkipPicture_Flag            |
-               kSkipPipe_Flag               |
-               kSkipPipeCrossProcess_Flag   |
-               kSkipTiled_Flag              |
-               kSkipScaledReplay_Flag;
-    }
-
 private:
     SkAutoTUnref<SkPicture> fPicture;
     typedef GM INHERITED;
index 461f7e9..eccf6a6 100644 (file)
@@ -42,9 +42,6 @@ public:
     }
 
 protected:
-    uint32_t onGetFlags() const SK_OVERRIDE {
-        return kSkipTiled_Flag;
-    }
 
     SkString onShortName() SK_OVERRIDE {
         return SkString("pictureshader");
index e2acc9e..46a4386 100644 (file)
@@ -82,9 +82,6 @@ static void draw_scene(SkCanvas* canvas, SkScalar pictureSize) {
 
 class PictureShaderTileGM : public skiagm::GM {
 protected:
-    uint32_t onGetFlags() const SK_OVERRIDE {
-        return kSkipTiled_Flag;
-    }
 
     SkString onShortName() SK_OVERRIDE {
         return SkString("pictureshadertile");
index 58233dd..a634f3b 100644 (file)
@@ -15,9 +15,6 @@ public:
     PointsGM() {}
 
 protected:
-    uint32_t onGetFlags() const SK_OVERRIDE {
-        return kSkipTiled_Flag;
-    }
 
     SkString onShortName() SK_OVERRIDE {
         return SkString("points");
index 728650e..823616e 100644 (file)
@@ -179,9 +179,6 @@ public:
     Poly2PolyGM() {}
 
 protected:
-    uint32_t onGetFlags() const SK_OVERRIDE {
-        return kSkipTiled_Flag;
-    }
 
     SkString onShortName() SK_OVERRIDE {
         return SkString("poly2poly");
index e8d8504..1751ead 100644 (file)
@@ -22,9 +22,6 @@ public:
     PolygonsGM() {}
 
 protected:
-    uint32_t onGetFlags() const SK_OVERRIDE {
-        return kSkipTiled_Flag;
-    }
 
     SkString onShortName() SK_OVERRIDE {
         return SkString("polygons");
index 6d4f517..d9448b6 100644 (file)
@@ -16,9 +16,6 @@ public:
     QuadPathGM() {}
 
 protected:
-    uint32_t onGetFlags() const SK_OVERRIDE {
-        return kSkipTiled_Flag;
-    }
 
     SkString onShortName() SK_OVERRIDE {
         return SkString("quadpath");
@@ -164,9 +161,6 @@ public:
     QuadClosePathGM() {}
 
 protected:
-    uint32_t onGetFlags() const SK_OVERRIDE {
-        return kSkipTiled_Flag;
-    }
 
     SkString onShortName() SK_OVERRIDE {
         return SkString("quadclosepath");
index e0b25ae..686f03d 100644 (file)
@@ -25,9 +25,6 @@ public:
     }
 
 protected:
-    uint32_t onGetFlags() const SK_OVERRIDE {
-        return kSkipTiled_Flag;
-    }
 
     SkString onShortName() SK_OVERRIDE {
         return SkString("rects");
index 351421e..e87dff0 100644 (file)
@@ -25,13 +25,6 @@ protected:
         return SkString("resizeimagefilter");
     }
 
-#ifdef SK_CPU_ARM64
-    // Skip tiled drawing on 64-bit ARM until https://skbug.com/2908 is fixed.
-    uint32_t onGetFlags() const SK_OVERRIDE {
-        return kSkipTiled_Flag;
-    }
-#endif
-
     void draw(SkCanvas* canvas,
               const SkRect& rect,
               const SkSize& deviceSize,
index 8d81d16..b012e51 100644 (file)
@@ -27,9 +27,6 @@ public:
     }
 
 protected:
-    uint32_t onGetFlags() const SK_OVERRIDE {
-        return kSkipTiled_Flag;
-    }
 
     SkString onShortName() SK_OVERRIDE {
         return SkString("roundrects");
index 9bc1995..f174c96 100644 (file)
@@ -126,9 +126,6 @@ public:
     RRectGM() {}
 
 protected:
-    uint32_t onGetFlags() const SK_OVERRIDE {
-        return kSkipTiled_Flag;
-    }
 
     SkString onShortName() SK_OVERRIDE {
         return SkString("rrect");
index 26d9194..046e5e6 100644 (file)
@@ -56,14 +56,6 @@ protected:
 
     SkISize onISize() SK_OVERRIDE { return SkISize::Make(kImageWidth, kImageHeight); }
 
-    uint32_t onGetFlags() const SK_OVERRIDE {
-        if (kEffect_Type == fType) {
-            return kGPUOnly_Flag | kSkipTiled_Flag;
-        } else {
-            return kSkipTiled_Flag;
-        }
-    }
-
     void onDraw(SkCanvas* canvas) SK_OVERRIDE {
 #if SK_SUPPORT_GPU
         GrRenderTarget* rt = canvas->internal_private_accessTopLayerRenderTarget();
index 51dfde2..7f77463 100644 (file)
@@ -28,9 +28,6 @@ public:
     }
 
 protected:
-    uint32_t onGetFlags() const SK_OVERRIDE {
-        return kSkipTiled_Flag;
-    }
 
     SkString onShortName() SK_OVERRIDE {
         return SkString("gpusamplerstress");
index a332c5d..332a0cd 100644 (file)
@@ -16,8 +16,8 @@
 
 class SelfTestGM : public skiagm::GM {
 public:
-    SelfTestGM(const char name[], SkColor color, uint32_t flags) :
-        fName(name), fColor(color), fFlags(flags) {}
+    SelfTestGM(const char name[], SkColor color) :
+        fName(name), fColor(color) {}
     const static int kWidth = 300;
     const static int kHeight = 200;
 
@@ -35,12 +35,9 @@ protected:
         canvas->drawRectCoords(0, 0, SkIntToScalar(kWidth), SkIntToScalar(kHeight), paint);
     }
 
-    virtual uint32_t onGetFlags() const { return fFlags; }
-
 private:
     const SkString fName;
     const SkColor fColor;
-    const uint32_t fFlags;
 };
 
 //////////////////////////////////////////////////////////////////////////////
@@ -52,10 +49,10 @@ static SkColor kTranslucentGreen = 0x7700EE00;
 static SkColor kTranslucentBlue  = 0x770000DD;
 
 static skiagm::GM* F1(void*) {
-    return new SelfTestGM("selftest1", kTranslucentGreen, 0);
+    return new SelfTestGM("selftest1", kTranslucentGreen);
 }
 static skiagm::GM* F2(void*) {
-    return new SelfTestGM("selftest2", kTranslucentBlue,  skiagm::GM::kSkipPipe_Flag);
+    return new SelfTestGM("selftest2", kTranslucentBlue);
 }
 
 static skiagm::GMRegistry gR1(F1);
index e6fcc4e..a7de479 100644 (file)
@@ -34,9 +34,6 @@ public:
     }
 
 protected:
-    uint32_t onGetFlags() const SK_OVERRIDE {
-        return kSkipTiled_Flag;
-    }
 
     SkString onShortName() SK_OVERRIDE {
         return fName;
index f51aba2..fa8e2b5 100644 (file)
@@ -91,9 +91,6 @@ public:
     }
 
 protected:
-    uint32_t onGetFlags() const SK_OVERRIDE {
-        return kSkipTiled_Flag;
-    }
 
     SkString onShortName() SK_OVERRIDE {
         return SkString("shadertext");
index 23e42ad..31d1e89 100644 (file)
@@ -199,11 +199,6 @@ protected:
         }
     }
 
-    uint32_t onGetFlags() const SK_OVERRIDE {
-        // disable 565 for now, til mike fixes the debug assert
-        return kSkip565_Flag | kSkipTiled_Flag;
-    }
-
 private:
     typedef GM INHERITED;
 };
index 77a9447..7b84f69 100644 (file)
@@ -52,9 +52,6 @@ public:
     }
 
 protected:
-    uint32_t onGetFlags() const SK_OVERRIDE {
-        return kSkipTiled_Flag;
-    }
 
     SkString onShortName() SK_OVERRIDE {
         return SkString("shadertext3");
index 6cd2ed6..2b7dfbd 100644 (file)
@@ -42,12 +42,6 @@ public:
     }
 
 protected:
-    uint32_t onGetFlags() const SK_OVERRIDE {
-        if (fName.contains("linear") || fName.contains("radial")) {
-            return kSkipTiled_Flag;
-        }
-        return 0;
-    }
 
     SkString onShortName() SK_OVERRIDE {
         return fName;
index 4efff3c..eeb4b73 100644 (file)
@@ -62,9 +62,6 @@ public:
     SpriteBitmapGM() {}
 
 protected:
-    uint32_t onGetFlags() const SK_OVERRIDE {
-        return kSkipTiled_Flag;
-    }
 
     SkString onShortName() SK_OVERRIDE {
         return SkString("spritebitmap");
index 050d78f..fe69626 100644 (file)
@@ -23,9 +23,6 @@ public:
     StringArtGM() {}
 
 protected:
-    uint32_t onGetFlags() const SK_OVERRIDE {
-        return kSkipTiled_Flag;
-    }
 
     SkString onShortName() SK_OVERRIDE {
         return SkString("stringart");
index 836feda..256c7a2 100644 (file)
@@ -19,9 +19,6 @@ public:
     }
 
 protected:
-    uint32_t onGetFlags() const SK_OVERRIDE {
-        return kSkipTiled_Flag;
-    }
 
     SkString onShortName() SK_OVERRIDE {
         return SkString("stroke-fill");
index 3fcb42c..da551e7 100644 (file)
@@ -46,9 +46,6 @@ public:
     StrokeRectGM() {}
 
 protected:
-    uint32_t onGetFlags() const SK_OVERRIDE {
-        return kSkipTiled_Flag;
-    }
 
     SkString onShortName() SK_OVERRIDE {
         return SkString("strokerect");
index dea8e8c..5609c5f 100644 (file)
@@ -25,9 +25,6 @@ public:
     StrokeRectsGM() {}
 
 protected:
-    uint32_t onGetFlags() const SK_OVERRIDE {
-        return kSkipTiled_Flag;
-    }
 
     SkString onShortName() SK_OVERRIDE {
         return SkString("strokerects");
index 1a6937e..1039769 100644 (file)
@@ -39,9 +39,6 @@ public:
     StrokesGM() {}
 
 protected:
-    uint32_t onGetFlags() const SK_OVERRIDE {
-        return kSkipTiled_Flag;
-    }
 
     SkString onShortName() SK_OVERRIDE {
         return SkString("strokes_round");
@@ -95,9 +92,6 @@ public:
     }
 
 protected:
-    uint32_t onGetFlags() const SK_OVERRIDE {
-        return kSkipTiled_Flag;
-    }
 
     SkString onShortName() SK_OVERRIDE {
         return SkString("strokes_poly");
@@ -196,9 +190,6 @@ public:
     Strokes3GM() {}
 
 protected:
-    uint32_t onGetFlags() const SK_OVERRIDE {
-        return kSkipTiled_Flag;
-    }
 
     SkString onShortName() SK_OVERRIDE {
         return SkString("strokes3");
index 082ae54..914462c 100644 (file)
@@ -71,9 +71,6 @@ public:
     StrokeTextGM() {}
 
 protected:
-    uint32_t onGetFlags() const SK_OVERRIDE {
-        return kSkipTiled_Flag;
-    }
 
     SkString onShortName() SK_OVERRIDE {
         return SkString("stroketext");
index db55ddd..06b2335 100644 (file)
@@ -91,11 +91,6 @@ protected:
             canvas->drawBitmapRect(bmp, &subRect, dstRect, &paint);
             canvas->translate(SkIntToScalar(bmp.width() + 10), 0);
         }
-    }    
-
-    uint32_t onGetFlags() const SK_OVERRIDE {
-        // This GM causes issues in replay modes.
-        return kSkipTiled_Flag | kNoBBH_Flag | kSkipPicture_Flag;
     }
 
 private:
index c77f83c..f52e48b 100644 (file)
@@ -359,10 +359,6 @@ protected:
         fProc(canvas, fShowGL, fFlags);
     }
 
-    uint32_t onGetFlags() const SK_OVERRIDE {
-        return  kSkipPDF_Flag | kSkipPicture_Flag | kSkipPipe_Flag | kSkipTiled_Flag;
-    }
-
 private:
     typedef skiagm::GM INHERITED;
 };
index 82fa833..b85c0aa 100644 (file)
@@ -101,10 +101,6 @@ protected:
         return SkString("testimagefilters");
     }
 
-    uint32_t onGetFlags() const SK_OVERRIDE {
-        return kSkipTiled_Flag;
-    }
-
     SkISize onISize() SK_OVERRIDE { return SkISize::Make(700, 460); }
 
     void onDraw(SkCanvas* canvas) SK_OVERRIDE {
index 99087b5..dafd579 100644 (file)
@@ -35,8 +35,6 @@ protected:
         return SkISize::Make(2*S, 2*S);
     }
 
-    uint32_t onGetFlags() const SK_OVERRIDE { return kGPUOnly_Flag; }
-
     void onDraw(SkCanvas* canvas) SK_OVERRIDE {
         GrRenderTarget* target = canvas->internal_private_accessTopLayerRenderTarget();
         GrContext* ctx = canvas->getGrContext();
index bf851f6..0738d82 100644 (file)
@@ -73,10 +73,6 @@ protected:
                                                      SkShader::kRepeat_TileMode));
     }
 
-    uint32_t onGetFlags() const SK_OVERRIDE {
-        return kSkip565_Flag;
-    }
-
     SkString onShortName() SK_OVERRIDE {
         return SkString("textblobshader");
     }
index 7b74cec..570ec25 100644 (file)
@@ -41,11 +41,6 @@ protected:
         return SkISize::Make(SkScalarCeilToInt(canvasWidth), 800);
     }
 
-    uint32_t onGetFlags() const SK_OVERRIDE {
-        // This is a GPU-specific GM.
-        return kGPUOnly_Flag;
-    }
-
     void onOnceBeforeDraw() SK_OVERRIDE {
         fBmp.allocN32Pixels(kTargetWidth, kTargetHeight);
         SkCanvas canvas(fBmp);
index bc6c61f..6b47315 100644 (file)
@@ -39,10 +39,6 @@ protected:
         return SkISize::Make(1016, 616);
     }
 
-    uint32_t onGetFlags() const SK_OVERRIDE {
-        return kSkipTiled_Flag;
-    }
-
     static SkBitmap make_bm(int width, int height) {
         SkBitmap bm;
         bm.allocN32Pixels(width, height);
index ad8aa12..8dc54a9 100644 (file)
@@ -194,9 +194,6 @@ public:
     }
 
 protected:
-    uint32_t onGetFlags() const SK_OVERRIDE {
-        return kSkipTiled_Flag;
-    }
 
     SkString onShortName() SK_OVERRIDE {
         return fName;
index 4edd115..508e576 100644 (file)
@@ -73,13 +73,6 @@ protected:
         return name;
     }
 
-#ifdef SK_CPU_ARM64
-    // Skip tiled drawing on 64-bit ARM until https://skbug.com/2908 is fixed.
-    uint32_t onGetFlags() const SK_OVERRIDE {
-        return kSkipTiled_Flag;
-    }
-#endif
-
     SkISize onISize() SK_OVERRIDE { return SkISize::Make(880, 760); }
 
     void onOnceBeforeDraw() SK_OVERRIDE {
@@ -213,9 +206,6 @@ public:
     }
 
 protected:
-    uint32_t onGetFlags() const SK_OVERRIDE {
-        return kSkipTiled_Flag;
-    }
 
     SkString onShortName() SK_OVERRIDE {
         return fName;
index dba5d70..bbdbc8a 100644 (file)
@@ -136,11 +136,6 @@ protected:
         }
     }
 
-    uint32_t onGetFlags() const SK_OVERRIDE {
-        // The aa hairline stroked rects used to visualize the clip draw slightly differently in
-        // quilt mode in dm.
-        return kSkipTiled_Flag;
-    }
     bool runAsBench() const SK_OVERRIDE { return true; }
 
 private:
index bfeb8f5..5d15633 100644 (file)
@@ -38,9 +38,6 @@ public:
     }
 
 protected:
-    uint32_t onGetFlags() const SK_OVERRIDE {
-        return kSkipTiled_Flag;
-    }
 
     void onOnceBeforeDraw() SK_OVERRIDE {
         const SkScalar X = 150;
index 18211e9..ba355a5 100644 (file)
@@ -49,9 +49,6 @@ public:
     //SkAutoTUnref<SkTypeface> fFace;
 
 protected:
-    uint32_t onGetFlags() const SK_OVERRIDE {
-        return kSkipTiled_Flag;
-    }
 
     SkString onShortName() SK_OVERRIDE {
         return SkString("verttext");
index 9eba68b..6cb5235 100644 (file)
@@ -31,9 +31,6 @@ public:
     }
 
 protected:
-    uint32_t onGetFlags() const SK_OVERRIDE {
-        return kSkipTiled_Flag;
-    }
 
 
     SkString onShortName() SK_OVERRIDE {
index 00d1b71..1cfecd8 100644 (file)
@@ -99,18 +99,6 @@ protected:
         show_bm(canvas, veryBig, small, colors);
     }
 
-#ifdef SK_BUILD_FOR_WIN32
-    virtual uint32_t onGetFlags() const {
-        // The Windows bot runs out of memory in replay modes on this test in 32bit builds:
-        // http://skbug.com/1756
-        return kSkipPicture_Flag            |
-               kSkipPipe_Flag               |
-               kSkipPipeCrossProcess_Flag   |
-               kSkipTiled_Flag              |
-               kSkipScaledReplay_Flag;
-    }
-#endif
-
 private:
     typedef skiagm::GM INHERITED;
 };
index 21d42cb..1a2c9c9 100644 (file)
@@ -38,11 +38,6 @@ protected:
         return SkISize::Make(334, 128);
     }
 
-    uint32_t onGetFlags() const SK_OVERRIDE {
-        // This is a GPU-specific GM.
-        return kGPUOnly_Flag;
-    }
-
     void onOnceBeforeDraw() SK_OVERRIDE {
         SkImageInfo info = SkImageInfo::MakeA8(24, 24);
         fBmp[0].allocPixels(info);