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

mojo/public/cpp/bindings/lib/validation_errors.cc

 // Copyright 2014 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "mojo/public/cpp/bindings/lib/validation_errors.h"
 
 #include "mojo/public/cpp/environment/logging.h"
 
 namespace mojo {
 namespace internal {
@@ skipping 22 matching lines @@
     case VALIDATION_ERROR_UNEXPECTED_INVALID_HANDLE:
       return "VALIDATION_ERROR_UNEXPECTED_INVALID_HANDLE";
     case VALIDATION_ERROR_ILLEGAL_POINTER:
       return "VALIDATION_ERROR_ILLEGAL_POINTER";
     case VALIDATION_ERROR_UNEXPECTED_NULL_POINTER:
       return "VALIDATION_ERROR_UNEXPECTED_NULL_POINTER";
     case VALIDATION_ERROR_MESSAGE_HEADER_INVALID_FLAG_COMBINATION:
       return "VALIDATION_ERROR_MESSAGE_HEADER_INVALID_FLAG_COMBINATION";
     case VALIDATION_ERROR_MESSAGE_HEADER_MISSING_REQUEST_ID:
       return "VALIDATION_ERROR_MESSAGE_HEADER_MISSING_REQUEST_ID";
+    case VALIDATION_ERROR_MESSAGE_HEADER_UNKNOWN_METHOD:

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

Please use the exact same order as the enum definition.
(May you could move the value in the definition to match this, so that all
"message header" values are together.)

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

Done.

+      return "VALIDATION_ERROR_MESSAGE_HEADER_UNKNOWN_METHOD";
     case VALIDATION_ERROR_DIFFERENT_SIZED_ARRAYS_IN_MAP:
       return "VALIDATION_ERROR_DIFFERENT_SIZED_ARRAYS_IN_MAP";
     case VALIDATION_ERROR_UNKOWN_UNION_TAG:
       return "VALIDATION_ERROR_UNKOWN_UNION_TAG";
   }
 
   return "Unknown error";
 }
 
 void ReportValidationError(ValidationError error, const char* description) {
@@ skipping 34 matching lines @@
 }
 
 SerializationWarningObserverForTesting::
     ~SerializationWarningObserverForTesting() {
   MOJO_DCHECK(g_serialization_warning_observer == this);
   g_serialization_warning_observer = nullptr;
 }
 
 }  // namespace internal
 }  // namespace mojo