Opened on 05/14/2014 at 10:57:30 AM

Closed on 11/10/2017 at 11:15:18 AM

#483 closed change (rejected)

Evaluate replacing Utils.makeURI() and by URL constructor

Reported by: trev Assignee:
Priority: P4 Milestone:
Module: Adblock-Plus-for-Firefox Keywords:
Cc: Blocked By:
Blocking: Platform: Firefox
Ready: yes Confidential: no
Tester: Verified working: no
Review URL(s):



We are currently using and occasionally Utils.makeURI() to construct nsIURI instances. As of Firefox 19 the URL constructor is a faster and simpler alternative. It is also available in Chrome but unfortunately not in Safari yet (Safari will happily construct URL objects but not provide any properties on them whatsoever).

What to change

Evaluate where in our code base we can use URL constructor. I guess that it won't be possible anywhere where XPCOM is involved: URL doesn't seem to implement the nsIURI interface. Also, it won't be possible when we verify website signatures: we need nsIURI.asciiHost to get the Punycode-encoded value. Filter composer should be able to use URL however, Subscription constructor and other places validating URLs probably as well.

Attachments (0)

Change History (2)

comment:1 Changed on 07/09/2014 at 12:38:11 PM by philll

  • Platform set to Firefox

comment:2 Changed on 11/10/2017 at 11:15:18 AM by trev

  • Resolution set to rejected
  • Status changed from new to closed

Mass-closing all bugs in Adblock Plus for Firefox module, the codebase of Adblock Plus 3.0 belongs into Platform and User-Interface modules. Old bugs are unlikely to still apply.

Add Comment

Modify Ticket

Change Properties
as closed .
The resolution will be deleted. Next status will be 'reopened'.
to The owner will be changed from (none).
Note: See TracTickets for help on using tickets.