Created attachment 558053 [details] [diff] [review]
I think you own this...
Comment on attachment 558053 [details] [diff] [review]
>diff --git a/embedding/components/printingui/src/unixshared/nsPrintingPromptService.cpp b/embedding/components/printingui/src/unixshared/nsPrintingPromptService.cpp
>+ nsCOMPtr<nsIWebProgressListener> myWebProgressListener = this;
This is weird. The normal way to do this is NS_ADDREF(*webProgressListener = this);
r=me with that change. I didn't review the OS/2 bits at all, since I just don't care.
I'm no fan of NS_ADDREF, but sure.