Closed
Bug 1594432
Opened 5 years ago
Closed 4 years ago
Remove `aContext` from `nsIChildChannel.completeRedirectSetup`
Categories
(Core :: Networking, task, P2)
Core
Networking
Tracking
()
RESOLVED
FIXED
mozilla76
Tracking | Status | |
---|---|---|
firefox76 | --- | fixed |
People
(Reporter: mayhemer, Assigned: sonakshisaxena1)
Details
(Whiteboard: [necko-triaged])
Attachments
(1 file)
We removed that recently from AsyncOpen, here it doesn't make sense any longer too.
Updated•5 years ago
|
Priority: -- → P2
Whiteboard: [necko-triaged]
Assignee | ||
Comment 1•4 years ago
|
||
(In reply to Honza Bambas (:mayhemer) from comment #0)
We removed that recently from AsyncOpen, here it doesn't make sense any longer too.
Hey Honza
Can I work on this?
Thanks :)
Reporter | ||
Comment 2•4 years ago
|
||
Yes! Thanks!
Assignee | ||
Comment 3•4 years ago
|
||
Updated•4 years ago
|
Assignee: nobody → sonakshisaxena1
Status: NEW → ASSIGNED
Assignee | ||
Comment 4•4 years ago
|
||
(In reply to Honza Bambas (:mayhemer) from comment #2)
Yes! Thanks!
I have submitted the patch for this https://phabricator.services.mozilla.com/D68177 but it says that
This is a secure revision.
Please use the CC list of the associated bug in Bugzilla to manage access and subscribership of this revision. Changes made here may be overwritten.
Please do not land this revision using `arc land` to prevent data leakage.
I am not sure how to submit, as the revision is only saved as a draft.
Assignee | ||
Comment 5•4 years ago
|
||
Ah, it got submitted now :)
Please review.
Thanks :)
Pushed by valentin.gosu@gmail.com: https://hg.mozilla.org/integration/autoland/rev/0cd0ef64e426 Remove aContext from nsIChildChannel.completeRedirectSetup r=valentin
Comment 7•4 years ago
|
||
bugherder |
Status: ASSIGNED → RESOLVED
Closed: 4 years ago
status-firefox76:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla76
You need to log in
before you can comment on or make changes to this bug.
Description
•