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)
MailNews Core
General
Tracking
(Not tracked)
NEW
People
(Reporter: jorgk-bmo, Unassigned)
References
Details
Attachments
(1 file)
(deleted),
patch
|
Details | Diff | Splinter Review |
+++ 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:
- Maybe fix the hard-coded include paths at the top of the file
Make sure JS Account is adequately catered for(done in bug 1555633)- Consider removing some "simple" URL implementations, see bug 1550945 comment #0, quote: "remove some of the implementations".
Comment 1•5 years ago
|
||
... assuming this was what you were looking for for part 1?
Reporter | ||
Comment 2•5 years ago
|
||
I guess so, but part 3 is more interesting.
Updated•2 years ago
|
Severity: normal → S3
You need to log in
before you can comment on or make changes to this bug.
Description
•