Cpp bindings: Return false from Validator::Accept() on unrecognized message or invalid flags

mojo/public/js/validation_unittests.js

Index: mojo/public/js/validation_unittests.js
diff --git a/mojo/public/js/validation_unittests.js b/mojo/public/js/validation_unittests.js
index a04879cd4b00f3bda1d466b88837bcdcae35371d..a235bd8c9db8b42a26ad567c6b24d9f4df9f946d 100644
--- a/mojo/public/js/validation_unittests.js
+++ b/mojo/public/js/validation_unittests.js
@@ -225,11 +225,13 @@ define([
     expect(testFiles.length).toBeGreaterThan(0);
 
     for (var i = 0; i < testFiles.length; i++) {
-      // TODO(hansmuller, yzshen): Temporarily skipping:
+      // TODO(hansmuller, yzshen, rudominer): Temporarily skipping:
       //   - array pointer overflow tests;
       //   - struct versioning tests (tests with "mthd11" in the name).
+      //   - no-such-method validation check tests.
       if (testFiles[i].indexOf("overflow") != -1 ||
-          testFiles[i].indexOf("mthd11") != -1) {
+          testFiles[i].indexOf("mthd11") != -1 ||
+          testFiles[i].indexOf("no_such_method") != -1) {
         console.log("[Skipping " + testFiles[i] + "]");
         continue;
       }
@@ -258,7 +260,9 @@ define([
 
     for (var i = 0; i < testFiles.length; i++) {
       // TODO(hansmuller): Temporarily skipping array pointer overflow tests.

[yzshen1, 2015/03/03 18:05:37]

This whole if-statement can be removed, because we don't have either "overflow"
or "no_such_method" integration tests.

[rudominer, 2015/03/04 00:12:01]

Done.

-      if (testFiles[i].indexOf("overflow") != -1) {
+      // TODO(rudominer) Temporarily skipping no-such-method verifcation tests.
+      if (testFiles[i].indexOf("overflow") != -1 ||
+          testFiles[i].indexOf("no_such_method") != -1) {
         console.log("[Skipping " + testFiles[i] + "]");
         continue;
       }