Closed Bug 1222829 Opened 9 years ago Closed 9 years ago

Remove URIChecker

Categories

(Core :: Networking, defect)

defect
Not set
normal

Tracking

()

RESOLVED FIXED
mozilla45
Tracking Status
firefox45 --- fixed

People

(Reporter: sicking, Unassigned)

References

Details

(Keywords: addon-compat)

Attachments

(1 file)

Attached patch Patch to fixSplinter Review
It doesn't seem to be used. There hasn't been any "real" changes to this code since we migrated to hg. All changes are global changes that happened to touch urichecker.cpp/h as well.

http://mxr.mozilla.org/mozilla-central/search?string=urichecker

The only risk here is that addons might use it. If that's the case we can always put it back.
Attachment #8684690 - Flags: review?(mcmanus)
Comment on attachment 8684690 [details] [diff] [review]
Patch to fix

thanks
Attachment #8684690 - Flags: review?(mcmanus) → review+
https://hg.mozilla.org/mozilla-central/rev/a5d633bb299c
Status: NEW → RESOLVED
Closed: 9 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla45
Is there a way of checking for valid links other than using the removed code?
https://dxr.mozilla.org/comm-central/search?q=urichecker&redirect=false
Flags: needinfo?(mcmanus)
Flags: needinfo?(mcmanus) → needinfo?(valentin.gosu)
(In reply to Ian Neal from comment #5)
> Is there a way of checking for valid links other than using the removed code?
> https://dxr.mozilla.org/comm-central/search?q=urichecker&redirect=false

If by valid links you mean valid URLs, I think creating a URL and seeing if the result is NS_OK should work.
I think calling NS_NewURI is the preferred way of creating a URL. If you need it to be a URL (instead of a URI) check if the result QIs to nsIStandardURL.

If you want to check if the link is loadable, you can do something similar to what the code used to do - open a channel to the URL and see if it loads.
Flags: needinfo?(valentin.gosu)
You need to log in before you can comment on or make changes to this bug.