| Index: blimp/BUILD.gn
|
| diff --git a/blimp/BUILD.gn b/blimp/BUILD.gn
|
| index 8410796e765c1d4caadbf6a11936fc2545ac9a69..39ff707e380bc453e851890d00e25890f4d99a88 100644
|
| --- a/blimp/BUILD.gn
|
| +++ b/blimp/BUILD.gn
|
| @@ -3,14 +3,8 @@
|
| # found in the LICENSE file.
|
|
|
| import("//build/config/chrome_build.gni")
|
| -import("//build/config/ui.gni")
|
| import("//testing/test.gni")
|
|
|
| -if (is_android) {
|
| - import("//build/config/android/config.gni")
|
| - import("//build/config/android/rules.gni")
|
| -}
|
| -
|
| # Convenience meta-target for all of Blimp's production & test code.
|
| group("blimp") {
|
| # In spite of the name, this really just means that non-test targets
|
| @@ -24,13 +18,6 @@ group("blimp") {
|
| "//blimp/common",
|
| ]
|
|
|
| - if (is_android) {
|
| - deps += [
|
| - "//blimp/client:blimp_apk",
|
| - "//blimp/client:blimp_apk_incremental",
|
| - ]
|
| - }
|
| -
|
| if (is_linux) {
|
| deps += [
|
| ":client_engine_integration",
|
| @@ -42,24 +29,17 @@ group("blimp") {
|
| deps += [ "//blimp/engine:blimp_symbols" ]
|
| }
|
| }
|
| -
|
| - if (is_linux && !is_chromeos && use_x11) {
|
| - deps += [ "//blimp/client:blimp_shell" ]
|
| - }
|
| }
|
|
|
| -# Builds all Blimp test code (unit tests, APKs).
|
| +# Builds all Blimp test code (unit tests, test APKs).
|
| group("blimp_tests") {
|
| testonly = true
|
|
|
| deps = [
|
| ":blimp_unittests",
|
| + "//blimp/client:test_binaries",
|
| ]
|
|
|
| - if (is_android) {
|
| - deps += [ "//blimp/client:blimp_test_apk" ]
|
| - }
|
| -
|
| if (is_linux) {
|
| deps += [ ":blimp_browsertests" ]
|
| }
|
| @@ -67,8 +47,7 @@ group("blimp_tests") {
|
|
|
| test("blimp_unittests") {
|
| deps = [
|
| - "//blimp/client:app_unit_tests",
|
| - "//blimp/client/core:unit_tests",
|
| + "//blimp/client:unit_tests",
|
| "//blimp/common:unit_tests",
|
| "//blimp/net:unit_tests",
|
| "//blimp/test:run_all_unittests",
|
| @@ -79,32 +58,7 @@ test("blimp_unittests") {
|
| }
|
|
|
| if (is_android) {
|
| - deps += [ ":blimp_unittests_java_deps" ]
|
| - }
|
| -}
|
| -
|
| -if (is_android) {
|
| - # This is the list of targets that the tests need to depend on in order to add
|
| - # the java classes for their native counterparts to the test apk.
|
| - # We could directly include them in the test target, but in order to
|
| - # keep the visibility of the //blimp/client/core:core_java target restricted,
|
| - # we make it visible to this group (which is testonly) and depend on this
|
| - # group instead.
|
| - # We could not add ":blimp_unittests" to the visibility list of
|
| - # //blimp/client/core:core_java, since the targets that actually need to
|
| - # depend on it are generated by the test template (see testing/test.gni). So
|
| - # it was better to add this indirection and expose it to this testonly target,
|
| - # which can be used in this file only.
|
| - java_group("blimp_unittests_java_deps") {
|
| - testonly = true
|
| - visibility = [ ":*" ]
|
| -
|
| - deps = [
|
| - "//base:base_java",
|
| - "//blimp/client/core:core_java",
|
| - "//net/android:net_java",
|
| - "//ui/android:ui_java",
|
| - ]
|
| + deps += [ "//blimp/client:blimp_unittests_java_deps" ]
|
| }
|
| }
|
|
|
|
|