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

chrome/test/perf/dromaeo_benchmark_uitest.cc

 // Copyright (c) 2012 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 "base/command_line.h"
 #include "base/file_path.h"
 #include "base/file_util.h"
 #include "base/path_service.h"
 #include "base/string_util.h"
 #include "base/test/test_timeouts.h"
@@ skipping 129 matching lines @@
 #else
 #define MAYBE_DOMCorePerf DOMCorePerf
 #endif
 TEST_F(DromaeoTest, MAYBE_DOMCorePerf) {
   if (!CommandLine::ForCurrentProcess()->HasSwitch(kRunDromaeo))
     return;
 
   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;
 
   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;
 
   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;
 
   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