Skip to content

Commit

Permalink
Cleanup test_support target names.
Browse files Browse the repository at this point in the history
In GN it is redundat to include the parent directory name, as in
ui/gfx:gfx_test_support. Instead we can write it as ui/gfx:test_support
which is shorter and nicer to read.

Targets found with the following command line:
$ for f in $(g ls-files | grep BUILD); do grep _test_support $f; echo
$f; done

BUG=None
TEST=None
R=brettw@chromium.org
TBR=darin@chromium.org

Review URL: https://codereview.chromium.org/409493002

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@284800 0039d316-1c4b-4281-b951-d872f2087c98
  • Loading branch information
tfarina@chromium.org committed Jul 22, 2014
1 parent 49fef79 commit e54a1f5
Show file tree
Hide file tree
Showing 12 changed files with 36 additions and 36 deletions.
2 changes: 1 addition & 1 deletion cc/BUILD.gn
Original file line number Diff line number Diff line change
Expand Up @@ -614,7 +614,7 @@ source_set("test_support") {
#"//third_party/mesa/mesa.gyp:osmesa",
"//ui/gfx",
"//ui/gfx/geometry",
"//ui/gfx:gfx_test_support",
"//ui/gfx:test_support",
"//ui/gl",
]
}
Expand Down
2 changes: 1 addition & 1 deletion components/wifi/BUILD.gn
Original file line number Diff line number Diff line change
Expand Up @@ -37,7 +37,7 @@ component("wifi") {
}
}

source_set("wifi_test_support") {
source_set("test_support") {
sources = [
"fake_wifi_service.cc",
"fake_wifi_service.h",
Expand Down
6 changes: 3 additions & 3 deletions content/test/BUILD.gn
Original file line number Diff line number Diff line change
Expand Up @@ -21,12 +21,12 @@ static_library("test_support") {
"//testing/gtest",
"//ui/accessibility:ax_gen",
"//ui/base",
"//ui/base:ui_base_test_support",
"//ui/base:test_support",
"//ui/events:dom4_keycode_converter",
"//ui/events:events_base",
"//ui/events:events_test_support",
"//ui/events:test_support",
"//ui/events:gesture_detection",
"//ui/gfx:gfx_test_support",
"//ui/gfx:test_support",
"//ui/resources",
"//url",
"//webkit/common",
Expand Down
8 changes: 4 additions & 4 deletions dbus/BUILD.gn
Original file line number Diff line number Diff line change
Expand Up @@ -46,13 +46,13 @@ component("dbus") {
]
}

proto_library("dbus_test_proto") {
proto_library("test_proto") {
sources = [ "test_proto.proto" ]
}

# This target contains mocks that can be used to write unit tests without
# issuing actual D-Bus calls.
source_set("dbus_test_support") {
source_set("test_support") {
sources = [
"mock_bus.cc",
"mock_bus.h",
Expand Down Expand Up @@ -95,8 +95,8 @@ test("dbus_unittests") {

deps = [
":dbus",
":dbus_test_proto",
":dbus_test_support",
":test_proto",
":test_support",
"//base/test:test_support",
"//testing/gmock",
"//testing/gtest",
Expand Down
4 changes: 2 additions & 2 deletions gpu/BUILD.gn
Original file line number Diff line number Diff line change
Expand Up @@ -104,7 +104,7 @@ test("gl_tests") {
"//testing/gtest",
"//third_party/angle:translator",
"//ui/gfx",
"//ui/gfx:gfx_test_support",
"//ui/gfx:test_support",
"//ui/gfx/geometry",
"//ui/gl",
#"//gpu/command_buffer/client", # These are all part of //gpu, needed?
Expand Down Expand Up @@ -231,7 +231,7 @@ test("gpu_unittests") {
"//testing/gtest",
"//third_party/angle:translator",
"//ui/gfx",
"//ui/gfx:gfx_test_support",
"//ui/gfx:test_support",
"//ui/gfx/geometry",
"//ui/gl",
"//gpu/command_buffer/client:gles2_c_lib",
Expand Down
18 changes: 9 additions & 9 deletions media/BUILD.gn
Original file line number Diff line number Diff line change
Expand Up @@ -833,7 +833,7 @@ test("media_unittests") {
# }],
deps = [
":media",
":media_test_support",
":test_support",
"//base/test:test_support",
"//media/audio:unittests",
"//media/audio:test_support",
Expand All @@ -843,7 +843,7 @@ test("media_unittests") {
# TODO(dalecurtis): Port the rest of Widevine stuff.
"//third_party/widevine/cdm:version_h",
"//ui/base",
"//ui/gfx:gfx_test_support",
"//ui/gfx:test_support",
]
if (media_use_ffmpeg) {
deps += [
Expand Down Expand Up @@ -871,16 +871,16 @@ test("media_perftests") {
configs += [ ":media_config" ]
deps = [
":media",
":media_test_support",
":shared_memory_support",
":test_support",
"//base/test:test_support",
"//media/audio:test_support",
"//testing/gmock",
"//testing/gtest",
"//testing/perf",
"//third_party/widevine/cdm:version_h",
"//ui/gl",
"//ui/gfx:gfx_test_support",
"//ui/gfx:test_support",
]
if (media_use_ffmpeg) {
deps += [
Expand Down Expand Up @@ -966,7 +966,7 @@ if (cpu_arch == "x86" || cpu_arch == "x64") {
}
}

source_set("media_test_support") {
source_set("test_support") {
sources = [
"base/fake_audio_render_callback.cc",
"base/fake_audio_render_callback.h",
Expand Down Expand Up @@ -1028,13 +1028,13 @@ if (media_use_ffmpeg) {

deps = [
":media",
":media_test_support",
":test_support",
"//base",
"//base:i18n",
"//base/test:test_support",
"//testing/gtest",
"//third_party/ffmpeg",
"//ui/gfx:gfx_test_support",
"//ui/gfx:test_support",
]
}

Expand All @@ -1047,14 +1047,14 @@ if (media_use_ffmpeg) {
configs += [ ":media_config" ]
deps = [
":media",
":media_test_support",
":test_support",
"//base/test:test_support",
"//media/audio:test_support",
"//testing/gmock",
"//testing/gtest",
"//third_party/ffmpeg",
"//ui/gfx/geometry",
"//ui/gfx:gfx_test_support",
"//ui/gfx:test_support",
]
# TODO(ajwong): This was in the original gyp, but it seems silly.
# ['os_posix==1 and OS!="mac"', {
Expand Down
2 changes: 1 addition & 1 deletion printing/BUILD.gn
Original file line number Diff line number Diff line change
Expand Up @@ -216,7 +216,7 @@ test("printing_unittests") {
"//testing/gtest",
"//ui/base",
"//ui/gfx",
"//ui/gfx:gfx_test_support",
"//ui/gfx:test_support",
"//ui/gfx/geometry",
]
}
Expand Down
4 changes: 2 additions & 2 deletions ui/accessibility/BUILD.gn
Original file line number Diff line number Diff line change
Expand Up @@ -37,7 +37,7 @@ component("accessibility") {
]
}

source_set("accessibility_test_support") {
source_set("test_support") {
sources = [
"tree_generator.cc",
"tree_generator.h"
Expand All @@ -57,7 +57,7 @@ test("accessibility_unittests") {

deps = [
":accessibility",
":accessibility_test_support",
":test_support",
"//base",
"//base/test:run_all_unittests",
"//testing/gtest",
Expand Down
14 changes: 7 additions & 7 deletions ui/aura/BUILD.gn
Original file line number Diff line number Diff line change
Expand Up @@ -176,11 +176,11 @@ source_set("test_support") {
"//skia",
"//testing/gtest",
"//ui/base",
"//ui/base:ui_base_test_support",
"//ui/base:test_support",
"//ui/compositor:test_support",
"//ui/events",
"//ui/events:events_base",
"//ui/events:events_test_support",
"//ui/events:test_support",
"//ui/gfx",
"//ui/gfx/geometry",
]
Expand Down Expand Up @@ -214,7 +214,7 @@ source_set("test_support") {
#
# deps = [
# ":aura",
# ":aura_test_support",
# ":test_support",
# "//base",
# "//base:i18n",
# "//skia",
Expand Down Expand Up @@ -242,7 +242,7 @@ source_set("test_support") {
#
# deps = [
# ":aura",
# ":aura_test_support",
# ":test_support",
# "//base",
# "//base:i18n",
# "//cc",
Expand Down Expand Up @@ -275,12 +275,12 @@ source_set("test_support") {
#
# deps = [
# ":aura",
# ":aura_test_support",
# "//base/test:test_support",
# ":test_support",
# "//base/test:test_support",
# "//skia",
# "//testing/gtest",
# "//ui/base",
# "//ui/base:ui_base_test_support",
# "//ui/base:test_support",
# "//ui/compositor",
# "//ui/compositor:test_support",
# "//ui/events",
Expand Down
2 changes: 1 addition & 1 deletion ui/base/BUILD.gn
Original file line number Diff line number Diff line change
Expand Up @@ -600,7 +600,7 @@ component("base") {
}
}

source_set("ui_base_test_support") {
source_set("test_support") {
sources = [
"test/ui_controls.h",
"test/ui_controls_aura.cc",
Expand Down
6 changes: 3 additions & 3 deletions ui/events/BUILD.gn
Original file line number Diff line number Diff line change
Expand Up @@ -225,7 +225,7 @@ component("gesture_detection") {
}
}

source_set("events_test_support") {
source_set("test_support") {
sources = [
"test/cocoa_test_event_utils.h",
"test/cocoa_test_event_utils.mm",
Expand Down Expand Up @@ -303,12 +303,12 @@ test("events_unittests") {
deps = [
":events",
":events_base",
":events_test_support",
":gesture_detection",
":test_support",
"//base",
"//base/test:run_all_unittests",
"//skia",
"//testing/gtest",
"//ui/gfx:gfx_test_support",
"//ui/gfx:test_support",
]
}
4 changes: 2 additions & 2 deletions ui/gfx/BUILD.gn
Original file line number Diff line number Diff line change
Expand Up @@ -363,7 +363,7 @@ component("gfx") {

# Looking for gfx_geometry? It's //ui/gfx/geometry:geometry

source_set("gfx_test_support") {
source_set("test_support") {
sources = [
"test/fontconfig_util_linux.cc",
"test/fontconfig_util_linux.h",
Expand Down Expand Up @@ -412,7 +412,7 @@ test("gfx_unittests") {

deps = [
":gfx",
":gfx_test_support",
":test_support",
"//base",
"//base/test:test_support",
"//testing/gtest",
Expand Down

0 comments on commit e54a1f5

Please sign in to comment.