Split out Dromaeo DOM and JSLib tests into smaller components. Will remove large tests after buildb…

chrome/test/perf/dromaeo_benchmark_uitest.cc

Index: chrome/test/perf/dromaeo_benchmark_uitest.cc
diff --git a/chrome/test/perf/dromaeo_benchmark_uitest.cc b/chrome/test/perf/dromaeo_benchmark_uitest.cc
index 71a148f8801c9e4b2377481880e66ea0fdac5bcf..6f8c175d1b525e2ed71dcefa350728e1764a0079 100644
--- a/chrome/test/perf/dromaeo_benchmark_uitest.cc
+++ b/chrome/test/perf/dromaeo_benchmark_uitest.cc
@@ -147,6 +147,34 @@ TEST_F(DromaeoTest, MAYBE_DOMCorePerf) {
   RunTest("dom");
 }
 
+TEST_F(DromaeoTest, DOMCoreAttrPerf) {
+  if (!CommandLine::ForCurrentProcess()->HasSwitch(kRunDromaeo))
+    return;
+
+  RunTest("dom-attr");
+}
+
+TEST_F(DromaeoTest, DOMCoreModifyPerf) {
+  if (!CommandLine::ForCurrentProcess()->HasSwitch(kRunDromaeo))
+    return;
+
+  RunTest("dom-modify");
+}
+
+TEST_F(DromaeoTest, DOMCoreQueryPerf) {
+  if (!CommandLine::ForCurrentProcess()->HasSwitch(kRunDromaeo))
+    return;
+
+  RunTest("dom-query");
+}
+
+TEST_F(DromaeoTest, DOMCoreTraversePerf) {
+  if (!CommandLine::ForCurrentProcess()->HasSwitch(kRunDromaeo))
+    return;
+
+  RunTest("dom-traverse");

[cmp, 2012/08/28 16:58:32]

i noticed you were able to split jslib further than just 4 tests.  is it worth
trying to split domcore further?

+}
+
 TEST_F(DromaeoTest, JSLibPerf) {
   if (!CommandLine::ForCurrentProcess()->HasSwitch(kRunDromaeo))
     return;
@@ -154,6 +182,76 @@ TEST_F(DromaeoTest, JSLibPerf) {
   RunTest("jslib");
 }
 
+TEST_F(DromaeoTest, JSLibAttrJqueryPerf) {
+  if (!CommandLine::ForCurrentProcess()->HasSwitch(kRunDromaeo))
+    return;
+
+  RunTest("jslib-attr-jquery");
+}
+
+TEST_F(DromaeoTest, JSLibAttrPrototypePerf) {
+  if (!CommandLine::ForCurrentProcess()->HasSwitch(kRunDromaeo))
+    return;
+
+  RunTest("jslib-attr-prototype");
+}
+
+TEST_F(DromaeoTest, JSLibEventJqueryPerf) {
+  if (!CommandLine::ForCurrentProcess()->HasSwitch(kRunDromaeo))
+    return;
+
+  RunTest("jslib-event-jquery");
+}
+
+TEST_F(DromaeoTest, JSLibEventPrototypePerf) {
+  if (!CommandLine::ForCurrentProcess()->HasSwitch(kRunDromaeo))
+    return;
+
+  RunTest("jslib-event-prototype");
+}
+
+TEST_F(DromaeoTest, JSLibModifyJqueryPerf) {
+  if (!CommandLine::ForCurrentProcess()->HasSwitch(kRunDromaeo))
+    return;
+
+  RunTest("jslib-modify-jquery");
+}
+
+TEST_F(DromaeoTest, JSLibModifyPrototypePerf) {
+  if (!CommandLine::ForCurrentProcess()->HasSwitch(kRunDromaeo))
+    return;
+
+  RunTest("jslib-modify-prototype");
+}
+
+TEST_F(DromaeoTest, JSLibTraverseJqueryPerf) {
+  if (!CommandLine::ForCurrentProcess()->HasSwitch(kRunDromaeo))
+    return;
+
+  RunTest("jslib-traverse-jquery");
+}
+
+TEST_F(DromaeoTest, JSLibTraversePrototypePerf) {
+  if (!CommandLine::ForCurrentProcess()->HasSwitch(kRunDromaeo))
+    return;
+
+  RunTest("jslib-traverse-prototype");
+}
+
+TEST_F(DromaeoTest, JSLibStyleJqueryPerf) {
+  if (!CommandLine::ForCurrentProcess()->HasSwitch(kRunDromaeo))
+    return;
+
+  RunTest("jslib-style-jquery");
+}
+
+TEST_F(DromaeoTest, JSLibStylePrototypePerf) {
+  if (!CommandLine::ForCurrentProcess()->HasSwitch(kRunDromaeo))
+    return;
+
+  RunTest("jslib-style-prototype");
+}
+
 TEST_F(DromaeoReferenceTest, MAYBE_DOMCorePerf) {
   if (!CommandLine::ForCurrentProcess()->HasSwitch(kRunDromaeo))
     return;
@@ -161,6 +259,34 @@ TEST_F(DromaeoReferenceTest, MAYBE_DOMCorePerf) {
   RunTest("dom");
 }
 
+TEST_F(DromaeoReferenceTest, DOMCoreAttrPerf) {
+  if (!CommandLine::ForCurrentProcess()->HasSwitch(kRunDromaeo))
+    return;
+
+  RunTest("dom-attr");
+}
+
+TEST_F(DromaeoReferenceTest, DOMCoreModifyPerf) {
+  if (!CommandLine::ForCurrentProcess()->HasSwitch(kRunDromaeo))
+    return;
+
+  RunTest("dom-modify");
+}
+
+TEST_F(DromaeoReferenceTest, DOMCoreQueryPerf) {
+  if (!CommandLine::ForCurrentProcess()->HasSwitch(kRunDromaeo))
+    return;
+
+  RunTest("dom-query");
+}
+
+TEST_F(DromaeoReferenceTest, DOMCoreTraversePerf) {
+  if (!CommandLine::ForCurrentProcess()->HasSwitch(kRunDromaeo))
+    return;
+
+  RunTest("dom-traverse");
+}
+
 TEST_F(DromaeoReferenceTest, JSLibPerf) {
   if (!CommandLine::ForCurrentProcess()->HasSwitch(kRunDromaeo))
     return;
@@ -168,5 +294,75 @@ TEST_F(DromaeoReferenceTest, JSLibPerf) {
   RunTest("jslib");
 }
 
+TEST_F(DromaeoReferenceTest, JSLibAttrJqueryPerf) {
+  if (!CommandLine::ForCurrentProcess()->HasSwitch(kRunDromaeo))
+    return;
+
+  RunTest("jslib-attr-jquery");
+}
+
+TEST_F(DromaeoReferenceTest, JSLibAttrPrototypePerf) {
+  if (!CommandLine::ForCurrentProcess()->HasSwitch(kRunDromaeo))
+    return;
+
+  RunTest("jslib-attr-prototype");
+}
+
+TEST_F(DromaeoReferenceTest, JSLibEventJqueryPerf) {
+  if (!CommandLine::ForCurrentProcess()->HasSwitch(kRunDromaeo))
+    return;
+
+  RunTest("jslib-event-jquery");
+}
+
+TEST_F(DromaeoReferenceTest, JSLibEventPrototypePerf) {
+  if (!CommandLine::ForCurrentProcess()->HasSwitch(kRunDromaeo))
+    return;
+
+  RunTest("jslib-event-prototype");
+}
+
+TEST_F(DromaeoReferenceTest, JSLibModifyJqueryPerf) {
+  if (!CommandLine::ForCurrentProcess()->HasSwitch(kRunDromaeo))
+    return;
+
+  RunTest("jslib-modify-jquery");
+}
+
+TEST_F(DromaeoReferenceTest, JSLibModifyPrototypePerf) {
+  if (!CommandLine::ForCurrentProcess()->HasSwitch(kRunDromaeo))
+    return;
+
+  RunTest("jslib-modify-prototype");
+}
+
+TEST_F(DromaeoReferenceTest, JSLibTraverseJqueryPerf) {
+  if (!CommandLine::ForCurrentProcess()->HasSwitch(kRunDromaeo))
+    return;
+
+  RunTest("jslib-traverse-jquery");
+}
+
+TEST_F(DromaeoReferenceTest, JSLibTraversePrototypePerf) {
+  if (!CommandLine::ForCurrentProcess()->HasSwitch(kRunDromaeo))

[darin (slow to review), 2012/08/28 21:04:26]

perhaps you could fold this command line check into the RunTest function to
avoid having to replicate it?

[sullivan, 2012/09/04 15:43:41]

Done.

+    return;
+
+  RunTest("jslib-traverse-prototype");
+}
+
+TEST_F(DromaeoReferenceTest, JSLibStyleJqueryPerf) {
+  if (!CommandLine::ForCurrentProcess()->HasSwitch(kRunDromaeo))
+    return;
+
+  RunTest("jslib-style-jquery");
+}
+
+TEST_F(DromaeoReferenceTest, JSLibStylePrototypePerf) {
+  if (!CommandLine::ForCurrentProcess()->HasSwitch(kRunDromaeo))
+    return;
+
+  RunTest("jslib-style-prototype");
+}
+
 
 }  // namespace