C++ bindings: A struct's Deserialize() now does validation before deserializing.

mojo/public/cpp/bindings/lib/bindings_internal.h

 // Copyright 2013 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.
 
 #ifndef MOJO_PUBLIC_CPP_BINDINGS_LIB_BINDINGS_INTERNAL_H_
 #define MOJO_PUBLIC_CPP_BINDINGS_LIB_BINDINGS_INTERNAL_H_
 
 #include <type_traits>
 
 #include "mojo/public/cpp/bindings/lib/template_util.h"
@@ skipping 80 matching lines @@
   T temp = *ptr;
   *ptr = T();
   return temp;
 }
 
 template <typename H>
 struct IsHandle {
   enum { value = std::is_base_of<Handle, H>::value };
 };
 
+// TODO(vardhan): Should RemoveStructPtr<> be internal-only? Should be /the/ way

[viettrungluu, 2016/03/30 17:49:01]

Hmmm. Or is specializing std::pointer_traits the right thing to do? (I *think*
this is what the standard library does for, unique_ptr/shared_ptr/etc.)

[vardhan, 2016/03/30 22:49:07]

yeah, good point;  std::pointer_traits will definitely serve both
RemoveStructPtr<> (via pointer_traits::element_type) and UnwrapStructPtr <> (via
pointer_traits::pointer).

i'll change the TODO to reflect this.

+// to get T from StructPtr<T>?
 template <typename T>
 struct RemoveStructPtr {
   typedef T type;
 };
 
 template <typename T>
 struct RemoveStructPtr<StructPtr<T>> {
   typedef T type;
 };
 
@@ skipping 159 matching lines @@
   static bool Equals(const InterfaceRequest<I>& a,
                      const InterfaceRequest<I>& b) {
     return (&a == &b) || (!a.is_pending() && !b.is_pending());
   }
 };
 
 }  // namespace internal
 }  // namespace mojo
 
 #endif  // MOJO_PUBLIC_CPP_BINDINGS_LIB_BINDINGS_INTERNAL_H_