C bindings: Implement _Validate(), and some pre-requisites

mojo/public/cpp/bindings/tests/BUILD.gn

 # 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.
 
 import("../../../mojo_application.gni")
 import("../../../mojo_sdk.gni")
 
 mojo_sdk_source_set("tests") {
   testonly = true
 
@@ skipping 30 matching lines @@
     "struct_unittest.cc",
     "synchronous_connector_unittest.cc",
     "synchronous_interface_ptr_unittest.cc",
     "type_conversion_unittest.cc",
     "union_unittest.cc",
     "validation_unittest.cc",
   ]
 
   deps = [
     ":mojo_public_bindings_test_utils",
+    ":validation_util",
     "//testing/gtest",
   ]
 
   mojo_sdk_deps = [
     "mojo/public/cpp/bindings",
     "mojo/public/cpp/bindings:callback",
     "mojo/public/cpp/environment:standalone",
     "mojo/public/cpp/system",
     "mojo/public/cpp/test_support",
     "mojo/public/cpp/test_support:test_utils",
@@ skipping 27 matching lines @@
 
 mojo_sdk_source_set("mojo_public_bindings_test_utils") {
   sources = [
     "validation_test_input_parser.cc",
     "validation_test_input_parser.h",
   ]
 
   mojo_sdk_deps = [ "mojo/public/c/system" ]
 }
 
+mojo_sdk_source_set("validation_util") {

[viettrungluu, 2016/07/28 23:03:01]

Do you really need a separate target here, instead of moving it into the
mojo_public_bindings_test_utils target above?

[vardhan, 2016/07/29 21:29:22]

yeah, i tried this initially :( mojo_public_bindings_test_utils is used by other
languages (cython) to parse validation test+golden files. And since this target
has some gtest references, it would force mojo_public_bindings_test_utils to be
testonly, which created some problems for cython.  It was much cleaner to just
do this (or remove gtest dependencies, which means we lose some ADD_FAILURES()
which i think we should keep)

[viettrungluu, 2016/08/01 21:17:19]

Blech. Not having a gtest dependency seems reasonable, but it seems to me that
whatever cython target is depending on mojo_public_bindings_test_utils should be
testonly (and so should mojo_public_bindings_test_utils). Could you add a TODO
about this?

[vardhan, 2016/08/01 22:18:16]

Done.

+  testonly = true
+
+  sources = [
+    "validation_util.cc",
+    "validation_util.h",
+  ]
+
+  mojo_sdk_deps = [
+    "mojo/public/c/system",
+    "mojo/public/cpp/test_support",
+  ]
+
+  deps = [
+    ":mojo_public_bindings_test_utils",
+    "//testing/gtest",
+  ]
+}
+
 mojo_native_application("versioning_test_service") {
   testonly = true
 
   deps = [
     ":versioning_test_service_source_set",
   ]
 }
 
 mojo_sdk_source_set("versioning_test_service_source_set") {
   testonly = true
@@ skipping 31 matching lines @@
   ]
 
   mojo_sdk_deps = [
     "mojo/public/cpp/application:standalone",
     "mojo/public/cpp/application:test_support_standalone",
     "mojo/public/cpp/bindings",
     "mojo/public/cpp/system",
     "mojo/public/interfaces/bindings/tests:versioning_test_client_interfaces",
   ]
 }