Bug 1529686 - Part 1 - fixed ipdl unittests for devirtualization, which requires protocol names to start with 'P'; r=mccr8
authorAlex Gaynor <agaynor@mozilla.com>
Tue, 26 Feb 2019 16:54:04 +0000
changeset 519071 14dc79650c63e7054014f6f8166e0cccbbe2d0b5
parent 519070 d48af3b7c76df42e0f66e818a73a0720ff07f092
child 519072 bba070f5b4d3fad2020a63b1a646374d3052a856
push id10862
push userffxbld-merge
push dateMon, 11 Mar 2019 13:01:11 +0000
treeherdermozilla-beta@a2e7f5c935da [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
reviewersmccr8
bugs1529686
milestone67.0a1
first release with
nightly linux32
nightly linux64
nightly mac
nightly win32
nightly win64
last release without
nightly linux32
nightly linux64
nightly mac
nightly win32
nightly win64
Bug 1529686 - Part 1 - fixed ipdl unittests for devirtualization, which requires protocol names to start with 'P'; r=mccr8 Differential Revision: https://phabricator.services.mozilla.com/D21066
ipc/ipdl/test/ipdl/error/Nullable.ipdl
ipc/ipdl/test/ipdl/error/PasyncMessageListed.ipdl
ipc/ipdl/test/ipdl/error/asyncMessageListed.ipdl
ipc/ipdl/test/ipdl/ok/AsyncReturn.ipdl
ipc/ipdl/test/ipdl/ok/Delete.ipdl
ipc/ipdl/test/ipdl/ok/DeleteSub.ipdl
ipc/ipdl/test/ipdl/ok/EndpointDecl.ipdl
ipc/ipdl/test/ipdl/ok/EndpointUse.ipdl
ipc/ipdl/test/ipdl/ok/Nullable.ipdl
ipc/ipdl/test/ipdl/ok/PAsyncReturn.ipdl
ipc/ipdl/test/ipdl/ok/PDelete.ipdl
ipc/ipdl/test/ipdl/ok/PDeleteSub.ipdl
ipc/ipdl/test/ipdl/ok/PEndpointDecl.ipdl
ipc/ipdl/test/ipdl/ok/PEndpointUse.ipdl
ipc/ipdl/test/ipdl/ok/PNullable.ipdl
ipc/ipdl/test/ipdl/ok/PStruct.ipdl
ipc/ipdl/test/ipdl/ok/PSyncSyncManagee.ipdl
ipc/ipdl/test/ipdl/ok/PSyncSyncManager.ipdl
ipc/ipdl/test/ipdl/ok/Pactorparam.ipdl
ipc/ipdl/test/ipdl/ok/Pactorreturn.ipdl
ipc/ipdl/test/ipdl/ok/Parray_Basic.ipdl
ipc/ipdl/test/ipdl/ok/Parray_OfActors.ipdl
ipc/ipdl/test/ipdl/ok/Parray_OfActorsSub.ipdl
ipc/ipdl/test/ipdl/ok/Parray_Union.ipdl
ipc/ipdl/test/ipdl/ok/PbasicUsing.ipdl
ipc/ipdl/test/ipdl/ok/Pbuiltins.ipdl
ipc/ipdl/test/ipdl/ok/Pbytebuf.ipdl
ipc/ipdl/test/ipdl/ok/Pempty.ipdl
ipc/ipdl/test/ipdl/ok/PemptyStruct.ipdl
ipc/ipdl/test/ipdl/ok/PheaderProto.ipdl
ipc/ipdl/test/ipdl/ok/PintrProtocol.ipdl
ipc/ipdl/test/ipdl/ok/Pjetpack.ipdl
ipc/ipdl/test/ipdl/ok/PmanageSelf.ipdl
ipc/ipdl/test/ipdl/ok/PmanageSelf_Toplevel.ipdl
ipc/ipdl/test/ipdl/ok/PmanagedProtocol.ipdl
ipc/ipdl/test/ipdl/ok/PmanagerProtocol.ipdl
ipc/ipdl/test/ipdl/ok/Pmaybe_Basic.ipdl
ipc/ipdl/test/ipdl/ok/Pmaybe_OfActors.ipdl
ipc/ipdl/test/ipdl/ok/Pmaybe_OfActorsSub.ipdl
ipc/ipdl/test/ipdl/ok/Pmaybe_Union.ipdl
ipc/ipdl/test/ipdl/ok/Pmedia.ipdl
ipc/ipdl/test/ipdl/ok/PmessageCompress.ipdl
ipc/ipdl/test/ipdl/ok/PmessageVerify.ipdl
ipc/ipdl/test/ipdl/ok/PmessageVerifyTopLevel.ipdl
ipc/ipdl/test/ipdl/ok/PmultiManaged.ipdl
ipc/ipdl/test/ipdl/ok/PmultiManager1.ipdl
ipc/ipdl/test/ipdl/ok/PmultiManager2.ipdl
ipc/ipdl/test/ipdl/ok/PmultipleUsingCxxTypes.ipdl
ipc/ipdl/test/ipdl/ok/PmutualRecStructUnion.ipdl
ipc/ipdl/test/ipdl/ok/PmutualRecUnion.ipdl
ipc/ipdl/test/ipdl/ok/Pnamespace_Basic.ipdl
ipc/ipdl/test/ipdl/ok/PnoRedeclCrossMessage.ipdl
ipc/ipdl/test/ipdl/ok/Pplugin.ipdl
ipc/ipdl/test/ipdl/ok/PselfRecUnion.ipdl
ipc/ipdl/test/ipdl/ok/Pshmem.ipdl
ipc/ipdl/test/ipdl/ok/PsyncProtocol.ipdl
ipc/ipdl/test/ipdl/ok/PthreeDirections.ipdl
ipc/ipdl/test/ipdl/ok/Punion_Basic.ipdl
ipc/ipdl/test/ipdl/ok/Punion_Namespaced.ipdl
ipc/ipdl/test/ipdl/ok/Struct.ipdl
ipc/ipdl/test/ipdl/ok/SyncSyncManagee.ipdl
ipc/ipdl/test/ipdl/ok/SyncSyncManager.ipdl
ipc/ipdl/test/ipdl/ok/actorparam.ipdl
ipc/ipdl/test/ipdl/ok/actorreturn.ipdl
ipc/ipdl/test/ipdl/ok/array_Basic.ipdl
ipc/ipdl/test/ipdl/ok/array_OfActors.ipdl
ipc/ipdl/test/ipdl/ok/array_OfActorsSub.ipdl
ipc/ipdl/test/ipdl/ok/array_Union.ipdl
ipc/ipdl/test/ipdl/ok/basicUsing.ipdl
ipc/ipdl/test/ipdl/ok/builtins.ipdl
ipc/ipdl/test/ipdl/ok/bytebuf.ipdl
ipc/ipdl/test/ipdl/ok/empty.ipdl
ipc/ipdl/test/ipdl/ok/emptyStruct.ipdl
ipc/ipdl/test/ipdl/ok/headerProto.ipdl
ipc/ipdl/test/ipdl/ok/intrProtocol.ipdl
ipc/ipdl/test/ipdl/ok/jetpack.ipdl
ipc/ipdl/test/ipdl/ok/manageSelf.ipdl
ipc/ipdl/test/ipdl/ok/manageSelf_Toplevel.ipdl
ipc/ipdl/test/ipdl/ok/managedProtocol.ipdl
ipc/ipdl/test/ipdl/ok/managerProtocol.ipdl
ipc/ipdl/test/ipdl/ok/maybe_Basic.ipdl
ipc/ipdl/test/ipdl/ok/maybe_OfActors.ipdl
ipc/ipdl/test/ipdl/ok/maybe_OfActorsSub.ipdl
ipc/ipdl/test/ipdl/ok/maybe_Union.ipdl
ipc/ipdl/test/ipdl/ok/media.ipdl
ipc/ipdl/test/ipdl/ok/messageCompress.ipdl
ipc/ipdl/test/ipdl/ok/messageVerify.ipdl
ipc/ipdl/test/ipdl/ok/messageVerifyTopLevel.ipdl
ipc/ipdl/test/ipdl/ok/multiManaged.ipdl
ipc/ipdl/test/ipdl/ok/multiManager1.ipdl
ipc/ipdl/test/ipdl/ok/multiManager2.ipdl
ipc/ipdl/test/ipdl/ok/multipleUsingCxxTypes.ipdl
ipc/ipdl/test/ipdl/ok/mutualRecStructUnion.ipdl
ipc/ipdl/test/ipdl/ok/mutualRecUnion.ipdl
ipc/ipdl/test/ipdl/ok/namespace_Basic.ipdl
ipc/ipdl/test/ipdl/ok/noRedeclCrossMessage.ipdl
ipc/ipdl/test/ipdl/ok/plugin.ipdl
ipc/ipdl/test/ipdl/ok/selfRecUnion.ipdl
ipc/ipdl/test/ipdl/ok/shmem.ipdl
ipc/ipdl/test/ipdl/ok/syncProtocol.ipdl
ipc/ipdl/test/ipdl/ok/threeDirections.ipdl
ipc/ipdl/test/ipdl/ok/union_Basic.ipdl
ipc/ipdl/test/ipdl/ok/union_Namespaced.ipdl
ipc/ipdl/test/ipdl/sync-messages.ini
--- a/ipc/ipdl/test/ipdl/error/Nullable.ipdl
+++ b/ipc/ipdl/test/ipdl/error/Nullable.ipdl
@@ -1,6 +1,6 @@
 //error: `nullable' qualifier for type `int' makes no sense
 
-protocol Nullable {
+protocol PNullable {
 child:
     async Msg(nullable int i);
 };
rename from ipc/ipdl/test/ipdl/error/asyncMessageListed.ipdl
rename to ipc/ipdl/test/ipdl/error/PasyncMessageListed.ipdl
--- a/ipc/ipdl/test/ipdl/error/asyncMessageListed.ipdl
+++ b/ipc/ipdl/test/ipdl/error/PasyncMessageListed.ipdl
@@ -1,6 +1,6 @@
-//error: IPC message asyncMessageListed::Msg is async, can be delisted
+//error: IPC message PasyncMessageListed::Msg is async, can be delisted
 
-protocol asyncMessageListed {
+protocol PasyncMessageListed {
 parent:
     async Msg();
 };
rename from ipc/ipdl/test/ipdl/ok/AsyncReturn.ipdl
rename to ipc/ipdl/test/ipdl/ok/PAsyncReturn.ipdl
--- a/ipc/ipdl/test/ipdl/ok/AsyncReturn.ipdl
+++ b/ipc/ipdl/test/ipdl/ok/PAsyncReturn.ipdl
@@ -1,8 +1,8 @@
 // Async messages are not allowed to return values.
 
 //error: asynchronous message `Msg' declares return values
 
-protocol AsyncReturn {
+protocol PAsyncReturn {
 child:
     async Msg() returns(int32_t aNumber);
 };
rename from ipc/ipdl/test/ipdl/ok/Delete.ipdl
rename to ipc/ipdl/test/ipdl/ok/PDelete.ipdl
--- a/ipc/ipdl/test/ipdl/ok/Delete.ipdl
+++ b/ipc/ipdl/test/ipdl/ok/PDelete.ipdl
@@ -1,8 +1,8 @@
-include protocol DeleteSub;
+include protocol PDeleteSub;
 
-sync protocol Delete {
-    manages DeleteSub;
+sync protocol PDelete {
+    manages PDeleteSub;
 
 child:
-    async DeleteSub();
+    async PDeleteSub();
 };
rename from ipc/ipdl/test/ipdl/ok/DeleteSub.ipdl
rename to ipc/ipdl/test/ipdl/ok/PDeleteSub.ipdl
--- a/ipc/ipdl/test/ipdl/ok/DeleteSub.ipdl
+++ b/ipc/ipdl/test/ipdl/ok/PDeleteSub.ipdl
@@ -1,10 +1,10 @@
-include protocol Delete;
+include protocol PDelete;
 
-sync protocol DeleteSub {
-    manager Delete;
+sync protocol PDeleteSub {
+    manager PDelete;
 
 parent:
     sync __delete__(int x) returns (double d);
 
 };
 
rename from ipc/ipdl/test/ipdl/ok/EndpointDecl.ipdl
rename to ipc/ipdl/test/ipdl/ok/PEndpointDecl.ipdl
--- a/ipc/ipdl/test/ipdl/ok/EndpointDecl.ipdl
+++ b/ipc/ipdl/test/ipdl/ok/PEndpointDecl.ipdl
@@ -1,18 +1,18 @@
 
 // Basic test that Endpoint types are declared for protocols, within
 // that protocol.
 
 struct Whatever {
-  Endpoint<EndpointDeclParent> par;
-  Endpoint<EndpointDeclChild> chi;
+  Endpoint<PEndpointDeclParent> par;
+  Endpoint<PEndpointDeclChild> chi;
 };
 
 namespace mozilla {
 
-protocol EndpointDecl {
+protocol PEndpointDecl {
   child:
-    async Message(Endpoint<EndpointDeclParent> aEndpointParent,
-                  Endpoint<EndpointDeclChild> aEndpointChild);
+    async Message(Endpoint<PEndpointDeclParent> aEndpointParent,
+                  Endpoint<PEndpointDeclChild> aEndpointChild);
 };
 
 }
rename from ipc/ipdl/test/ipdl/ok/EndpointUse.ipdl
rename to ipc/ipdl/test/ipdl/ok/PEndpointUse.ipdl
--- a/ipc/ipdl/test/ipdl/ok/EndpointUse.ipdl
+++ b/ipc/ipdl/test/ipdl/ok/PEndpointUse.ipdl
@@ -1,9 +1,9 @@
-include protocol EndpointDecl;
+include protocol PEndpointDecl;
 
 // Basic test that Endpoint types are declared for included protocols.
 
-protocol EndpointUse {
+protocol PEndpointUse {
   child:
-    async Message(Endpoint<EndpointDeclParent> aEndpointParent,
-                  Endpoint<EndpointDeclChild> aEndpointChild);
+    async Message(Endpoint<PEndpointDeclParent> aEndpointParent,
+                  Endpoint<PEndpointDeclChild> aEndpointChild);
 };
rename from ipc/ipdl/test/ipdl/ok/Nullable.ipdl
rename to ipc/ipdl/test/ipdl/ok/PNullable.ipdl
--- a/ipc/ipdl/test/ipdl/ok/Nullable.ipdl
+++ b/ipc/ipdl/test/ipdl/ok/PNullable.ipdl
@@ -1,11 +1,11 @@
 union Union {
-    nullable Nullable;
-    nullable Nullable[];
+    nullable PNullable;
+    nullable PNullable[];
 };
 
-protocol Nullable {
+protocol PNullable {
 child:
-    async Msg(nullable Nullable n);
-    async Msg2(nullable Nullable[] N);
+    async Msg(nullable PNullable n);
+    async Msg2(nullable PNullable[] N);
     async Msg3(Union u);
 };
rename from ipc/ipdl/test/ipdl/ok/Struct.ipdl
rename to ipc/ipdl/test/ipdl/ok/PStruct.ipdl
--- a/ipc/ipdl/test/ipdl/ok/Struct.ipdl
+++ b/ipc/ipdl/test/ipdl/ok/PStruct.ipdl
@@ -1,10 +1,10 @@
 struct S {
     int i;
     double d;
 };
 
-sync protocol Struct {
+sync protocol PStruct {
 parent:
     sync test(S s) returns (S ss);
     async __delete__();
 };
rename from ipc/ipdl/test/ipdl/ok/SyncSyncManagee.ipdl
rename to ipc/ipdl/test/ipdl/ok/PSyncSyncManagee.ipdl
--- a/ipc/ipdl/test/ipdl/ok/SyncSyncManagee.ipdl
+++ b/ipc/ipdl/test/ipdl/ok/PSyncSyncManagee.ipdl
@@ -1,7 +1,7 @@
-include protocol SyncSyncManager;
+include protocol PSyncSyncManager;
 
-sync protocol SyncSyncManagee {
-    manager SyncSyncManager;
+sync protocol PSyncSyncManagee {
+    manager PSyncSyncManager;
 child:
     async __delete__();
 };
rename from ipc/ipdl/test/ipdl/ok/SyncSyncManager.ipdl
rename to ipc/ipdl/test/ipdl/ok/PSyncSyncManager.ipdl
--- a/ipc/ipdl/test/ipdl/ok/SyncSyncManager.ipdl
+++ b/ipc/ipdl/test/ipdl/ok/PSyncSyncManager.ipdl
@@ -1,11 +1,11 @@
-include protocol SyncSyncManagee;
+include protocol PSyncSyncManagee;
 
 /* The main reason for this test is that it would have caught a bug
  * in the Rust IPDL parser that was treating "sync" like "async" in the
  * nested case.
  */
-nested(upto not) sync protocol SyncSyncManager {
-    manages SyncSyncManagee;
+nested(upto not) sync protocol PSyncSyncManager {
+    manages PSyncSyncManagee;
 parent:
-    async SyncSyncManagee();
+    async PSyncSyncManagee();
 };
rename from ipc/ipdl/test/ipdl/ok/actorparam.ipdl
rename to ipc/ipdl/test/ipdl/ok/Pactorparam.ipdl
--- a/ipc/ipdl/test/ipdl/ok/actorparam.ipdl
+++ b/ipc/ipdl/test/ipdl/ok/Pactorparam.ipdl
@@ -1,5 +1,5 @@
-protocol actorparam {
+protocol Pactorparam {
 
-child: async Msg(actorparam p);
+child: async Msg(Pactorparam p);
 
 };
rename from ipc/ipdl/test/ipdl/ok/actorreturn.ipdl
rename to ipc/ipdl/test/ipdl/ok/Pactorreturn.ipdl
--- a/ipc/ipdl/test/ipdl/ok/actorreturn.ipdl
+++ b/ipc/ipdl/test/ipdl/ok/Pactorreturn.ipdl
@@ -1,6 +1,6 @@
-sync protocol actorreturn {
+sync protocol Pactorreturn {
 
 parent:
-    sync Msg(actorreturn p) returns (actorreturn r);
+    sync Msg(Pactorreturn p) returns (Pactorreturn r);
 
 };
rename from ipc/ipdl/test/ipdl/ok/array_Basic.ipdl
rename to ipc/ipdl/test/ipdl/ok/Parray_Basic.ipdl
--- a/ipc/ipdl/test/ipdl/ok/array_Basic.ipdl
+++ b/ipc/ipdl/test/ipdl/ok/Parray_Basic.ipdl
@@ -1,4 +1,4 @@
-protocol array_Basic {
+protocol Parray_Basic {
 child:
     async Msg(int[] array);
 };
rename from ipc/ipdl/test/ipdl/ok/array_OfActors.ipdl
rename to ipc/ipdl/test/ipdl/ok/Parray_OfActors.ipdl
--- a/ipc/ipdl/test/ipdl/ok/array_OfActors.ipdl
+++ b/ipc/ipdl/test/ipdl/ok/Parray_OfActors.ipdl
@@ -1,10 +1,10 @@
-include protocol array_OfActorsSub;
+include protocol Parray_OfActorsSub;
 
-protocol array_OfActors {
-    manages array_OfActorsSub;
+protocol Parray_OfActors {
+    manages Parray_OfActorsSub;
 
 child:
-    async Msg(array_OfActorsSub[] p);
+    async Msg(Parray_OfActorsSub[] p);
 
-    async array_OfActorsSub();
+    async Parray_OfActorsSub();
 };
rename from ipc/ipdl/test/ipdl/ok/array_OfActorsSub.ipdl
rename to ipc/ipdl/test/ipdl/ok/Parray_OfActorsSub.ipdl
--- a/ipc/ipdl/test/ipdl/ok/array_OfActorsSub.ipdl
+++ b/ipc/ipdl/test/ipdl/ok/Parray_OfActorsSub.ipdl
@@ -1,7 +1,7 @@
-include protocol array_OfActors;
+include protocol Parray_OfActors;
 
-protocol array_OfActorsSub {
-    manager array_OfActors;
+protocol Parray_OfActorsSub {
+    manager Parray_OfActors;
 
 child: async __delete__();
 };
rename from ipc/ipdl/test/ipdl/ok/array_Union.ipdl
rename to ipc/ipdl/test/ipdl/ok/Parray_Union.ipdl
--- a/ipc/ipdl/test/ipdl/ok/array_Union.ipdl
+++ b/ipc/ipdl/test/ipdl/ok/Parray_Union.ipdl
@@ -1,10 +1,10 @@
 union Union {
     int[];
     int;
     double;
 };
 
-sync protocol array_Union {
+sync protocol Parray_Union {
 parent:
     sync Msg(Union u, Union[] au) returns (Union r);
 };
rename from ipc/ipdl/test/ipdl/ok/basicUsing.ipdl
rename to ipc/ipdl/test/ipdl/ok/PbasicUsing.ipdl
--- a/ipc/ipdl/test/ipdl/ok/basicUsing.ipdl
+++ b/ipc/ipdl/test/ipdl/ok/PbasicUsing.ipdl
@@ -11,12 +11,12 @@ union SomeUnion
   SomeType;
   SomeClass;
   SomeStruct;
   SomeRefcountedType;
   SomeRefcountedClass;
   SomeRefcountedStruct;
 };
 
-protocol basicUsing {
+protocol PbasicUsing {
 child:
     async Msg(SomeUnion foo);
 };
rename from ipc/ipdl/test/ipdl/ok/builtins.ipdl
rename to ipc/ipdl/test/ipdl/ok/Pbuiltins.ipdl
--- a/ipc/ipdl/test/ipdl/ok/builtins.ipdl
+++ b/ipc/ipdl/test/ipdl/ok/Pbuiltins.ipdl
@@ -1,9 +1,9 @@
-protocol builtins {
+protocol Pbuiltins {
 
     // sanity-check that "essential" builtins are being declared
 
 child: async Msg(bool b,
            char c,
            int i,
            long l,
 
rename from ipc/ipdl/test/ipdl/ok/bytebuf.ipdl
rename to ipc/ipdl/test/ipdl/ok/Pbytebuf.ipdl
--- a/ipc/ipdl/test/ipdl/ok/bytebuf.ipdl
+++ b/ipc/ipdl/test/ipdl/ok/Pbytebuf.ipdl
@@ -1,13 +1,13 @@
 union Foo {
     int;
     ByteBuf;
 };
 
-intr protocol bytebuf {
+intr protocol Pbytebuf {
 parent:
     async Msg(ByteBuf s, Foo f);
     sync SyncMsg(ByteBuf s, Foo f)
         returns (ByteBuf t, Foo g);
     intr InterruptMsg(ByteBuf s, Foo f)
         returns (ByteBuf t, Foo g);
 };
rename from ipc/ipdl/test/ipdl/ok/empty.ipdl
rename to ipc/ipdl/test/ipdl/ok/Pempty.ipdl
--- a/ipc/ipdl/test/ipdl/ok/empty.ipdl
+++ b/ipc/ipdl/test/ipdl/ok/Pempty.ipdl
@@ -1,3 +1,3 @@
-protocol empty {
+protocol Pempty {
 child: async Msg();
 };
rename from ipc/ipdl/test/ipdl/ok/emptyStruct.ipdl
rename to ipc/ipdl/test/ipdl/ok/PemptyStruct.ipdl
--- a/ipc/ipdl/test/ipdl/ok/emptyStruct.ipdl
+++ b/ipc/ipdl/test/ipdl/ok/PemptyStruct.ipdl
@@ -1,3 +1,3 @@
 struct empty { };
 
-protocol emptyStruct { child:  async __delete__(); };
+protocol PemptyStruct { child:  async __delete__(); };
rename from ipc/ipdl/test/ipdl/ok/headerProto.ipdl
rename to ipc/ipdl/test/ipdl/ok/PheaderProto.ipdl
--- a/ipc/ipdl/test/ipdl/ok/headerProto.ipdl
+++ b/ipc/ipdl/test/ipdl/ok/PheaderProto.ipdl
@@ -1,10 +1,10 @@
 include header;
 
 namespace c {
 
-protocol headerProto {
+protocol PheaderProto {
 child:
     async __delete__(foo a, baz b, Inner1 c, Inner2 d, X x);
 };
 
 }
rename from ipc/ipdl/test/ipdl/ok/intrProtocol.ipdl
rename to ipc/ipdl/test/ipdl/ok/PintrProtocol.ipdl
--- a/ipc/ipdl/test/ipdl/ok/intrProtocol.ipdl
+++ b/ipc/ipdl/test/ipdl/ok/PintrProtocol.ipdl
@@ -1,9 +1,9 @@
-intr protocol intrProtocol {
+intr protocol PintrProtocol {
 
     // sanity check of Interrupt protocols
 child:
     async AsyncMsg();
 
 parent:
     sync SyncMsg(int i) returns (int r);
 
rename from ipc/ipdl/test/ipdl/ok/jetpack.ipdl
rename to ipc/ipdl/test/ipdl/ok/Pjetpack.ipdl
--- a/ipc/ipdl/test/ipdl/ok/jetpack.ipdl
+++ b/ipc/ipdl/test/ipdl/ok/Pjetpack.ipdl
@@ -1,5 +1,5 @@
-sync protocol jetpack {
+sync protocol Pjetpack {
 child:
     async __delete__();
 
 };
rename from ipc/ipdl/test/ipdl/ok/manageSelf.ipdl
rename to ipc/ipdl/test/ipdl/ok/PmanageSelf.ipdl
--- a/ipc/ipdl/test/ipdl/ok/manageSelf.ipdl
+++ b/ipc/ipdl/test/ipdl/ok/PmanageSelf.ipdl
@@ -1,10 +1,10 @@
-include protocol manageSelf_Toplevel;
+include protocol PmanageSelf_Toplevel;
 
-protocol manageSelf {
-    manager manageSelf_Toplevel or manageSelf;
-    manages manageSelf;
+protocol PmanageSelf {
+    manager PmanageSelf_Toplevel or PmanageSelf;
+    manages PmanageSelf;
 
 child:
-    async manageSelf();
+    async PmanageSelf();
     async __delete__();
 };
rename from ipc/ipdl/test/ipdl/ok/manageSelf_Toplevel.ipdl
rename to ipc/ipdl/test/ipdl/ok/PmanageSelf_Toplevel.ipdl
--- a/ipc/ipdl/test/ipdl/ok/manageSelf_Toplevel.ipdl
+++ b/ipc/ipdl/test/ipdl/ok/PmanageSelf_Toplevel.ipdl
@@ -1,9 +1,9 @@
-include protocol manageSelf;
+include protocol PmanageSelf;
 
-protocol manageSelf_Toplevel {
-    manages manageSelf;
+protocol PmanageSelf_Toplevel {
+    manages PmanageSelf;
 
 child:
-    async manageSelf();
+    async PmanageSelf();
     async __delete__();
 };
rename from ipc/ipdl/test/ipdl/ok/managedProtocol.ipdl
rename to ipc/ipdl/test/ipdl/ok/PmanagedProtocol.ipdl
--- a/ipc/ipdl/test/ipdl/ok/managedProtocol.ipdl
+++ b/ipc/ipdl/test/ipdl/ok/PmanagedProtocol.ipdl
@@ -1,8 +1,8 @@
-include protocol managerProtocol;
+include protocol PmanagerProtocol;
 
-protocol managedProtocol {
-    manager managerProtocol;
+protocol PmanagedProtocol {
+    manager PmanagerProtocol;
 
 child:
     async __delete__();
 };
rename from ipc/ipdl/test/ipdl/ok/managerProtocol.ipdl
rename to ipc/ipdl/test/ipdl/ok/PmanagerProtocol.ipdl
--- a/ipc/ipdl/test/ipdl/ok/managerProtocol.ipdl
+++ b/ipc/ipdl/test/ipdl/ok/PmanagerProtocol.ipdl
@@ -1,11 +1,11 @@
-include protocol managedProtocol;
+include protocol PmanagedProtocol;
 
 // sanity check of managed/manager protocols
 
-protocol managerProtocol {
-    manages managedProtocol;
+protocol PmanagerProtocol {
+    manages PmanagedProtocol;
 
 parent:
-    async managedProtocol(int i);
+    async PmanagedProtocol(int i);
 
 };
rename from ipc/ipdl/test/ipdl/ok/maybe_Basic.ipdl
rename to ipc/ipdl/test/ipdl/ok/Pmaybe_Basic.ipdl
--- a/ipc/ipdl/test/ipdl/ok/maybe_Basic.ipdl
+++ b/ipc/ipdl/test/ipdl/ok/Pmaybe_Basic.ipdl
@@ -1,4 +1,4 @@
-protocol maybe_Basic {
+protocol Pmaybe_Basic {
 child:
     async Msg(int? maybe);
 };
rename from ipc/ipdl/test/ipdl/ok/maybe_OfActors.ipdl
rename to ipc/ipdl/test/ipdl/ok/Pmaybe_OfActors.ipdl
--- a/ipc/ipdl/test/ipdl/ok/maybe_OfActors.ipdl
+++ b/ipc/ipdl/test/ipdl/ok/Pmaybe_OfActors.ipdl
@@ -1,10 +1,10 @@
-include protocol maybe_OfActorsSub;
+include protocol Pmaybe_OfActorsSub;
 
-protocol maybe_OfActors {
-    manages maybe_OfActorsSub;
+protocol Pmaybe_OfActors {
+    manages Pmaybe_OfActorsSub;
 
 child:
-    async Msg(maybe_OfActorsSub? p);
+    async Msg(Pmaybe_OfActorsSub? p);
 
-    async maybe_OfActorsSub();
+    async Pmaybe_OfActorsSub();
 };
rename from ipc/ipdl/test/ipdl/ok/maybe_OfActorsSub.ipdl
rename to ipc/ipdl/test/ipdl/ok/Pmaybe_OfActorsSub.ipdl
--- a/ipc/ipdl/test/ipdl/ok/maybe_OfActorsSub.ipdl
+++ b/ipc/ipdl/test/ipdl/ok/Pmaybe_OfActorsSub.ipdl
@@ -1,7 +1,7 @@
-include protocol maybe_OfActors;
+include protocol Pmaybe_OfActors;
 
-protocol maybe_OfActorsSub {
-    manager maybe_OfActors;
+protocol Pmaybe_OfActorsSub {
+    manager Pmaybe_OfActors;
 
 child: async __delete__();
 };
rename from ipc/ipdl/test/ipdl/ok/maybe_Union.ipdl
rename to ipc/ipdl/test/ipdl/ok/Pmaybe_Union.ipdl
--- a/ipc/ipdl/test/ipdl/ok/maybe_Union.ipdl
+++ b/ipc/ipdl/test/ipdl/ok/Pmaybe_Union.ipdl
@@ -1,10 +1,10 @@
 union MaybeUnion {
     int[];
     int;
     double;
 };
 
-sync protocol maybe_Union {
+sync protocol Pmaybe_Union {
 parent:
     async Msg(MaybeUnion u, MaybeUnion? au) returns (MaybeUnion r);
 };
rename from ipc/ipdl/test/ipdl/ok/media.ipdl
rename to ipc/ipdl/test/ipdl/ok/Pmedia.ipdl
--- a/ipc/ipdl/test/ipdl/ok/media.ipdl
+++ b/ipc/ipdl/test/ipdl/ok/Pmedia.ipdl
@@ -1,5 +1,5 @@
-sync protocol media {
+sync protocol Pmedia {
 child:
     async __delete__();
 
 };
rename from ipc/ipdl/test/ipdl/ok/messageCompress.ipdl
rename to ipc/ipdl/test/ipdl/ok/PmessageCompress.ipdl
--- a/ipc/ipdl/test/ipdl/ok/messageCompress.ipdl
+++ b/ipc/ipdl/test/ipdl/ok/PmessageCompress.ipdl
@@ -1,4 +1,4 @@
-intr protocol messageCompress {
+intr protocol PmessageCompress {
 child:
     async foo() compress;
 };
rename from ipc/ipdl/test/ipdl/ok/messageVerify.ipdl
rename to ipc/ipdl/test/ipdl/ok/PmessageVerify.ipdl
--- a/ipc/ipdl/test/ipdl/ok/messageVerify.ipdl
+++ b/ipc/ipdl/test/ipdl/ok/PmessageVerify.ipdl
@@ -1,12 +1,12 @@
-include protocol messageVerifyTopLevel;
+include protocol PmessageVerifyTopLevel;
 
-intr protocol messageVerify {
-  manager messageVerifyTopLevel;
+intr protocol PmessageVerify {
+  manager PmessageVerifyTopLevel;
 
 parent:
     sync __delete__(uint32_t x) returns (double rv1) verify;
     async msg1() verify;
     async msg2(uint32_t aParam1) verify;
     sync msg3()
       returns (uint32_t rv1) verify;
     sync msg4(uint32_t aParam1)
rename from ipc/ipdl/test/ipdl/ok/messageVerifyTopLevel.ipdl
rename to ipc/ipdl/test/ipdl/ok/PmessageVerifyTopLevel.ipdl
--- a/ipc/ipdl/test/ipdl/ok/messageVerifyTopLevel.ipdl
+++ b/ipc/ipdl/test/ipdl/ok/PmessageVerifyTopLevel.ipdl
@@ -1,7 +1,7 @@
-include protocol messageVerify;
+include protocol PmessageVerify;
 
-intr protocol messageVerifyTopLevel{
-  manages messageVerify;
+intr protocol PmessageVerifyTopLevel{
+  manages PmessageVerify;
   parent:
-    sync messageVerify(uint32_t aParam1) returns (double rv1) verify;
+    sync PmessageVerify(uint32_t aParam1) returns (double rv1) verify;
 };
rename from ipc/ipdl/test/ipdl/ok/multiManaged.ipdl
rename to ipc/ipdl/test/ipdl/ok/PmultiManaged.ipdl
--- a/ipc/ipdl/test/ipdl/ok/multiManaged.ipdl
+++ b/ipc/ipdl/test/ipdl/ok/PmultiManaged.ipdl
@@ -1,9 +1,9 @@
-include protocol multiManager1;
-include protocol multiManager2;
+include protocol PmultiManager1;
+include protocol PmultiManager2;
 
-protocol multiManaged {
-    manager multiManager1 or multiManager2;
+protocol PmultiManaged {
+    manager PmultiManager1 or PmultiManager2;
 
 child:
     async __delete__();
 };
rename from ipc/ipdl/test/ipdl/ok/multiManager1.ipdl
rename to ipc/ipdl/test/ipdl/ok/PmultiManager1.ipdl
--- a/ipc/ipdl/test/ipdl/ok/multiManager1.ipdl
+++ b/ipc/ipdl/test/ipdl/ok/PmultiManager1.ipdl
@@ -1,8 +1,8 @@
-include protocol multiManaged;
+include protocol PmultiManaged;
 
-protocol multiManager1 {
-    manages multiManaged;
+protocol PmultiManager1 {
+    manages PmultiManaged;
 
 child:
-    async multiManaged();
+    async PmultiManaged();
 };
rename from ipc/ipdl/test/ipdl/ok/multiManager2.ipdl
rename to ipc/ipdl/test/ipdl/ok/PmultiManager2.ipdl
--- a/ipc/ipdl/test/ipdl/ok/multiManager2.ipdl
+++ b/ipc/ipdl/test/ipdl/ok/PmultiManager2.ipdl
@@ -1,8 +1,8 @@
-include protocol multiManaged;
+include protocol PmultiManaged;
 
-protocol multiManager2 {
-    manages multiManaged;
+protocol PmultiManager2 {
+    manages PmultiManaged;
 
 child:
-    async multiManaged();
+    async PmultiManaged();
 };
rename from ipc/ipdl/test/ipdl/ok/multipleUsingCxxTypes.ipdl
rename to ipc/ipdl/test/ipdl/ok/PmultipleUsingCxxTypes.ipdl
--- a/ipc/ipdl/test/ipdl/ok/multipleUsingCxxTypes.ipdl
+++ b/ipc/ipdl/test/ipdl/ok/PmultipleUsingCxxTypes.ipdl
@@ -1,7 +1,7 @@
 using struct mozilla::void_t from "ipc/IPCMessageUtils.h";
 using struct mozilla::void_t from "ipc/IPCMessageUtils.h";
 
-protocol multipleUsingCxxTypes {
+protocol PmultipleUsingCxxTypes {
 child:
     async Msg(void_t foo);
 };
rename from ipc/ipdl/test/ipdl/ok/mutualRecStructUnion.ipdl
rename to ipc/ipdl/test/ipdl/ok/PmutualRecStructUnion.ipdl
--- a/ipc/ipdl/test/ipdl/ok/mutualRecStructUnion.ipdl
+++ b/ipc/ipdl/test/ipdl/ok/PmutualRecStructUnion.ipdl
@@ -9,13 +9,13 @@ union Y {
     Z;
 };
 
 struct Z {
     X x;
     Y y;
 };
 
-protocol mutualRecStructUnion {
+protocol PmutualRecStructUnion {
 child:
     async Test(X x, Y y, Z z);
     async __delete__();
 };
rename from ipc/ipdl/test/ipdl/ok/mutualRecUnion.ipdl
rename to ipc/ipdl/test/ipdl/ok/PmutualRecUnion.ipdl
--- a/ipc/ipdl/test/ipdl/ok/mutualRecUnion.ipdl
+++ b/ipc/ipdl/test/ipdl/ok/PmutualRecUnion.ipdl
@@ -9,13 +9,13 @@ union Y {
     Z;
 };
 
 union Z {
     double;
     X;
 };
 
-protocol mutualRecUnion {
+protocol PmutualRecUnion {
 child:
     async Test(X x, Y y, Z z);
     async __delete__();
 };
rename from ipc/ipdl/test/ipdl/ok/namespace_Basic.ipdl
rename to ipc/ipdl/test/ipdl/ok/Pnamespace_Basic.ipdl
--- a/ipc/ipdl/test/ipdl/ok/namespace_Basic.ipdl
+++ b/ipc/ipdl/test/ipdl/ok/Pnamespace_Basic.ipdl
@@ -1,12 +1,12 @@
 namespace basic {
 
 // sanity check of namespaced protocols
 
-protocol namespace_Basic {
+protocol Pnamespace_Basic {
 
 child:
     async Msg();
 
 };
 
 } // namespace basic
rename from ipc/ipdl/test/ipdl/ok/noRedeclCrossMessage.ipdl
rename to ipc/ipdl/test/ipdl/ok/PnoRedeclCrossMessage.ipdl
--- a/ipc/ipdl/test/ipdl/ok/noRedeclCrossMessage.ipdl
+++ b/ipc/ipdl/test/ipdl/ok/PnoRedeclCrossMessage.ipdl
@@ -1,9 +1,9 @@
-protocol noRedeclCrossMessage {
+protocol PnoRedeclCrossMessage {
 
     // each message has its own scope for param/return names
 
 child:
     async Msg1(int f);
     async Msg2(int f);
 
 };
rename from ipc/ipdl/test/ipdl/ok/plugin.ipdl
rename to ipc/ipdl/test/ipdl/ok/Pplugin.ipdl
--- a/ipc/ipdl/test/ipdl/ok/plugin.ipdl
+++ b/ipc/ipdl/test/ipdl/ok/Pplugin.ipdl
@@ -1,5 +1,5 @@
-intr protocol plugin {
+intr protocol Pplugin {
 child:
     async __delete__();
 
 };
rename from ipc/ipdl/test/ipdl/ok/selfRecUnion.ipdl
rename to ipc/ipdl/test/ipdl/ok/PselfRecUnion.ipdl
--- a/ipc/ipdl/test/ipdl/ok/selfRecUnion.ipdl
+++ b/ipc/ipdl/test/ipdl/ok/PselfRecUnion.ipdl
@@ -1,11 +1,11 @@
 union R {
     int;
     double;
     R;
 };
 
-protocol selfRecUnion {
+protocol PselfRecUnion {
 child:
     async Test(R r);
     async __delete__();
 };
rename from ipc/ipdl/test/ipdl/ok/shmem.ipdl
rename to ipc/ipdl/test/ipdl/ok/Pshmem.ipdl
--- a/ipc/ipdl/test/ipdl/ok/shmem.ipdl
+++ b/ipc/ipdl/test/ipdl/ok/Pshmem.ipdl
@@ -1,13 +1,13 @@
 union Foo {
     int;
     Shmem;
 };
 
-intr protocol shmem {
+intr protocol Pshmem {
 parent:
     async Msg(Shmem s, Foo f);
     sync SyncMsg(Shmem s, Foo f)
         returns (Shmem t, Foo g);
     intr InterruptMsg(Shmem s, Foo f)
         returns (Shmem t, Foo g);
 };
rename from ipc/ipdl/test/ipdl/ok/syncProtocol.ipdl
rename to ipc/ipdl/test/ipdl/ok/PsyncProtocol.ipdl
--- a/ipc/ipdl/test/ipdl/ok/syncProtocol.ipdl
+++ b/ipc/ipdl/test/ipdl/ok/PsyncProtocol.ipdl
@@ -1,9 +1,9 @@
-sync protocol syncProtocol {
+sync protocol PsyncProtocol {
 
     // sanity check of sync protocols
 
 child:
     async AsyncMsg();
 
 parent:
     sync SyncMsg() returns (int i);
rename from ipc/ipdl/test/ipdl/ok/threeDirections.ipdl
rename to ipc/ipdl/test/ipdl/ok/PthreeDirections.ipdl
--- a/ipc/ipdl/test/ipdl/ok/threeDirections.ipdl
+++ b/ipc/ipdl/test/ipdl/ok/PthreeDirections.ipdl
@@ -1,9 +1,9 @@
-protocol threeDirections {
+protocol PthreeDirections {
 
     // sanity check that the three direction specifiers are being accepted
 child:
     async ChildMsg();
 
 parent:
     async ParentMsg();
 
rename from ipc/ipdl/test/ipdl/ok/union_Basic.ipdl
rename to ipc/ipdl/test/ipdl/ok/Punion_Basic.ipdl
--- a/ipc/ipdl/test/ipdl/ok/union_Basic.ipdl
+++ b/ipc/ipdl/test/ipdl/ok/Punion_Basic.ipdl
@@ -1,11 +1,11 @@
 union Basic {
     int;
     double;
 };
 
-sync protocol union_Basic {
+sync protocol Punion_Basic {
 
 parent:
     sync Msg(Basic p) returns (Basic r);
 
 };
rename from ipc/ipdl/test/ipdl/ok/union_Namespaced.ipdl
rename to ipc/ipdl/test/ipdl/ok/Punion_Namespaced.ipdl
--- a/ipc/ipdl/test/ipdl/ok/union_Namespaced.ipdl
+++ b/ipc/ipdl/test/ipdl/ok/Punion_Namespaced.ipdl
@@ -5,14 +5,14 @@ union Socks {
     double;
 };
 
 } // namespace kitties
 
 
 namespace puppies {
 
-protocol union_Namespaced {
+protocol Punion_Namespaced {
 child:
     async Msg(Socks s);
 };
 
 } // namespace puppies
--- a/ipc/ipdl/test/ipdl/sync-messages.ini
+++ b/ipc/ipdl/test/ipdl/sync-messages.ini
@@ -1,24 +1,24 @@
-[actorreturn::Msg]
-[array_Union::Msg]
-[union_Basic::Msg]
-[Struct::test]
-[intrProtocol::SyncMsg]
-[intrProtocol::InterruptMsg]
-[shmem::SyncMsg]
-[shmem::InterruptMsg]
-[bytebuf::SyncMsg]
-[bytebuf::InterruptMsg]
-[syncProtocol::SyncMsg]
-[messageVerify::__delete__]
-[messageVerify::msg3]
-[messageVerify::msg4]
-[messageVerifyTopLevel::messageVerify]
-[DeleteSub::__delete__]
-[intrMessageCompress::foo]
-[intrMessageCompress::bar]
-[syncMessageCompress::foo]
-[syncParentToChild::Msg]
-[tooWeakIntrSync::Msg]
-[tooWeakSyncAsync::Msg]
-[undeclReturnType::Msg]
-[asyncMessageListed::Msg]
+[Pactorreturn::Msg]
+[Parray_Union::Msg]
+[Punion_Basic::Msg]
+[PStruct::test]
+[PintrProtocol::SyncMsg]
+[PintrProtocol::InterruptMsg]
+[Pshmem::SyncMsg]
+[Pshmem::InterruptMsg]
+[Pbytebuf::SyncMsg]
+[Pbytebuf::InterruptMsg]
+[PsyncProtocol::SyncMsg]
+[PmessageVerify::__delete__]
+[PmessageVerify::msg3]
+[PmessageVerify::msg4]
+[PmessageVerifyTopLevel::PmessageVerify]
+[PDeleteSub::__delete__]
+[PintrMessageCompress::foo]
+[PintrMessageCompress::bar]
+[PsyncMessageCompress::foo]
+[PsyncParentToChild::Msg]
+[PtooWeakIntrSync::Msg]
+[PtooWeakSyncAsync::Msg]
+[PundeclReturnType::Msg]
+[PasyncMessageListed::Msg]