Added test and fixed bugs with release dependency

Fixed 'visited' bug where sibling were poluting the visit set, preventing dependency expantion.
Fixed release list generation where a node's typeref releases were not constrained by the node's min/max.
Added generator unit tests to PRESUBMIT when changes to generator are detected.
Added test for release dependency when a child uses different releases than the parent.




Fix release dependencies and tests.
R=sehr@chromium.org
BUG=157017


Review URL: https://chromiumcodereview.appspot.com/11419173

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@171499 0039d316-1c4b-4281-b951-d872f2087c98

2 files changed, 20 insertions, 47 deletions

diff --git a/ppapi/generators/test_cgen_range/versions.h b/ppapi/generators/test_cgen_range/versions.h
index 92b5324..bb0f2b1 100644
--- a/ppapi/generators/test_cgen_range/versions.h
+++ b/ppapi/generators/test_cgen_range/versions.h
@@ -3,17 +3,17 @@
  * found in the LICENSE file.
  */
 
-/* From test_cgen_range/versions.idl modified Wed Aug 24 19:49:19 2011. */
+/* From test_cgen_range/versions.idl modified Wed Nov 21 15:18:23 2012. */
 
 #ifndef PPAPI_C_TEST_CGEN_RANGE_VERSIONS_H_
 #define PPAPI_C_TEST_CGEN_RANGE_VERSIONS_H_
 
 #include "ppapi/c/pp_macros.h"
 
-#define BAR_INTERFACE_0_0 "Bar;0.0"
-#define BAR_INTERFACE_1_0 "Bar;1.0"
-#define BAR_INTERFACE_2_0 "Bar;2.0"
-#define BAR_INTERFACE BAR_INTERFACE_2_0
+#define FOO_INTERFACE_0_0 "Foo;0.0"
+#define FOO_INTERFACE_1_0 "Foo;1.0"
+#define FOO_INTERFACE_2_0 "Foo;2.0"
+#define FOO_INTERFACE FOO_INTERFACE_2_0
 
 /**
  * @file
@@ -21,42 +21,25 @@
 
 
 /**
- * @addtogroup Structs
+ * @addtogroup Interfaces
  * @{
  */
-/* Bogus Struct Foo */
-struct Foo {
+/* Bogus Interface Foo */
+struct Foo_2_0 {
   /**
    * Comment for function x,y,z
    */
-  int32_t (*Foo)(int32_t x, int32_t y, int32_t z);
+  int32_t (*Bar)(int32_t x, int32_t y, int32_t z);
 };
+
+typedef struct Foo_2_0 Foo;
+
 struct Foo_0_0 {
-  int32_t (*Foo)(int32_t x);
-};
-struct Foo_1_0 {
-  int32_t (*Foo)(int32_t x, int32_t y);
+  int32_t (*Bar)(int32_t x);
 };
-/**
- * @}
- */
 
-/**
- * @addtogroup Interfaces
- * @{
- */
-/* Inherit revisions thanks to Foo */
-struct Bar {
-  /**
-   * Comment for function
-   */
-  int32_t (*UseFoo)( struct Foo* val);
-};
-struct Bar_0_0 {
-  int32_t (*UseFoo)( struct Foo* val);
-};
-struct Bar_1_0 {
-  int32_t (*UseFoo)( struct Foo* val);
+struct Foo_1_0 {
+  int32_t (*Bar)(int32_t x, int32_t y);
 };
 /**
  * @}
diff --git a/ppapi/generators/test_cgen_range/versions.idl b/ppapi/generators/test_cgen_range/versions.idl
index 11b4de0..3dcc87e 100644
--- a/ppapi/generators/test_cgen_range/versions.idl
+++ b/ppapi/generators/test_cgen_range/versions.idl
@@ -15,30 +15,20 @@ describe {
   int32_t;
 };
 
-
-/* Bogus Struct Foo */
+/* Bogus Interface Foo */
 [version=0.0]
-struct Foo {
+interface Foo {
   /**
    * Comment for function x
    */
-  [version=0.0] int32_t Foo(int32_t x);
+  [version=0.0] int32_t Bar(int32_t x);
   /**
    * Comment for function x,y
    */
-  [version=1.0] int32_t Foo(int32_t x, int32_t y);
+  [version=1.0] int32_t Bar(int32_t x, int32_t y);
   /**
    * Comment for function x,y,z
    */
-  [version=2.0] int32_t Foo(int32_t x, int32_t y, int32_t z);
+  [version=2.0] int32_t Bar(int32_t x, int32_t y, int32_t z);
 };
 
-
-/* Inherit revisions thanks to Foo */
-[version=0.0]
-interface Bar {
-  /**
-   * Comment for function
-   */
-  int32_t UseFoo(Foo val);
-};