diff options
author | senorblanco <senorblanco@chromium.org> | 2015-08-03 13:04:03 -0700 |
---|---|---|
committer | Commit bot <commit-bot@chromium.org> | 2015-08-03 13:04:03 -0700 |
commit | 468dfa72eb6694145487be17876804dfca3b7adb (patch) | |
tree | 2cd289705ee10b6763616b593c6ba163fda3f43d /tests/PathTest.cpp | |
parent | 4dea94f621ee84c03535f07389fc0cb69a2a1eec (diff) |
Implement caching of filled paths in the tessellated path renderer.
Paths are cached as tessellated triangle meshes in vertex buffers on the GPU. Stroked paths are not (yet) cached.
Paths containing no curved segments (linear paths) are reused at all scales. Paths containing curved segments are reused within a scale tolerance threshold.
In order to invalidate the cache when an SkPath is changed or deleted,
this required implementing genID change notification in SkPath. This is
modelled almost exactly on SkPixelRef::GenIDChangeListener.
However, It does not currently implement the check for unique genIDs,
so notifiers will fire when the first instance of an SkPathRef
using a given genID is destroyed.
Another caveat is that you cannot successfully add a change notifier
to an empty path, since it uses the "canonical" empty path which is
never modified or destroyed. For this reason, we prevent adding
listeners to it.
BUG=skia:4121,skia:4122, 497403
DOCS_PREVIEW= https://skia.org/?cl=1114353004
Review URL: https://codereview.chromium.org/1114353004
Diffstat (limited to 'tests/PathTest.cpp')
-rw-r--r-- | tests/PathTest.cpp | 60 |
1 files changed, 60 insertions, 0 deletions
diff --git a/tests/PathTest.cpp b/tests/PathTest.cpp index 8272692167..b0bd66726d 100644 --- a/tests/PathTest.cpp +++ b/tests/PathTest.cpp @@ -3715,6 +3715,21 @@ static void test_dump(skiatest::Reporter* reporter) { "path.lineTo(3, 4);\n"); } +namespace { + +class ChangeListener : public SkPathRef::GenIDChangeListener { +public: + ChangeListener(bool *changed) : fChanged(changed) { *fChanged = false; } + virtual ~ChangeListener() {} + void onChange() override { + *fChanged = true; + } +private: + bool* fChanged; +}; + +} + class PathTest_Private { public: static void TestPathTo(skiatest::Reporter* reporter) { @@ -3734,6 +3749,50 @@ public: SkRect reverseExpected = {-4, -4, 8, 8}; REPORTER_ASSERT(reporter, p.getBounds() == reverseExpected); } + + static void TestPathrefListeners(skiatest::Reporter* reporter) { + SkPath p; + + bool changed = false; + p.moveTo(0, 0); + + // Check that listener is notified on moveTo(). + + SkPathPriv::AddGenIDChangeListener(p, SkNEW(ChangeListener(&changed))); + REPORTER_ASSERT(reporter, !changed); + p.moveTo(10, 0); + REPORTER_ASSERT(reporter, changed); + + // Check that listener is notified on lineTo(). + SkPathPriv::AddGenIDChangeListener(p, SkNEW(ChangeListener(&changed))); + REPORTER_ASSERT(reporter, !changed); + p.lineTo(20, 0); + REPORTER_ASSERT(reporter, changed); + + // Check that listener is notified on reset(). + SkPathPriv::AddGenIDChangeListener(p, SkNEW(ChangeListener(&changed))); + REPORTER_ASSERT(reporter, !changed); + p.reset(); + REPORTER_ASSERT(reporter, changed); + + p.moveTo(0, 0); + + // Check that listener is notified on rewind(). + SkPathPriv::AddGenIDChangeListener(p, SkNEW(ChangeListener(&changed))); + REPORTER_ASSERT(reporter, !changed); + p.rewind(); + REPORTER_ASSERT(reporter, changed); + + // Check that listener is notified when pathref is deleted. + { + SkPath q; + q.moveTo(10, 10); + SkPathPriv::AddGenIDChangeListener(q, SkNEW(ChangeListener(&changed))); + REPORTER_ASSERT(reporter, !changed); + } + // q went out of scope. + REPORTER_ASSERT(reporter, changed); + } }; DEF_TEST(Paths, reporter) { @@ -3880,6 +3939,7 @@ DEF_TEST(Paths, reporter) { test_contains(reporter); PathTest_Private::TestPathTo(reporter); PathRefTest_Private::TestPathRef(reporter); + PathTest_Private::TestPathrefListeners(reporter); test_dump(reporter); test_path_crbug389050(reporter); test_path_crbugskia2820(reporter); |