Most callers do not set params, paramsLength, URI, source, lineNumber. columnNumber is not used at all! Most callers do not set non-zero windowID. So I propose the follwing changes: - Move flags, category and doc params onto other rarer used params and make the rest params optional. - Remove the windowID version method.
Created attachment 576481 [details] [diff] [review] patch
Looks like the only caller that passes a raw window ID is the nsIDOMWindowInternal one, so it's probably fine to remove that variant.
Created attachment 577582 [details] [diff] [review] patch Updated to tip
Please mark this for check-in once the dependencies are cleared.
Created attachment 580453 [details] [diff] [review] Simplify nsContentUtils::ReportToConsole. r=smaug Updated to tip
The dependent bug has been fixed.
In my queue with a few other checkin-neededs that are being sent to try first and then onto inbound :-) https://tbpl.mozilla.org/?tree=Try&rev=fd440327d5e4