I think it would make more sense to have the C++ default to false, remove the "true" from all.js, and enable it per-app. This way embeddors won't accidentally ship ping enabled without user-visible UI for it... Also, this way non-browser Gecko consumers won't be affected.
Status: NEW → ASSIGNED
OS: Linux → All
Hardware: PC → All
Target Milestone: --- → mozilla1.9alpha
Created attachment 210645 [details] [diff] [review] v1 patch My recent patch to introduce nsIWebBrowserChrome2, also had the effect of disabling <a ping> by default in the code. I decided to leave the pref defined in all.js so that it would appear in about:config in products where it is not explicitly defined. Otherwise, this patch just moves the pref to firefox.js, so that it only affects firefox.
Status: ASSIGNED → RESOLVED
Last Resolved: 13 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.