Closed Bug 1476928 Opened 2 years ago Closed 2 years ago
IURI .Clone Ignoring Ref and ns IURI .Clone With New Ref
46 bytes, text/x-phabricator-request
|Details | Review|
These methods are now irrelevant, since cloning is not needed anymore. Moreover, this allows us to optimize these operations, since we don't need to clone/mutate when the ref we are trying to set (be it empty or not) already matches the ref on the URI. In that case, the "clone" becomes just an addref.
The patch introduces NS_GetURIWithNewRef and NS_GetURIWithNewRef which perform the same function.
Comment on attachment 8993318 [details] Bug 1476928 - Remove nsIURI.CloneIgnoringRef and nsIURI.CloneWithNewRef Junior Hsu has approved the revision. https://phabricator.services.mozilla.com/D2239
Attachment #8993318 - Flags: review+
Pushed by firstname.lastname@example.org: https://hg.mozilla.org/integration/autoland/rev/5288e1d49e71 Remove nsIURI.CloneIgnoringRef and nsIURI.CloneWithNewRef r=JuniorHsu
Whiteboard: [necko-triaged] → [necko-triaged][MemShrink][qf]
You need to log in before you can comment on or make changes to this bug.