Bug 1329182 - Make newURI's last two parameters optional, r=mcmanus.
authorFlorian Quèze <florian@queze.net>
Mon, 09 Jan 2017 20:27:24 +0100
changeset 328549 52f8121f00650836089d6f309aee266c7b5f878d
parent 328548 8f6416404aeac786f727c76d609e00bd4612bf1e
child 328550 35cb8eb1856d275ca3f2e849974e5b33343ebab2
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)
reviewersmcmanus
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 - Make newURI's last two parameters optional, r=mcmanus.
netwerk/base/nsIIOService.idl
netwerk/base/nsIProtocolHandler.idl
--- a/netwerk/base/nsIIOService.idl
+++ b/netwerk/base/nsIIOService.idl
@@ -45,18 +45,18 @@ interface nsIIOService : nsISupports
      * This method constructs a new URI by determining the scheme of the
      * URI spec, and then delegating the construction of the URI to the
      * protocol handler for that scheme. QueryInterface can be used on
      * the resulting URI object to obtain a more specific type of URI.
      *
      * @see nsIProtocolHandler::newURI
      */
     nsIURI newURI(in AUTF8String aSpec,
-                  in string aOriginCharset,
-                  in nsIURI aBaseURI);
+                  [optional] in string aOriginCharset,
+                  [optional] in nsIURI aBaseURI);
 
     /**
      * This method constructs a new URI from a nsIFile.
      *
      * @param aFile specifies the file path
      * @return reference to a new nsIURI object
      *
      * Note: in the future, for perf reasons we should allow 
--- a/netwerk/base/nsIProtocolHandler.idl
+++ b/netwerk/base/nsIProtocolHandler.idl
@@ -82,18 +82,18 @@ interface nsIProtocolHandler : nsISuppor
      *                         no charset transformation from aSpec).
      * @param aBaseURI       - if null, aSpec must specify an absolute URI.
      *                         otherwise, aSpec may be resolved relative
      *                         to aBaseURI, depending on the protocol. 
      *                         If the protocol has no concept of relative 
      *                         URI aBaseURI will simply be ignored.
      */
     nsIURI newURI(in AUTF8String aSpec,
-                  in string aOriginCharset,
-                  in nsIURI aBaseURI);
+                  [optional] in string aOriginCharset,
+                  [optional] in nsIURI aBaseURI);
 
     /**
      * Constructs a new channel from the given URI for this protocol handler and
      * sets the loadInfo for the constructed channel.
      */
     nsIChannel newChannel2(in nsIURI aURI, in nsILoadInfo aLoadinfo);
 
     /**