Build NaCl browser tests with GN

ppapi/BUILD.gn

Index: ppapi/BUILD.gn
diff --git a/ppapi/BUILD.gn b/ppapi/BUILD.gn
index f45a2248b49c14a4a75716a1b94d845082213cd7..fa52265b1ce3c1bd39a8074f46b2a84bee18d14e 100644
--- a/ppapi/BUILD.gn
+++ b/ppapi/BUILD.gn
@@ -150,7 +150,7 @@ executable("pepper_hash_for_uma") {
   ]
 }
 
-source_set("ppapi_cpp_lib") {
+static_library("ppapi_cpp_lib") {

[brettw, 2015/11/19 19:08:16]

Why was this changed? If something depends on a static library to link properly,
I think there should be a comment. Otherwise, we should keep it a source set.

[Roland McGrath, 2015/11/19 19:16:13]

A comment certainly doesn't hurt.  But these are static libraries that were
always intended to be static libraries and are delivered as static libraries in
the NaCl SDK.  When the SDK build switches to using GN, it will need these to be
libraries anyway.

[Petr Hosek, 2015/11/19 20:17:56]

Comment added.

   sources = ppapi_sources.cpp_source_files
   sources += [
     "cpp/module_embedder.h",