Preprend Test to test function name generated by DEF_TEST() macro.

tests/Test.h

Index: tests/Test.h
diff --git a/tests/Test.h b/tests/Test.h
index fc1890096e8cb6fca8955d7b8c289692d302700a..46b90f7572b18601a89b78925cc2a84fc1798d19 100644
--- a/tests/Test.h
+++ b/tests/Test.h
@@ -138,38 +138,38 @@ namespace skiatest {
         (reporter)->reportFailed(desc);             \
     } while(0)
 
-#define DEF_TEST(name, reporter)                                   \
-    static void name(skiatest::Reporter*);                         \
-    namespace skiatest {                                           \
-    class name##Class : public Test {                              \
-    public:                                                        \
-        static Test* Factory(void*) { return SkNEW(name##Class); } \
-    protected:                                                     \
-        virtual void onGetName(SkString* name) SK_OVERRIDE {       \
-            name->set(#name);                                      \
-        }                                                          \
-        virtual void onRun(Reporter* r) SK_OVERRIDE { name(r); }   \
-    };                                                             \
-    static TestRegistry gReg_##name##Class(name##Class::Factory);  \
-    }                                                              \
-    static void name(skiatest::Reporter* reporter)
-
-#define DEF_GPUTEST(name, reporter, factory)                       \
-    static void name(skiatest::Reporter*, GrContextFactory*);      \
-    namespace skiatest {                                           \
-    class name##Class : public GpuTest {                           \
-    public:                                                        \
-        static Test* Factory(void*) { return SkNEW(name##Class); } \
-    protected:                                                     \
-        virtual void onGetName(SkString* name) SK_OVERRIDE {       \
-            name->set(#name);                                      \
-        }                                                          \
-        virtual void onRun(Reporter* r) SK_OVERRIDE {              \
-            name(r, fGrContextFactory);                            \
-        }                                                          \
-    };                                                             \
-    static TestRegistry gReg_##name##Class(name##Class::Factory);  \
-    }                                                              \
-    static void name(skiatest::Reporter* reporter, GrContextFactory* factory)
+#define DEF_TEST(name, reporter)                                       \
+    static void Test##name(skiatest::Reporter*);                       \

[mtklein, 2014/06/23 17:18:21]

Let's do test_##name to match the style guide for static functions?

[tfarina, 2014/06/23 22:47:32]

Done.

+    namespace skiatest {                                               \
+    class name##Class : public Test {                                  \
+    public:                                                            \
+        static Test* Factory(void*) { return SkNEW(name##Class); }     \
+    protected:                                                         \
+        virtual void onGetName(SkString* name) SK_OVERRIDE {           \
+            name->set(#name);                                          \
+        }                                                              \
+        virtual void onRun(Reporter* r) SK_OVERRIDE { Test##name(r); } \
+    };                                                                 \
+    static TestRegistry gReg_##name##Class(name##Class::Factory);      \
+    }                                                                  \
+    static void Test##name(skiatest::Reporter* reporter)
+
+#define DEF_GPUTEST(name, reporter, factory)                        \
+    static void Test##name(skiatest::Reporter*, GrContextFactory*); \
+    namespace skiatest {                                            \
+    class name##Class : public GpuTest {                            \
+    public:                                                         \
+        static Test* Factory(void*) { return SkNEW(name##Class); }  \
+    protected:                                                      \
+        virtual void onGetName(SkString* name) SK_OVERRIDE {        \
+            name->set(#name);                                       \
+        }                                                           \
+        virtual void onRun(Reporter* r) SK_OVERRIDE {               \
+            Test##name(r, fGrContextFactory);                       \
+        }                                                           \
+    };                                                              \
+    static TestRegistry gReg_##name##Class(name##Class::Factory);   \
+    }                                                               \
+    static void Test##name(skiatest::Reporter* reporter, GrContextFactory* factory)
 
 #endif