Closed Bug 1518922 Opened 11 months ago Closed 11 months ago
simplify setting up a tmpdir for crashreporting
Summary: simply setting up a tmpdir for crashreporting → simplify setting up a tmpdir for crashreporting
The command-line parameter used by nsEmbedFunctions.cpp is turned into an nsIFile, and then said nsIFile is never used. Its last use was deleted in bug 1407693, where we reworked how extra annotations were done.
Attachment #9035686 - Flags: review?(agaynor)
After part 1, this function and the code associated with it is no longer used. The bug 1257098 reference makes me a little nervous, as that bug isn't fixed yet (maybe it no longer applies?), but given that I can't see exactly what difference this code makes in that case...I'll be doing a try run with these patches shortly.
Attachment #9035688 - Flags: review?(gsvelto)
Attachment #9035686 - Flags: review?(agaynor) → review+
Pushed by email@example.com: https://hg.mozilla.org/integration/mozilla-inbound/rev/4ef4c81995f6 part 1 - remove dead code for extra crashreporting directory; r=Alex_Gaynor https://hg.mozilla.org/integration/mozilla-inbound/rev/20e70e44d305 part 2 - remove GetChildProcessTmpDir; r=gsvelto
10 months ago
Assignee: nobody → nfroyd
You need to log in before you can comment on or make changes to this bug.