This will allow search engines to specify a keyword-specific <Url>, if using the default <Url type="text/html"> isn't suitable (e.g. if you want different parameters).
Attachment #466363 - Flags: review?(community) → review?(l10n)
That patch also makes keyword.url non-localizable, since there's no longer a reason for it to be.
Comment on attachment 466363 [details] [diff] [review] patch Looks good to me, not that I have a good idea on the nsDefaultURIFixup.cpp part, I guess jst is up for review on that?
Attachment #466363 - Flags: review?(l10n) → review+
Discussed this with Jeff, added a comment with followup accordingly.
Comment on attachment 466414 [details] [diff] [review] updated patch Looks fine, but add a comment describing why you're: pref("keyword.URL", ""); and maybe mention the future bug near: +const URLTYPE_MOZ_KEYWORDSEARCH = "application/x-moz-keywordsearch";
Attachment #466414 - Flags: review?(jmuizelaar) → review+
Patch summary: - makes change from bug 586821 apply to all locales - additional followup special-case bug 587730 for "ru" specifically is ready to land, coordinated with axel and ru localizer - removes a string (minimal l10n impact) - no code-level regression risk
blocking2.0: --- → beta4+
Status: ASSIGNED → RESOLVED
Closed: 9 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla2.0b4
You need to log in before you can comment on or make changes to this bug.