Closed
Bug 1528934
Opened 5 years ago
Closed 5 years ago
Remove unused nsSecCheckWrapChannel
Categories
(Core :: DOM: Security, enhancement, P2)
Core
DOM: Security
Tracking
()
RESOLVED
FIXED
mozilla67
| Tracking | Status | |
|---|---|---|
| firefox67 | --- | fixed |
People
(Reporter: ckerschb, Assigned: ckerschb)
References
Details
(Whiteboard: [domsecurity-active])
Attachments
(1 file)
Since we do not support addons anymore we can safely remove nsSecCheckWrapChannel which was only used for wrapping channels in case such channels did not attach a loadinfo.
We should not have that in our own codebase - hence we should remove it.
|
Assignee | |
Updated•5 years ago
|
Assignee: nobody → ckerschb
Status: NEW → ASSIGNED
Priority: -- → P2
Whiteboard: [domsecurity-active]
|
|
Assignee | |
Comment 1•5 years ago
|
||
|
|
Assignee | |
Comment 2•5 years ago
|
||
It seems we added an explicit testcase for that fallback mechanism within Bug 1351443.
I'll remove
netwerk/test/unit/test_1351443-missing-NewChannel2.js
within this changeset as well.
Pushed by mozilla@christophkerschbaumer.com: https://hg.mozilla.org/integration/mozilla-inbound/rev/69e0ccbd95d1 Remove unused nsSecCheckWrapChannel. r=valentin
|
||
Comment 4•5 years ago
|
||
| bugherder | ||
Status: ASSIGNED → RESOLVED
Closed: 5 years ago
status-firefox67:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla67
You need to log in
before you can comment on or make changes to this bug.
Description
•