Added the reference build to the frame rate test.

chrome/test/perf/frame_rate/frame_rate_tests.cc

 // Copyright (c) 2011 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include <map>
 
 #include "base/file_util.h"
 #include "base/path_service.h"
 #include "base/string_number_conversions.h"
 #include "base/test/test_timeouts.h"
@@ skipping 67 matching lines @@
 };
 
 class FrameRateTest_Reference : public FrameRateTest {
  public:
   void SetUp() {
     UseReferenceBuild();
     FrameRateTest::SetUp();
   }
 };
 
-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