[mojo] Add pickling support for native-only structs

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

 // Copyright 2015 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_util.h"
 
 #include <limits>
 
 #include "mojo/public/cpp/bindings/lib/bindings_serialization.h"
 #include "mojo/public/cpp/bindings/lib/message_internal.h"
@@ skipping 116 matching lines @@
 }
 
 bool ValidateAssociatedInterfaceId(InterfaceId input) {
   if (!IsMasterInterfaceId(input))
     return true;
 
   ReportValidationError(VALIDATION_ERROR_ILLEGAL_INTERFACE_ID);
   return false;
 }
 
+bool ValidateNative(const NativePointer& input, BoundsChecker* bounds_checker) {
+  if (!ValidateEncodedPointer(&input.offset)) {
+    ReportValidationError(VALIDATION_ERROR_ILLEGAL_POINTER);
+    return false;
+  }
+
+  return true;

[yzshen1, 2015/12/15 21:20:27]

I think here we need to use the Validate() of Array_Data<uint_8> to check that
it is a valid array. It claims the memory range from |bounds_checker|, making
sure that it doesn't overlap with other objects in the message, etc.

[Ken Rockot(use gerrit already), 2015/12/15 23:31:07]

Done.

+}
+
 }  // namespace internal
 }  // namespace mojo