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 fix (deleted) — Splinter 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+
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.

Attachment

General

Created:
Updated:
Size: