Preparing components_perftests

components/BUILD.gn

 # Copyright 2014 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.
 
 import("//build/config/features.gni")
 import("//build/config/ui.gni")
 import("//testing/test.gni")
 import("//tools/grit/repack.gni")
 
 if (is_android) {
@@ skipping 19 matching lines @@
     ]
   }
 }
 
 # To add a unit test to this target, make a "unit_test" source_set in your
 # component (it's important to use a source_set instead of a static library or
 # no tests will run) and add a reference here. You can add more than one unit
 # test target if convenient.
 test("components_unittests") {
   sources = [
+    "test/components_test_suite.cc",
+    "test/components_test_suite.h",

[Peter Kasting, 2016/10/17 18:44:45]

I don't think you want these sources (and their dependencies) to appear in both
this target and the one below.

You probably want these, and their dependencies, in a source_set that both
targets depend on.

     "test/run_all_unittests.cc",
   ]
 
   if (is_android || is_linux || is_mac || is_win) {
     data = [
       "test/data/",
 
       # TODO(dpranke): Remove the next two lines after GN has rolled to 339778.
       "$root_out_dir/components_tests_resources.pak",
       "$root_out_dir/ui_test.pak",
@@ skipping 90 matching lines @@
     "//components/update_client:unit_tests",
     "//components/upload_list:unit_tests",
     "//components/url_formatter:unit_tests",
     "//components/url_matcher:unit_tests",
     "//components/user_prefs/tracked:unit_tests",
     "//components/variations:unit_tests",
     "//components/variations/service:unit_tests",
     "//components/web_resource:unit_tests",
     "//components/webdata/common:unit_tests",
 
-    # Direct dependencies of components/test/run_all_unittests.cc.
+    # Direct dependencies of components/test/components_test_suite.cc.
     "//components/content_settings/core/common",
     "//mojo/edk/system",
     "//net",
     "//testing/gtest",
     "//ui/base",
     "//ui/resources:ui_test_pak",
   ]
 
   if (enable_nacl) {
     deps += [ "//components/nacl/browser:unit_tests" ]
@@ skipping 307 matching lines @@
     }
 
     if (enable_basic_printing || enable_print_preview) {
       sources += [ "printing/test/print_web_view_helper_browsertest.cc" ]
       deps += [ "//components/printing/test:test_support" ]
     }
   }
 
   test("components_perftests") {
     sources = [
+      "test/components_test_suite.cc",
+      "test/components_test_suite.h",
+      "test/run_all_perftests.cc",
       "visitedlink/test/visitedlink_perftest.cc",
     ]
 
     # TODO(jschuh): crbug.com/167187 fix size_t to int truncations.
     configs += [ "//build/config/compiler:no_size_t_to_int_warning" ]
 
     deps = [
       "//base",
-      "//base/test:test_support",
       "//components/visitedlink/browser",
       "//content/test:test_support",
-      "//mojo/edk/test:run_all_perftests",
       "//testing/gtest",
       "//testing/perf",
       "//url",
+
+      # Direct dependencies of components/test/components_test_suite.cc.
+      "//components/content_settings/core/common",
+      "//mojo/edk/system",
+      "//net",
+      "//testing/gtest",
+      "//ui/base",
+      "//ui/resources:ui_test_pak",
     ]
   }
 }