commit 126d7a263625c589cb3deb29232b6b768aa88a33
author Herb Derby <herb@google.com> Wed Nov 27 11:38:16 2019 -0500
committer Skia Commit-Bot <skia-commit-bot@chromium.org> Wed Nov 27 17:12:23 2019 +0000
tree c1f296575c20fdbcced996fefcc2e423ec4048f6
parent 161c8ed5f1780b71bee006507728b5cebc42c4dd

Cleanup unit test output

Change-Id: Ic6467a505f50c9bd10c29099aa45e265a0434169
Reviewed-on: https://skia-review.googlesource.com/c/skia/+/257046
Auto-Submit: Herb Derby <herb@google.com>
Commit-Queue: Ben Wagner <bungeman@google.com>
Reviewed-by: Ben Wagner <bungeman@google.com>

tests/MultiPictureDocumentTest.cpp

diff --git a/tests/MultiPictureDocumentTest.cpp b/tests/MultiPictureDocumentTest.cpp
index 28830c0..5d422ed 100644
--- a/tests/MultiPictureDocumentTest.cpp
+++ b/tests/MultiPictureDocumentTest.cpp
@@ -170,7 +170,7 @@
     std::unique_ptr<SkStreamAsset> writtenStream = stream.detachAsStream();
     REPORTER_ASSERT(reporter, writtenStream->getLength() > 24,
         "Written data length too short (%d)", writtenStream->getLength());
-    SkDebugf("Multi Frame file size = %d\n", writtenStream->getLength());
+    // SkDebugf("Multi Frame file size = %d\n", writtenStream->getLength());
 
     // Set up deserialization
     SkSharingDeserialContext deserialContext;

tests/SkParagraphTest.cpp

diff --git a/tests/SkParagraphTest.cpp b/tests/SkParagraphTest.cpp
index dfa6d14..fc02121 100644
--- a/tests/SkParagraphTest.cpp
+++ b/tests/SkParagraphTest.cpp
@@ -22,7 +22,7 @@
 #define DEF_TEST_DISABLED(name, reporter) \
 static void test_##name(skiatest::Reporter* reporter, const GrContextOptions&); \
 skiatest::TestRegistry name##TestRegistry(skiatest::Test(#name, false, test_##name)); \
-void test_##name(skiatest::Reporter* reporter, const GrContextOptions&) { SkDebugf("Disabled: " #name "\n"); } \
+void test_##name(skiatest::Reporter* reporter, const GrContextOptions&) { /* SkDebugf("Disabled:"#name "\n"); */ } \
 void disabled_##name(skiatest::Reporter* reporter, const GrContextOptions&)
 
 using namespace skia::textlayout;
@@ -61,7 +61,7 @@
         }
 
         if (!fFontsFound) {
-            SkDebugf("Fonts not found, skipping all the tests\n");
+            // SkDebugf("Fonts not found, skipping all the tests\n");
             return;
         }
         // Only register fonts if we have to
@@ -1783,7 +1783,7 @@
 }
 
 DEF_TEST(SkParagraph_WavyDecorationParagraph, reporter) {
-    SkDebugf("TODO: Fix decorations\n");
+    // SkDebugf("TODO: Fix decorations\n");
 }
 
 // Checked: NO DIFF
@@ -4358,7 +4358,7 @@
 
 // TODO: Implement font features
 DEF_TEST(SkParagraph_FontFeaturesParagraph, reporter) {
-    SkDebugf("TODO: Font features\n");
+    // SkDebugf("TODO: Font features\n");
 }
 
 // Not in Minikin