Implement correctly error handling and cancellation in WyciwygChannelChild and WyciwygChannelParent

RESOLVED FIXED

Status

()

Core
Networking: Cache
RESOLVED FIXED
8 years ago
8 years ago

People

(Reporter: michal, Assigned: jdm)

Tracking

(Blocks: 1 bug)

Trunk
Points:
---
Dependency tree / graph

Firefox Tracking Flags

(fennec2.0b3+)

Details

Attachments

(1 attachment)

(Reporter)

Description

8 years ago
This is a follow-up bug of #561085. TODOs that need to be resolved yet are in:

WyciwygChannelChild::RecvOnStartRequest
WyciwygChannelChild::RecvOnDataAvailable
WyciwygChannelParent::RecvInit
WyciwygChannelParent::RecvAsyncOpen
Assignee: nobody → josh
Created attachment 486068 [details] [diff] [review]
Add proper cancellation to e10s wyciwyg channels.
We do want this in fennec for completeness and correctness.
tracking-fennec: --- → ?
Attachment #486068 - Flags: review?(dwitte)

Comment 3

8 years ago
Comment on attachment 486068 [details] [diff] [review]
Add proper cancellation to e10s wyciwyg channels.

r=dwitte. You did compile this, right? ;)

Also, do we have tests for this? I'm guessing not. Shouldn't be hard to add a few.
Attachment #486068 - Flags: review?(dwitte) → review+

Comment 4

8 years ago
jdm, assuming this is close to being done, we are marking blocking on b3.
tracking-fennec: ? → 2.0b3+
Blocks: 613578
http://hg.mozilla.org/mozilla-central/rev/553678fc7c03
Status: NEW → RESOLVED
Last Resolved: 8 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.