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 35cb8eb1856d
parent 328549 52f8121f0065
child 328551 3e0d77afb2e0
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 {