Bug 1329182 - remove trailing newURI null parameters in XBL bindings, r=jaws.
authorFlorian Quèze <florian@queze.net>
Mon, 09 Jan 2017 20:27:25 +0100
changeset 328550 35cb8eb1856d275ca3f2e849974e5b33343ebab2
parent 328549 52f8121f00650836089d6f309aee266c7b5f878d
child 328551 3e0d77afb2e04228f6878c228cab954eb793540e
push id85477
push userflorian@queze.net
push dateMon, 09 Jan 2017 19:28:21 +0000
treeherdermozilla-inbound@79c7e3434630 [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
reviewersjaws
bugs1329182
milestone53.0a1
first release with
nightly linux32
nightly linux64
nightly mac
nightly win32
nightly win64
last release without
nightly linux32
nightly linux64
nightly mac
nightly win32
nightly win64
Bug 1329182 - remove trailing newURI null parameters in XBL bindings, r=jaws.
toolkit/content/widgets/autocomplete.xml
toolkit/content/widgets/remote-browser.xml
toolkit/content/widgets/text.xml
--- a/toolkit/content/widgets/autocomplete.xml
+++ b/toolkit/content/widgets/autocomplete.xml
@@ -2135,17 +2135,17 @@ extends="chrome://global/content/binding
           }
           this.setAttribute("displayurl", displayUrl);
 
           // Show the domain as the title if we don't have a title.
           if (!title) {
             title = displayUrl;
             titleLooksLikeUrl = true;
             try {
-              let uri = Services.io.newURI(originalUrl, null, null);
+              let uri = Services.io.newURI(originalUrl);
               // Not all valid URLs have a domain.
               if (uri.host)
                 title = uri.host;
             } catch (e) {}
           }
 
           this._tags.setAttribute("empty", "true");
 
--- a/toolkit/content/widgets/remote-browser.xml
+++ b/toolkit/content/widgets/remote-browser.xml
@@ -366,17 +366,17 @@
 
           this._remoteWebNavigation = Components.classes["@mozilla.org/remote-web-navigation;1"]
                                                 .createInstance(Components.interfaces.nsIWebNavigation);
           this._remoteWebNavigationImpl = this._remoteWebNavigation.wrappedJSObject;
           this._remoteWebNavigationImpl.swapBrowser(this);
 
           // Initialize contentPrincipal to the about:blank principal for this loadcontext
           let {Services} = Components.utils.import("resource://gre/modules/Services.jsm", {});
-          let aboutBlank = Services.io.newURI("about:blank", null, null);
+          let aboutBlank = Services.io.newURI("about:blank");
           let ssm = Services.scriptSecurityManager;
           this._contentPrincipal = ssm.getLoadContextCodebasePrincipal(aboutBlank, this.loadContext);
 
           this.messageManager.addMessageListener("Browser:Init", this);
           this.messageManager.addMessageListener("DOMTitleChanged", this);
           this.messageManager.addMessageListener("ImageDocumentLoaded", this);
           this.messageManager.addMessageListener("FullZoomChange", this);
           this.messageManager.addMessageListener("TextZoomChange", this);
--- a/toolkit/content/widgets/text.xml
+++ b/toolkit/content/widgets/text.xml
@@ -305,17 +305,17 @@
             const secMan =
                      Components.classes["@mozilla.org/scriptsecuritymanager;1"]
                                .getService(nsISSM);
 
             const ioService =
                      Components.classes["@mozilla.org/network/io-service;1"]
                                .getService(Components.interfaces.nsIIOService);
 
-            uri = ioService.newURI(href, null, null);
+            uri = ioService.newURI(href);
 
             let principal;
             if (this.getAttribute("useoriginprincipal") == "true") {
               principal = this.nodePrincipal;
             } else {
               principal = secMan.createNullPrincipal({});
             }
             try {