Revert of GN: Properly mark all NaCl test depencies as data_deps

ppapi/BUILD.gn

Index: ppapi/BUILD.gn
diff --git a/ppapi/BUILD.gn b/ppapi/BUILD.gn
index 182287ffb301b7c3b34555265a3d99c82c0379c4..842dbbd52af92ecd70ffd1aa5e12b0d70724c647 100644
--- a/ppapi/BUILD.gn
+++ b/ppapi/BUILD.gn
@@ -193,8 +193,6 @@
 if (enable_nacl) {
   if (is_nacl) {
     shared_library("ppapi_cpp_lib_shared") {
-      output_name = "libppapi_cpp"
-
       # When using gcc, we hide all symbols by default, but that breaks at
       # link time as the test executable requires symbols defined in the
       # shared library.
@@ -213,8 +211,7 @@
       ]
     }
 
-    executable("ppapi_nacl_tests_nexe") {
-      output_name = "ppapi_nacl_tests"
+    executable("ppapi_nacl_tests") {
       include_dirs = [ "lib/gl/include" ]
       sources = ppapi_sources.test_common_source_files +
                 ppapi_sources.test_nacl_source_files
@@ -238,7 +235,7 @@
       action("translate_pexe_to_nexe") {
         # We specify the toolchain explicitly because in the Non-SFI case, we
         # still want to use the pexe built using the newlib_pnacl toolchain.
-        tests = ":ppapi_nacl_tests_nexe(//build/toolchain/nacl:newlib_pnacl)"
+        tests = ":ppapi_nacl_tests(//build/toolchain/nacl:newlib_pnacl)"
 
         pexe = get_label_info(tests, "root_out_dir") + "/ppapi_nacl_tests.pexe"
         nexe = "${root_out_dir}/ppapi_nacl_tests.nexe"
@@ -298,29 +295,26 @@
       }
     }
 
-    copy("ppapi_nacl_tests_copy") {
+    copy("nacl_tests_copy") {
       sources = [
         "${root_out_dir}/ppapi_nacl_tests.nexe",
       ]
 
       # The CPU names used in tests/ppapi_nacl_tests_newlib.nmf
       # are the ones used in GYP (x32 for x86).
-      if (target_cpu == "x86" ||
-          (is_nacl_nonsfi && (target_cpu == "x86" || target_cpu == "x64"))) {
-        nmf_cpu = "x32"
+      if (current_cpu == "pnacl") {
+        suffix = "pnacl"
       } else {
-        nmf_cpu = target_cpu
-      }
-      if (current_cpu == "pnacl") {
-        if (is_nacl_nonsfi) {
-          suffix = "pnacl_newlib_${nmf_cpu}_nonsfi"
+        if (target_cpu == "x86") {
+          nmf_cpu = "x32"
         } else {
-          suffix = "pnacl_newlib_${nmf_cpu}"
+          nmf_cpu = target_cpu
         }
-      } else if (is_nacl_glibc) {
-        suffix = "glibc_${nmf_cpu}"
-      } else {
-        suffix = "newlib_${nmf_cpu}"
+        if (is_nacl_glibc) {
+          suffix = "glibc_${nmf_cpu}"
+        } else {
+          suffix = "newlib_${nmf_cpu}"
+        }
       }
       outputs = [
         "${root_build_dir}/{{source_name_part}}_${suffix}.nexe",
@@ -331,41 +325,31 @@
         ]
       } else {
         deps = [
-          ":ppapi_nacl_tests_nexe",
-        ]
-      }
-    }
-
-    if (is_nacl_nonsfi) {
-      generate_nonsfi_test_nmf("ppapi_nacl_tests_nmf") {
-        nmf = "${root_build_dir}/ppapi_nacl_tests_pnacl_nonsfi.nmf"
-        files = get_target_outputs(":ppapi_nacl_tests_copy")
-        executable = files[0]
-        deps = [
-          ":ppapi_nacl_tests_copy",
-        ]
-      }
-    } else {
-      generate_nmf("ppapi_nacl_tests_nmf") {
-        if (is_nacl_glibc) {
-          nmf = "${root_build_dir}/ppapi_nacl_tests_glibc.nmf"
-          stage_dependencies = root_build_dir
-        } else if (current_cpu == "pnacl") {
-          nmf = "${root_build_dir}/ppapi_nacl_tests_pnacl.nmf"
-        } else {
-          nmf = "${root_build_dir}/ppapi_nacl_tests_newlib.nmf"
-        }
-        executables = get_target_outputs(":ppapi_nacl_tests_copy")
-        deps = [
-          ":ppapi_nacl_tests_copy",
-        ]
-      }
-    }
-
-    group("ppapi_nacl_tests") {
-      data_deps = [
-        ":ppapi_nacl_tests_copy",
-        ":ppapi_nacl_tests_nmf",
+          ":ppapi_nacl_tests",
+        ]
+      }
+    }
+
+    generate_nmf("ppapi_nacl_tests_nmf") {
+      if (is_nacl_glibc) {
+        nmf = "${root_build_dir}/ppapi_nacl_tests_glibc.nmf"
+        stage_dependencies = root_build_dir
+      } else if (current_cpu == "pnacl") {
+        nmf = "${root_build_dir}/ppapi_nacl_tests_pnacl.nmf"
+      } else {
+        nmf = "${root_build_dir}/ppapi_nacl_tests_newlib.nmf"
+      }
+      executables = get_target_outputs(":nacl_tests_copy")
+      deps = [
+        ":nacl_tests_copy",
+      ]
+    }
+
+    generate_nonsfi_test_nmf("ppapi_nacl_tests_pnacl_nonsfi_nmf") {
+      nmf = "${root_build_dir}/ppapi_nacl_tests_pnacl_nonsfi.nmf"
+      executable = "${root_out_dir}/ppapi_nacl_tests.nexe"
+      deps = [
+        ":translate_pexe_to_nexe",
       ]
     }
   }
@@ -373,13 +357,13 @@
   group("ppapi_nacl_tests_all") {
     data_deps = [
       ":copy_test_files",
-      ":ppapi_nacl_tests_copy(//build/toolchain/nacl:clang_newlib_${target_cpu})",
-      ":ppapi_nacl_tests(//build/toolchain/nacl:glibc_${target_cpu})",
+      ":nacl_tests_copy(//build/toolchain/nacl:clang_newlib_${target_cpu})",
+      ":ppapi_nacl_tests_nmf(//build/toolchain/nacl:glibc_${target_cpu})",
     ]
     if (enable_pnacl) {
       data_deps += [
-        ":ppapi_nacl_tests(//build/toolchain/nacl:newlib_pnacl)",
-        ":ppapi_nacl_tests(//build/toolchain/nacl:newlib_pnacl_nonsfi)",
+        ":ppapi_nacl_tests_nmf(//build/toolchain/nacl:newlib_pnacl)",
+        ":ppapi_nacl_tests_pnacl_nonsfi_nmf(//build/toolchain/nacl:newlib_pnacl_nonsfi)",
       ]
     }
   }