Closed Bug 1460313 Opened 2 years ago Closed 2 years ago
Http Transaction::m For Take Response Head used?
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 firstname.lastname@example.org: https://hg.mozilla.org/integration/autoland/rev/9c6a3bba2137 Remove unused mForTakeResponseHead. r=mcmanus
You need to log in before you can comment on or make changes to this bug.