Closed
Bug 1657436
Opened 4 years ago
Closed 4 years ago
Port bug 1528649: Remove the context parameter from nsInputStreamPump::AsyncRead
Categories
(Thunderbird :: Upstream Synchronization, defect)
Thunderbird
Upstream Synchronization
Tracking
(thunderbird_esr78 unaffected)
RESOLVED
FIXED
81 Branch
| Tracking | Status | |
|---|---|---|
| thunderbird_esr78 | --- | unaffected |
People
(Reporter: Paenglab, Assigned: Paenglab)
References
(Regression)
Details
(Keywords: regression)
Attachments
(1 file)
|
9.90 KB,
patch
|
rjl
:
review+
|
Details | Diff | Splinter Review |
I have a patch.
|
Assignee | |
Comment 1•4 years ago
|
||
|
||
Comment 2•4 years ago
|
||
Comment on attachment 9168233 [details] [diff] [review] 1657436-port-bug-1528649.patch Review of attachment 9168233 [details] [diff] [review]: ----------------------------------------------------------------- Works for me.
Attachment #9168233 -
Flags: review?(rob) → review+
Pushed by richard.marti@gmail.com:
https://hg.mozilla.org/comm-central/rev/25614e6ebd26
Port bug 1528649: Remove the context parameter from nsInputStreamPump::AsyncRead. DONTBUILD r=rjl
Status: ASSIGNED → RESOLVED
Closed: 4 years ago
Resolution: --- → FIXED
|
|
Assignee | |
Updated•4 years ago
|
Target Milestone: --- → 81 Branch
|
||
Comment 4•4 years ago
|
||
calendar/lightning/components/CalItipProtocolHandler.jsm
38 asyncRead(listener, ctxt) { // found in ItipChannel
That Calendar thing needs cleanup, too, or am I missing something?
Flags: needinfo?(geoff)
|
||
Comment 5•4 years ago
|
||
That seems to be dead code. The class doesn't implement the interface in question and never has, AFAICT.
Flags: needinfo?(geoff)
|
||
Updated•4 years ago
|
|
||
Updated•3 years ago
|
Keywords: regression
You need to log in
before you can comment on or make changes to this bug.
Description
•