Bug 1319595, part 6 - Fix various IPDL tests. r=billm draft
authorAndrew McCreight <continuation@gmail.com>
Tue, 22 Nov 2016 15:26:08 -0800
changeset 443027 feccef0bebbc3721be5bdbb9a0608fdede4912b3
parent 443026 3c31432e8c45ae292f4ce086ef151ecd72592e18
child 443028 80bab77168a5b1eceabd7f9ab9da711d1daa479c
push id36884
push userbmo:continuation@gmail.com
push dateWed, 23 Nov 2016 17:11:18 +0000
reviewersbillm
bugs1319595
milestone53.0a1
Bug 1319595, part 6 - Fix various IPDL tests. r=billm managerNoCtor.ipdl: Add a message so that parsing doesn't just fail immediately with a syntax error. shmem_access_union.ipdl: Remove this test, which involves the semi-removed ACL feature for shmem. oldIncludesyntax.ipdl: To be safe, make the protocol more valid by adding a message. multimanNonexistentMgrs.ipdl: Make the name of the protocol match the name of the file. MozReview-Commit-ID: 9zx5fmAWIIc
ipc/ipdl/test/ipdl/error/managerNoCtor.ipdl
ipc/ipdl/test/ipdl/error/multimanNonexistentMgrs.ipdl
ipc/ipdl/test/ipdl/error/oldIncludeSyntax.ipdl
ipc/ipdl/test/ipdl/error/shmem_access_union.ipdl
--- a/ipc/ipdl/test/ipdl/error/managerNoCtor.ipdl
+++ b/ipc/ipdl/test/ipdl/error/managerNoCtor.ipdl
@@ -1,8 +1,10 @@
 include protocol managedNoCtor;
 
 protocol managerNoCtor {
     manages managedNoCtor;
 
 parent:
     // error: no ctor defined
+    async __delete__();
+    // We need at least one message declared so we don't just hit a syntax error.
 };
--- a/ipc/ipdl/test/ipdl/error/multimanNonexistentMgrs.ipdl
+++ b/ipc/ipdl/test/ipdl/error/multimanNonexistentMgrs.ipdl
@@ -1,7 +1,7 @@
-protocol multimanNonexistentManagers {
+protocol multimanNonexistentMgrs {
     manager Starsky or Hutch;
 
 child:
     async Dummy();
     async __delete__();
 };
--- a/ipc/ipdl/test/ipdl/error/oldIncludeSyntax.ipdl
+++ b/ipc/ipdl/test/ipdl/error/oldIncludeSyntax.ipdl
@@ -1,5 +1,5 @@
 include protocol "Foo.ipdl";
 
 protocol oldIncludeSyntax {
-
+child: async __delete__(X x);
 };
deleted file mode 100644
--- a/ipc/ipdl/test/ipdl/error/shmem_access_union.ipdl
+++ /dev/null
@@ -1,8 +0,0 @@
-union Union {
-    [-r-w|+r+w] Shmem;
-};
-
-protocol shmem_access_union {
-child:
-    async Msg(Union u);
-};