NaCl: create a separate namespace for Mojo handles.

mojo/edk/embedder/BUILD.gn

Index: mojo/edk/embedder/BUILD.gn
diff --git a/mojo/edk/embedder/BUILD.gn b/mojo/edk/embedder/BUILD.gn
index 9650209cf788c25c268bb45c0e39ef6ec21abf8f..bdb5277e2d1e4bebfdb2f3b09ed8a8731012487f 100644
--- a/mojo/edk/embedder/BUILD.gn
+++ b/mojo/edk/embedder/BUILD.gn
@@ -10,12 +10,14 @@ mojo_edk_source_set("embedder") {
   mojo_edk_visibility = [ "mojo/edk/system" ]
 
   sources = [
+    "../../public/platform/native/system_impl_private.h",  # HACK

[Nick Bray (chromium), 2015/04/06 22:50:59]

Where should this header live?

[viettrungluu, 2015/04/06 23:21:02]

I think it's fine where it is.

Technically, I don't think you even need to include it from
system_impl_private_entrypoints.cc, but it's probably good to to ensure
consistency (namely, that the declarations match the definitions).

Anyhow, I think what you really want is a suitable target/mojo_sdk_source_set
for it, and put that in mojo_sdk_public_deps[*] below, or something like that.

[*] Or maybe add a non-public mojo_sdk_deps (haven't thought about it too much).

[Nick Bray (chromium), 2015/04/07 23:51:05]

Done.

     "channel_info_forward.h",
     "configuration.h",
     "embedder.cc",
     "embedder.h",
     "embedder_internal.h",
     "entrypoints.cc",
+    "system_impl_private_entrypoints.cc",
 
     # Test-only code:
     # TODO(vtl): It's a little unfortunate that these end up in the same