Add REPORTF test macro.

tests/BitmapCopyTest.cpp

Index: tests/BitmapCopyTest.cpp
diff --git a/tests/BitmapCopyTest.cpp b/tests/BitmapCopyTest.cpp
index 455fc691e3ec004a3c8433d6eb095b37b68559b9..7028d9799a4d58f9dda33bb3f111d044d15192e2 100644
--- a/tests/BitmapCopyTest.cpp
+++ b/tests/BitmapCopyTest.cpp
@@ -21,11 +21,10 @@ static const char* gConfigName[] = {
 
 static void report_opaqueness(skiatest::Reporter* reporter, const SkBitmap& src,
                               const SkBitmap& dst) {
-    SkString str;
-    str.printf("src %s opaque:%d, dst %s opaque:%d",
-               gConfigName[src.config()], src.isOpaque(),
-               gConfigName[dst.config()], dst.isOpaque());
-    reporter->reportFailed(str);
+    REPORTF(reporter,
+            ("src %s opaque:%d, dst %s opaque:%d",
+             gConfigName[src.config()], src.isOpaque(),
+             gConfigName[dst.config()], dst.isOpaque()));
 }
 
 static bool canHaveAlpha(SkBitmap::Config config) {
@@ -192,10 +191,7 @@ static void reportCopyVerification(const SkBitmap& bm1, const SkBitmap& bm2,
     }
 
     if (!success) {
-        SkString str;
-        str.printf("%s [config = %s]",
-                   msg, getSkConfigName(bm1));
-        reporter->reportFailed(str);
+        REPORTF(reporter, ("%s [config = %s]", msg, getSkConfigName(bm1)));
     }
 }
 
@@ -247,20 +243,18 @@ DEF_TEST(BitmapCopy, reporter) {
             bool success = srcPremul.copyTo(&dst, gPairs[j].fConfig);
             bool expected = gPairs[i].fValid[j] != '0';
             if (success != expected) {
-                SkString str;
-                str.printf("SkBitmap::copyTo from %s to %s. expected %s returned %s",
-                           gConfigName[i], gConfigName[j], boolStr(expected),
-                           boolStr(success));
-                reporter->reportFailed(str);
+                REPORTF(reporter,
+                        ("SkBitmap::copyTo from %s to %s. expected %s "
+                         "returned %s", gConfigName[i], gConfigName[j],
+                         boolStr(expected), boolStr(success)));
             }
 
             bool canSucceed = srcPremul.canCopyTo(gPairs[j].fConfig);
             if (success != canSucceed) {
-                SkString str;
-                str.printf("SkBitmap::copyTo from %s to %s. returned %s canCopyTo %s",
-                           gConfigName[i], gConfigName[j], boolStr(success),
-                           boolStr(canSucceed));
-                reporter->reportFailed(str);
+                REPORTF(reporter,
+                        ("SkBitmap::copyTo from %s to %s. returned %s "
+                         "canCopyTo %s", gConfigName[i], gConfigName[j],
+                         boolStr(success), boolStr(canSucceed)));
             }
 
             if (success) {
@@ -345,10 +339,9 @@ DEF_TEST(BitmapCopy, reporter) {
                                   100000000U);
             int64_t safeSize = tstSafeSize.computeSafeSize64();
             if (safeSize < 0) {
-                SkString str;
-                str.printf("getSafeSize64() negative: %s",
-                    getSkConfigName(tstSafeSize));
-                reporter->reportFailed(str);
+                REPORTF(reporter,
+                        ("getSafeSize64() negative: %s",
+                         getSkConfigName(tstSafeSize)));
             }
             bool sizeFail = false;
             // Compare against hand-computed values.
@@ -380,10 +373,9 @@ DEF_TEST(BitmapCopy, reporter) {
                     break;
             }
             if (sizeFail) {
-                SkString str;
-                str.printf("computeSafeSize64() wrong size: %s",
-                    getSkConfigName(tstSafeSize));
-                reporter->reportFailed(str);
+                REPORTF(reporter,
+                        ("computeSafeSize64() wrong size: %s",
+                         getSkConfigName(tstSafeSize)));
             }
 
             int subW = 2;