Chromium Code Reviews| OLD | NEW |
|---|---|
| 1 // Copyright 2015 The Chromium Authors. All rights reserved. | 1 // Copyright 2015 The Chromium Authors. All rights reserved. |
| 2 // Use of this source code is governed by a BSD-style license that can be | 2 // Use of this source code is governed by a BSD-style license that can be |
| 3 // found in the LICENSE file. | 3 // found in the LICENSE file. |
| 4 | 4 |
| 5 #ifndef MOJO_PUBLIC_CPP_BINDINGS_LIB_VALIDATION_UTIL_H_ | 5 #ifndef MOJO_PUBLIC_CPP_BINDINGS_LIB_VALIDATION_UTIL_H_ |
| 6 #define MOJO_PUBLIC_CPP_BINDINGS_LIB_VALIDATION_UTIL_H_ | 6 #define MOJO_PUBLIC_CPP_BINDINGS_LIB_VALIDATION_UTIL_H_ |
| 7 | 7 |
| 8 #include <stdint.h> | 8 #include <stdint.h> |
| 9 | 9 |
| 10 #include "mojo/public/cpp/bindings/lib/bindings_internal.h" | 10 #include "mojo/public/cpp/bindings/lib/bindings_internal.h" |
| (...skipping 106 matching lines...) Expand 10 before | Expand all | Expand 10 after Loading... | |
| 117 | 117 |
| 118 template <typename T> | 118 template <typename T> |
| 119 bool ValidateInlinedUnion(const T& input, BoundsChecker* bounds_checker) { | 119 bool ValidateInlinedUnion(const T& input, BoundsChecker* bounds_checker) { |
| 120 return T::Validate(&input, bounds_checker, true); | 120 return T::Validate(&input, bounds_checker, true); |
| 121 } | 121 } |
| 122 | 122 |
| 123 bool ValidateHandle(const Handle& input, BoundsChecker* bounds_checker); | 123 bool ValidateHandle(const Handle& input, BoundsChecker* bounds_checker); |
| 124 | 124 |
| 125 bool ValidateAssociatedInterfaceId(InterfaceId input); | 125 bool ValidateAssociatedInterfaceId(InterfaceId input); |
| 126 | 126 |
| 127 bool ValidateNative(const NativePointer& input, BoundsChecker* bounds_checker); | |
|
yzshen1
2015/12/15 21:20:27
Please add validation tests. (Not necessarily in t
Ken Rockot(use gerrit already)
2015/12/15 23:31:08
No longer needed as we just use array validation n
| |
| 128 | |
| 127 } // namespace internal | 129 } // namespace internal |
| 128 } // namespace mojo | 130 } // namespace mojo |
| 129 | 131 |
| 130 #endif // MOJO_PUBLIC_CPP_BINDINGS_LIB_VALIDATION_UTIL_H_ | 132 #endif // MOJO_PUBLIC_CPP_BINDINGS_LIB_VALIDATION_UTIL_H_ |
| OLD | NEW |