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

Unified Diff: tests/SerializationTest.cpp

Issue 2080993002: Added API for Bevel NormalSource. (Closed) Base URL: https://skia.googlesource.com/skia@dvonbeck-diffuse-api-change
Patch Set: Small init fix Created 4 years, 5 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
« src/core/SkNormalSource.cpp ('K') | « src/core/SkNormalSource.cpp ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: tests/SerializationTest.cpp
diff --git a/tests/SerializationTest.cpp b/tests/SerializationTest.cpp
index 88f88fc2ac5bb524950ceb3677195d2bd663df8f..fbdf8cf82b9c9aae4416061b21c9d764eac682fe 100644
--- a/tests/SerializationTest.cpp
+++ b/tests/SerializationTest.cpp
@@ -595,6 +595,17 @@ DEF_TEST(Serialization, reporter) {
// TODO test equality?
}
+
+ // Test NormalBevelSource serialization
+ {
+ sk_sp<SkNormalSource> bevelSource = SkNormalSource::MakeBevel(
+ SkNormalSource::BevelType::kLinear, 2.0f, 5.0f);
+
+ SkAutoTUnref<SkNormalSource>(TestFlattenableSerialization(bevelSource.get(), true,
+ reporter));
robertphillips 2016/07/11 19:13:21 Do we need to hard cast to a NormalBevelSourceImpl
dvonbeck 2016/07/13 14:23:38 Not that easy, because serializing/deserializing c
+ // TODO test equality?
+
+ }
}
///////////////////////////////////////////////////////////////////////////////////////////////////
« src/core/SkNormalSource.cpp ('K') | « src/core/SkNormalSource.cpp ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698