Closed Bug 782235 Opened 7 years ago Closed 7 years ago

move HTTP convenience functions to nsHttp

Categories

(Core :: Networking, defect, trivial)

defect
Not set
trivial

Tracking

()

RESOLVED FIXED
mozilla18

People

(Reporter: julian.reschke, Assigned: julian.reschke)

References

Details

Attachments

(1 file, 2 obsolete files)

QA Contact: julian.reschke
Depends on: 758973
Attached patch Proposed patch (obsolete) — Splinter Review
Attachment #653189 - Flags: review?(bsmith)
Comment on attachment 653189 [details] [diff] [review]
Proposed patch

Review of attachment 653189 [details] [diff] [review]:
-----------------------------------------------------------------

Thanks!
Attachment #653189 - Flags: review?(bsmith) → review+
Keywords: checkin-needed
This doesn't apply cleanly. Please rebase. Also, please update your commit information to include your name along with your email address. Thanks!
Attached patch Proposed patch (obsolete) — Splinter Review
Patch updated to apply to trunk, try results at https://tbpl.mozilla.org/?tree=Try&rev=0974a5ea2077
Attachment #653189 - Attachment is obsolete: true
Attachment #664013 - Flags: review?
Comment on attachment 664013 [details] [diff] [review]
Proposed patch

wrong patch - please ignore
Attachment #664013 - Flags: review?
Attached patch Proposed patchSplinter Review
Patch rebased; try results at https://tbpl.mozilla.org/?tree=Try&rev=0974a5ea2077
Attachment #664013 - Attachment is obsolete: true
Attachment #664015 - Flags: review?(bsmith)
Comment on attachment 664015 [details] [diff] [review]
Proposed patch

Patch is just rebased version of https://bugzilla.mozilla.org/attachment.cgi?id=653189, r+'d by bsmith
Attachment #664015 - Flags: review?(bsmith) → review+
Keywords: checkin-needed
(In reply to Julian Reschke from comment #6)
> https://tbpl.mozilla.org/?tree=Try&rev=0974a5ea2077

Green on Try. Thanks for rebasing!

https://hg.mozilla.org/integration/mozilla-inbound/rev/1ebd4fc2448c
Flags: in-testsuite-
Keywords: checkin-needed
QA Contact: julian.reschke
https://hg.mozilla.org/mozilla-central/rev/1ebd4fc2448c
Status: NEW → RESOLVED
Closed: 7 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla18
You need to log in before you can comment on or make changes to this bug.