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

Unified Diff: tests/CanvasTest.cpp

Issue 143883006: No deduping dictionaries for matrices and regions. (Closed) Base URL: https://skia.googlesource.com/skia.git@master
Patch Set: bump picture version Created 6 years, 11 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 | « src/effects/SkEmbossMaskFilter.cpp ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: tests/CanvasTest.cpp
diff --git a/tests/CanvasTest.cpp b/tests/CanvasTest.cpp
index 4ff842b90197f7727a76e068f2b459c057d24de7..bddba59e0680da40278969ce9f9d2cbfdfe16855 100644
--- a/tests/CanvasTest.cpp
+++ b/tests/CanvasTest.cpp
@@ -669,14 +669,6 @@ private:
referenceRecord->fBitmapHeap->count() ==
testRecord->fBitmapHeap->count(), testStep->assertMessage());
REPORTER_ASSERT_MESSAGE(reporter,
- referenceRecord->fMatrices.count() ==
- testRecord->fMatrices.count(), testStep->assertMessage());
- for (int i = 0; i < referenceRecord->fMatrices.count(); ++i) {
- REPORTER_ASSERT_MESSAGE(reporter,
- EQ(referenceRecord->fMatrices[i], testRecord->fMatrices[i]),
- testStep->assertMessage());
- }
- REPORTER_ASSERT_MESSAGE(reporter,
referenceRecord->fPaints.count() ==
testRecord->fPaints.count(), testStep->assertMessage());
for (int i = 0; i < referenceRecord->fPaints.count(); ++i) {
@@ -685,14 +677,6 @@ private:
testStep->assertMessage());
}
REPORTER_ASSERT_MESSAGE(reporter,
- referenceRecord->fRegions.count() ==
- testRecord->fRegions.count(), testStep->assertMessage());
- for (int i = 0; i < referenceRecord->fRegions.count(); ++i) {
- REPORTER_ASSERT_MESSAGE(reporter,
- EQ(referenceRecord->fRegions[i], testRecord->fRegions[i]),
- testStep->assertMessage());
- }
- REPORTER_ASSERT_MESSAGE(reporter,
!referenceRecord->fPathHeap ==
!testRecord->fPathHeap,
testStep->assertMessage());
« no previous file with comments | « src/effects/SkEmbossMaskFilter.cpp ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698