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

Unified Diff: gm/bigtileimagefilter.cpp

Issue 1152553006: Fix dst bound reported by SkTileImageFilter (Closed) Base URL: https://skia.googlesource.com/skia.git@master
Patch Set: Fix line wraps Created 5 years, 6 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 | « no previous file | include/effects/SkTileImageFilter.h » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: gm/bigtileimagefilter.cpp
diff --git a/gm/bigtileimagefilter.cpp b/gm/bigtileimagefilter.cpp
index 97de0d2042c5ced7ae6bfe0aa51e1954194560cf..f465fe4d38c0a5a67f14d207ecb2a1dc8cc21202 100644
--- a/gm/bigtileimagefilter.cpp
+++ b/gm/bigtileimagefilter.cpp
@@ -9,6 +9,19 @@
#include "SkTileImageFilter.h"
#include "gm.h"
+static void create_circle_texture(SkBitmap* bm, SkColor color) {
+ SkCanvas canvas(*bm);
+ canvas.clear(0xFF000000);
+
+ SkPaint paint;
+ paint.setColor(color);
+ paint.setStrokeWidth(3);
+ paint.setStyle(SkPaint::kStroke_Style);
+
+ canvas.drawCircle(SkScalarHalf(bm->width()), SkScalarHalf(bm->height()),
+ SkScalarHalf(bm->width()), paint);
+}
+
namespace skiagm {
class BigTileImageFilterGM : public GM {
@@ -28,35 +41,52 @@ protected:
}
void onOnceBeforeDraw() override {
- fBitmap.allocN32Pixels(kBitmapSize, kBitmapSize);
-
- SkCanvas canvas(fBitmap);
- canvas.clear(0xFF000000);
-
- SkPaint paint;
- paint.setColor(SK_ColorRED);
- paint.setStrokeWidth(3);
- paint.setStyle(SkPaint::kStroke_Style);
+ fRedBitmap.allocN32Pixels(kBitmapSize, kBitmapSize);
+ create_circle_texture(&fRedBitmap, SK_ColorRED);
- canvas.drawCircle(SkScalarHalf(kBitmapSize), SkScalarHalf(kBitmapSize),
- SkScalarHalf(kBitmapSize), paint);
+ fGreenBitmap.allocN32Pixels(kBitmapSize, kBitmapSize);
+ create_circle_texture(&fGreenBitmap, SK_ColorGREEN);
}
void onDraw(SkCanvas* canvas) override {
canvas->clear(SK_ColorBLACK);
- SkPaint p;
+ {
+ SkPaint p;
- SkAutoTUnref<SkBitmapSource> bms(SkBitmapSource::Create(fBitmap));
- SkAutoTUnref<SkTileImageFilter> tif(SkTileImageFilter::Create(
+ SkRect bound = SkRect::MakeWH(SkIntToScalar(kWidth), SkIntToScalar(kHeight));
+ SkAutoTUnref<SkBitmapSource> bms(SkBitmapSource::Create(fRedBitmap));
+ SkAutoTUnref<SkTileImageFilter> tif(SkTileImageFilter::Create(
SkRect::MakeWH(SkIntToScalar(kBitmapSize), SkIntToScalar(kBitmapSize)),
SkRect::MakeWH(SkIntToScalar(kWidth), SkIntToScalar(kHeight)),
bms));
- p.setImageFilter(tif);
+ p.setImageFilter(tif);
- SkRect bound = SkRect::MakeWH(SkIntToScalar(kWidth), SkIntToScalar(kHeight));
- canvas->saveLayer(&bound, &p);
- canvas->restore();
+ canvas->saveLayer(&bound, &p);
+ canvas->restore();
+ }
+
+ {
+ SkPaint p2;
+
+ SkRect bound2 = SkRect::MakeWH(SkIntToScalar(kBitmapSize), SkIntToScalar(kBitmapSize));
+
+ SkAutoTUnref<SkTileImageFilter> tif2(SkTileImageFilter::Create(
+ SkRect::MakeWH(SkIntToScalar(kBitmapSize), SkIntToScalar(kBitmapSize)),
+ SkRect::MakeWH(SkIntToScalar(kBitmapSize), SkIntToScalar(kBitmapSize)),
+ NULL));
+ p2.setImageFilter(tif2);
+
+ canvas->translate(320, 320);
+ canvas->saveLayer(&bound2, &p2);
+ canvas->setMatrix(SkMatrix::I());
+
+ SkRect bound3 = SkRect::MakeXYWH(320, 320,
+ SkIntToScalar(kBitmapSize),
+ SkIntToScalar(kBitmapSize));
+ canvas->drawBitmapRectToRect(fGreenBitmap, &bound2, bound3);
+ canvas->restore();
+ }
}
private:
@@ -64,7 +94,8 @@ private:
static const int kHeight = 512;
static const int kBitmapSize = 64;
- SkBitmap fBitmap;
+ SkBitmap fRedBitmap;
+ SkBitmap fGreenBitmap;
typedef GM INHERITED;
};
« no previous file with comments | « no previous file | include/effects/SkTileImageFilter.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698