Closed Bug 1460313 Opened 2 years ago Closed 2 years ago

Is nsHttpTransaction::mForTakeResponseHead used?

Categories

(Core :: Networking: HTTP, enhancement, P3)

enhancement

Tracking

()

RESOLVED FIXED
mozilla62
Tracking Status
firefox62 --- fixed

People

(Reporter: u408661, Assigned: u408661)

Details

(Whiteboard: [necko-triaged])

Attachments

(1 file)

When looking at the code to fix bug 1436610 it looked to me like mForTakeRequestHead is no longer used (never set to anything but nullptr). In the interest of code hygiene (and future developer sanity) we should get rid of it, if possible.
That should say mForTakeResponseHead, not request.
Summary: Is nsHttpTransaction::mForTakeRequestHead used? → Is nsHttpTransaction::mForTakeResponseHead used?
Comment on attachment 8974928 [details]
Bug 1460313 - Remove unused mForTakeResponseHead.

https://reviewboard.mozilla.org/r/243314/#review249168
Attachment #8974928 - Flags: review?(mcmanus) → review+
Pushed by hurley@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/9c6a3bba2137
Remove unused mForTakeResponseHead. r=mcmanus
https://hg.mozilla.org/mozilla-central/rev/9c6a3bba2137
Status: NEW → RESOLVED
Closed: 2 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla62
You need to log in before you can comment on or make changes to this bug.