Added the reference build to the frame rate test.

chrome/test/perf/frame_rate/frame_rate_tests.cc

Index: chrome/test/perf/frame_rate/frame_rate_tests.cc
diff --git a/chrome/test/perf/frame_rate/frame_rate_tests.cc b/chrome/test/perf/frame_rate/frame_rate_tests.cc
index 52e7de4453113842aa73cbd559bfec545afbc535..3fa81ea0e79124448923c88d5c92b1a02b7196d7 100644
--- a/chrome/test/perf/frame_rate/frame_rate_tests.cc
+++ b/chrome/test/perf/frame_rate/frame_rate_tests.cc
@@ -85,28 +85,17 @@ class FrameRateTest_Reference : public FrameRateTest {
   }
 };
 
-TEST_F(FrameRateTest, Blank_Steady) {
-  RunTest("blank", "steady", "");
+#define FRAME_RATE_TEST(name, gesture) \
+TEST_F(FrameRateTest, name) { \
+  RunTest("blank", gesture, ""); \
+} \
+TEST_F(FrameRateTest_Reference, name) { \
+  RunTest("blank", gesture, "_ref"); \
 }
 
-TEST_F(FrameRateTest, Blank_Reading) {
-  RunTest("blank", "reading", "");
-}
-
-TEST_F(FrameRateTest, Blank_Mouse_Wheel) {
-  RunTest("blank", "mouse_wheel", "");
-}
-
-TEST_F(FrameRateTest, Blank_Mac_Fling) {
-  RunTest("blank", "mac_fling", "");
-}
-
-// TODO(darin): Need to update the reference build to a version that supports
-// the webkitRequestAnimationFrame API.
-#if 0
-TEST_F(FrameRateTest_Reference, Blank) {
-  RunTest("blank", "_ref");
-}
-#endif
+FRAME_RATE_TEST(Blank_Steady, "steady");

[darin (slow to review), 2011/07/07 23:51:02]

eventually, i imagine we'll have more than just the "blank" page to run tests
on, and we may also want frame rate tests that are about more than just
scrolling.

it might be nice to rework this so that you can write:

  FRAME_RATE_TEST(Blank_Steady, "blank", "steady");

it'd be sweet to go even further and do:

  FRAME_RATE_TEST(blank, steady);

this could be implemented as:

  #define FRAME_RATE_TEST(content, gesture) \
  TEST_F(FrameRateTest, content##_##gesture) { \
    RunTest(#content, #gesture, ""); \
  } ...

This creates tests named "blank_steady", but if you wanted it to be
"Blank_Steady", then you could make RunTest lowercase the input strings ;-)

[jrt, 2011/07/08 00:02:11]

Ah, much cleaner. Thanks.

+FRAME_RATE_TEST(Blank_Reading, "reading");
+FRAME_RATE_TEST(Blank_MouseWheel, "mouse_wheel");
+FRAME_RATE_TEST(Blank_MacFling, "mac_fling");
 
 }  // namespace