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

Unified Diff: tests/FillPathTest.cpp

Issue 2053823002: Make SkBlitter hierarchy explicit about what needs to be implemented. (Closed) Base URL: https://skia.googlesource.com/skia.git@master
Patch Set: Created 4 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 | « tests/BlitMaskClip.cpp ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: tests/FillPathTest.cpp
diff --git a/tests/FillPathTest.cpp b/tests/FillPathTest.cpp
index cc8d3296525979ce6158be256d67d8724e522314..1f14d4f7f332846ef50ac69e67c13013b82963b2 100644
--- a/tests/FillPathTest.cpp
+++ b/tests/FillPathTest.cpp
@@ -12,34 +12,37 @@
#include "Test.h"
struct FakeBlitter : public SkBlitter {
- FakeBlitter()
- : m_blitCount(0)
- {}
+ FakeBlitter()
+ : m_blitCount(0) { }
- virtual void blitH(int x, int y, int width) {
- m_blitCount++;
- }
+ void blitH(int x, int y, int width) override {
+ m_blitCount++;
+ }
- int m_blitCount;
+ void blitAntiH(int x, int y, const SkAlpha antialias[], const int16_t runs[]) override {
+ SkDEBUGFAIL("blitAntiH not implemented");
+ }
+
+ int m_blitCount;
};
// http://code.google.com/p/skia/issues/detail?id=87
// Lines which is not clipped by boundary based clipping,
// but skipped after tessellation, should be cleared by the blitter.
DEF_TEST(FillPathInverse, reporter) {
- FakeBlitter blitter;
- SkIRect clip;
- SkPath path;
- int height = 100;
- int width = 200;
- int expected_lines = 5;
- clip.set(0, height - expected_lines, width, height);
- path.moveTo(0.0f, 0.0f);
- path.quadTo(SkIntToScalar(width/2), SkIntToScalar(height),
+ FakeBlitter blitter;
+ SkIRect clip;
+ SkPath path;
+ int height = 100;
+ int width = 200;
+ int expected_lines = 5;
+ clip.set(0, height - expected_lines, width, height);
+ path.moveTo(0.0f, 0.0f);
+ path.quadTo(SkIntToScalar(width/2), SkIntToScalar(height),
SkIntToScalar(width), 0.0f);
- path.close();
- path.setFillType(SkPath::kInverseWinding_FillType);
- SkScan::FillPath(path, clip, &blitter);
+ path.close();
+ path.setFillType(SkPath::kInverseWinding_FillType);
+ SkScan::FillPath(path, clip, &blitter);
- REPORTER_ASSERT(reporter, blitter.m_blitCount == expected_lines);
+ REPORTER_ASSERT(reporter, blitter.m_blitCount == expected_lines);
}
« no previous file with comments | « tests/BlitMaskClip.cpp ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698