Closed
Bug 1842005
Opened 2 years ago
Closed 2 years ago
Avoid code duplication in IsSimpleHeader and IsCORSSafelistedRequestHeader
Categories
(Core :: DOM: Networking, task, P3)
Core
DOM: Networking
Tracking
()
RESOLVED
FIXED
118 Branch
| Tracking | Status | |
|---|---|---|
| firefox118 | --- | fixed |
People
(Reporter: dlrobertson, Assigned: twisniewski)
Details
(Whiteboard: [necko-triaged])
Attachments
(1 file, 1 obsolete file)
nsContentUtils::IsCORSSafelistedRequestHeader and InternalHeaders::IsSimpleHeader are almost identical. Reduce code duplication by:
- Using
nsContentUtils::IsCORSSafelistedRequestHeaderinstead ofInternalHeaders::IsSimpleHeaderfor all InternalHeaders::IsSimpleHeader callsites - Remove InternalHeaders::IsSimpleHeader definition.
- Remove InternalHeaders::IsSimpleHeader declaration
|
Reporter | |
Updated•2 years ago
|
Priority: -- → P5
|
||
Updated•2 years ago
|
Severity: -- → N/A
Priority: P5 → P3
Whiteboard: [necko-triaged]
|
Assignee | |
Comment 1•2 years ago
|
||
|
||
Updated•2 years ago
|
Assignee: nobody → twisniewski
Status: NEW → ASSIGNED
|
|
Assignee | |
Comment 2•2 years ago
|
||
|
|
||
Updated•2 years ago
|
Attachment #9346780 -
Attachment is obsolete: true
Pushed by twisniewski@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/97451d2d17d5
Remove InternalHeaders::IsSimpleHeader in favor of using the similar nsContentUtils::IsCORSSafelistedRequestHeader; r=kershaw
|
||
Comment 4•2 years ago
|
||
| bugherder | ||
Status: ASSIGNED → RESOLVED
Closed: 2 years ago
status-firefox118:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → 118 Branch
You need to log in
before you can comment on or make changes to this bug.
Description
•