NaCl: Simpler validation for main nexe.

components/nacl/loader/nonsfi/nonsfi_listener.cc

Index: components/nacl/loader/nonsfi/nonsfi_listener.cc
diff --git a/components/nacl/loader/nonsfi/nonsfi_listener.cc b/components/nacl/loader/nonsfi/nonsfi_listener.cc
index cd1aac300a1db4ecf3bdddea8c5b77644b5eef09..dfa1dafaff833373a7791718816540ef95cf7168 100644
--- a/components/nacl/loader/nonsfi/nonsfi_listener.cc
+++ b/components/nacl/loader/nonsfi/nonsfi_listener.cc
@@ -144,8 +144,8 @@ void NonSfiListener::OnStart(const nacl::NaClStartParams& params) {
   CHECK(params.process_type == kNativeNaClProcessType);
 
   CHECK(params.nexe_file != IPC::InvalidPlatformFileForTransit());
-  CHECK(params.nexe_token_lo == 0);
-  CHECK(params.nexe_token_hi == 0);
+  CHECK(params.nexe_file_path_metadata.empty());

[Mark Seaborn, 2014/10/16 21:58:40]

Have you tested this with packaged apps?  Packaged apps on Non-SFI mode aren't
tested on the bots, so I suspect this case will break, as it did in the case
that was fixed by https://codereview.chromium.org/560983003.

Could you add test coverage for that, please?

[Mark Seaborn, 2014/11/13 16:36:28]

It looks like you did that in https://codereview.chromium.org/723623002/
yesterday, so LGTM now, and thanks.  You could add a TEST= field that mentions
that test.

+
   MainStart(IPC::PlatformFileForTransitToPlatformFile(params.nexe_file));
 }