Fixing Canvas unit test to correctly verify object flattening in SkPicture
authorjunov@chromium.org <junov@chromium.org@2bbb7eff-a529-9590-31e7-b0007b416f81>
Thu, 23 Feb 2012 14:59:22 +0000 (14:59 +0000)
committerjunov@chromium.org <junov@chromium.org@2bbb7eff-a529-9590-31e7-b0007b416f81>
Thu, 23 Feb 2012 14:59:22 +0000 (14:59 +0000)
BUG=http://code.google.com/p/skia/issues/detail?id=507
TEST=Canvas unit test
REVIEW=http://codereview.appspot.com/5685082/

git-svn-id: http://skia.googlecode.com/svn/trunk@3235 2bbb7eff-a529-9590-31e7-b0007b416f81

tests/CanvasTest.cpp

index 313b9cf..099146f 100644 (file)
@@ -501,6 +501,9 @@ private:
             !referenceRecord->fPathHeap ==
             !testRecord->fPathHeap,
             testStep->assertMessage());
+        // The following tests are commented out because they currently
+        // fail. Issue: http://code.google.com/p/skia/issues/detail?id=507
+        /*
         if (referenceRecord->fPathHeap) {
             REPORTER_ASSERT_MESSAGE(reporter,
                 referenceRecord->fPathHeap->count() ==
@@ -512,6 +515,7 @@ private:
                     (*testRecord->fPathHeap)[i], testStep->assertMessage());
             }
         }
+        */
     
     }
 
@@ -583,25 +587,19 @@ public:
             kHeight);
         testStep->draw(referenceCanvas, reporter);
         SkPicture testPicture;
-        SkCanvas* testCanvas = referencePicture.beginRecording(kWidth,
+        SkCanvas* testCanvas = testPicture.beginRecording(kWidth,
             kHeight);
-        testStep->draw(referencePicture.beginRecording(kWidth, kHeight),
-            reporter);
+        testStep->draw(testCanvas, reporter);
         testStep->setAssertMessageFormat(kPictureSecondDrawAssertMessageFormat);
-        testStep->draw(referencePicture.beginRecording(kWidth, kHeight),
-            reporter);
+        testStep->draw(testCanvas, reporter);
 
         SkPictureRecord* referenceRecord = static_cast<SkPictureRecord*>(
             referenceCanvas);
         SkPictureRecord* testRecord = static_cast<SkPictureRecord*>(
             testCanvas);
-        // The following test currently fails on linux
-        // Issue: http://code.google.com/p/skia/issues/detail?id=507
-#if !defined(SK_BUILD_FOR_UNIX)
         testStep->setAssertMessageFormat(kPictureResourceReuseMessageFormat);
         AssertFlattenedObjectsEqual(referenceRecord, testRecord,
             reporter, testStep);
-#endif
     }
 };