+++ This bug was initially created as a clone of Bug #1491252 +++ See trimValue in urlbarBindings.xml.

This makes browser/base/content/test/urlbar/browser_urlbarTrimURLs.js pass

test_UrlbarInput_unit.js fixed

Comment on attachment 9011756 [details] [diff] [review] patch Review of attachment 9011756 [details] [diff] [review]: ----------------------------------------------------------------- ::: browser/components/urlbar/UrlbarPrefs.jsm @@ +170,5 @@ > * Returns the value for the preference with the given name. > * > * @param {string} pref > * The name of the preference to get. > + * @returns {string} The preference value. Digging into this, I think we currently return strings, arrays, booleans and integers. So we could make the value `{string|array|boolean|number}`, but I'm wonder if it is just better to go with `{*}` - basically admit it could be almost anything. (same with the changes below).

Pushed by nerli@mozilla.com: https://hg.mozilla.org/integration/mozilla-inbound/rev/8cb351adc7d3 Port URL trimming from urlbarBindings.xml to UrlbarInput.jsm. r=standard8

Backout by aiakab@mozilla.com: https://hg.mozilla.org/integration/mozilla-inbound/rev/d7a945fc6271 Backed out changeset 8cb351adc7d3 for failing test verify on browser_urlbarTrimURLs.js

Pushed by ryanvm@gmail.com: https://hg.mozilla.org/integration/mozilla-inbound/rev/184a229a8cc4 Port URL trimming from urlbarBindings.xml to UrlbarInput.jsm. r=standard8