bug 539552 - Add a GetMinidump() method to top-level, parent-side IPDL classes. r=bent

ipc/ipdl/ipdl/builtin.py

@@ -90,5 +90,6 @@ Includes = (
     'nsAutoPtr.h',
     'nsStringGlue.h',
     'nsTArray.h',
+    'nsIFile.h',
     'mozilla/ipc/ProtocolUtils.h',
 )

ipc/ipdl/ipdl/cxx/ast.py

@@ -275,7 +275,7 @@ class File(Node):
 
 class CppDirective(Node):
     '''represents |#[directive] [rest]|, where |rest| is any string'''
-    def __init__(self, directive, rest):
+    def __init__(self, directive, rest=''):
         Node.__init__(self)
         self.directive = directive
         self.rest = rest

ipc/ipdl/ipdl/lower.py

@@ -2448,14 +2448,23 @@ class _GenerateProtocolActorCode(ipdl.ast.Visitor):
             + _includeGuardEnd(hf))
 
         # make the .cpp file
-        cf.addthings((
+        cf.addthings([
-            [ disclaimer,
+            disclaimer,
             Whitespace.NL,
             CppDirective(
                 'include',
-                  '"'+ _protocolHeaderName(self.protocol, self.side) +'.h"'),
+                '"'+ _protocolHeaderName(self.protocol, self.side) +'.h"')
-              Whitespace.NL
+        ])
-            ]
+             
+        if self.protocol.decl.type.isToplevel():
+            cf.addthings([
+                CppDirective('ifdef', 'MOZ_CRASHREPORTER'),
+                CppDirective('  include', '"nsXULAppAPI.h"'),
+                CppDirective('endif')
+            ])
+
+        cf.addthings((
+            [ Whitespace.NL ]
             + self.protocolCxxIncludes
             + [ Whitespace.NL ]
             + self.standardTypedefs()
@@ -2807,6 +2816,39 @@ class _GenerateProtocolActorCode(ipdl.ast.Visitor):
         if p.usesShmem():
             self.cls.addstmts(self.makeShmemIface())
 
+        if p.decl.type.isToplevel() and self.side is 'parent':
+            ## bool GetMinidump(nsIFile** dump)
+            self.cls.addstmt(Label.PROTECTED)
+
+            otherpidvar = ExprVar('OtherSidePID')
+            otherpid = MethodDefn(MethodDecl(
+                otherpidvar.name, params=[ ],
+                ret=Type('base::ProcessId'),
+                const=1))
+            otherpid.addstmts([
+                StmtReturn(ExprCall(
+                    ExprVar('base::GetProcId'),
+                    args=[ p.otherProcessVar() ])),
+            ])
+
+            dumpvar = ExprVar('aDump')
+            getdump = MethodDefn(MethodDecl(
+                'GetMinidump',
+                params=[ Decl(Type('nsIFile', ptrptr=1), dumpvar.name) ],
+                ret=Type.BOOL,
+                const=1))
+            getdump.addstmts([
+                CppDirective('ifdef', 'MOZ_CRASHREPORTER'),
+                StmtReturn(ExprCall(
+                    ExprVar('XRE_GetMinidumpForChild'),
+                    args=[ ExprCall(otherpidvar), dumpvar ])),
+                CppDirective('else'),
+                StmtReturn(ExprLiteral.FALSE),
+                CppDirective('endif')
+            ])
+            self.cls.addstmts([ otherpid, Whitespace.NL,
+                                getdump, Whitespace.NL ])
+
         ## private methods
         self.cls.addstmt(Label.PRIVATE)