Build NaCl browser tests with GN

chrome/test/BUILD.gn

Index: chrome/test/BUILD.gn
diff --git a/chrome/test/BUILD.gn b/chrome/test/BUILD.gn
index a795e1237d787f803eb631161d1e2cc8fcaa0d3f..ccdd6ae3216afb182179fcc7c92365cf5f78a219 100644
--- a/chrome/test/BUILD.gn
+++ b/chrome/test/BUILD.gn
@@ -955,7 +955,7 @@ if (!is_android) {
         ]
 
         data_deps += [
-          # "test/data/nacl:shared_test_files",  # TODO(GYP) bug 512902
+          "//chrome/test/data/nacl",

[Roland McGrath, 2015/11/11 06:22:45]

Don't we usually use an explicit :something here?
Even if it's just "...:*"?

[Dirk Pranke, 2015/11/11 22:19:05]

not using a ":something" defaults to ":dirname", e.g.,
"//chrome/test/data/nacl:nacl".

GN does not support wildcards.

[Petr Hosek, 2015/11/13 22:18:32]

GN defaults to the name of the directory, so this line is equivalent to
"//chrome/test/data/nacl:nacl". I can rename the target if you prefer.

[Dirk Pranke, 2015/11/13 23:00:57]

Don't rename the target. This is the preferred style.

           "//ppapi/native_client:irt",
           "//ppapi:ppapi_nacl_tests_all",