Closed Bug 1450242 Opened 3 years ago Closed 3 years ago
Remove useless constants at the top of ns
Browser Content Handler .js
After the automated rewrite from bug 1433175, we are left with plenty of useless nsI* constants. This is especially obvious in nsBrowserContentHandler.js
Simple change done with a regexp for all the Ci. cases. Then I did by hand the Cr. cases, fixed the wrapping of a few long lines, and merged logSystemBasedSearch into its single caller, because separating that function was useless.
Attachment #8963925 - Flags: review?(standard8)
Of course I'll revert the unintended npm-shrinkwrap.json changes in the next version.
Attachment #8963925 - Flags: review?(standard8) → review+
Pushed by firstname.lastname@example.org: https://hg.mozilla.org/integration/mozilla-inbound/rev/fe0e01b08506 Remove useless constants at the top of nsBrowserContentHandler.js, r=Standard8.
You need to log in before you can comment on or make changes to this bug.