Open Bug 1560639 Opened 5 years ago Updated 2 years ago

Clean up Mailnews URL handling after bug 1550945 - take 2

Categories

(MailNews Core :: General, task)

Tracking

(Not tracked)

People

(Reporter: jorgk-bmo, Unassigned)

References

Details

Attachments

(1 file)

+++ This bug was initially created as a clone of Bug #1555633 +++
+++ This bug was initially created as a clone of Bug #1550945 +++

In bug 1536744, M-C removed nsIProtocolHandler.newURI and hard-coded all schemes in the system here:
https://searchfox.org/mozilla-central/rev/c0ca77697c6868482f30af873ec8069f2c080a34/netwerk/base/nsNetUtil.cpp#1707

We followed with
https://searchfox.org/comm-central/source/mailnews/base/util/nsNewMailnewsURI.cpp

So while bug 1550945 fixed the bustage, there is scope for some clean-up:

  1. Maybe fix the hard-coded include paths at the top of the file
  2. Make sure JS Account is adequately catered for (done in bug 1555633)
  3. Consider removing some "simple" URL implementations, see bug 1550945 comment #0, quote: "remove some of the implementations".
Attached patch Starting point (deleted) — Splinter Review

I guess so, but part 3 is more interesting.

Severity: normal → S3
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: