Emit callwatcher call events

RESOLVED FIXED

Status

()

Firefox
Developer Tools: Performance Tools (Profiler/Timeline)
RESOLVED FIXED
3 years ago
2 years ago

People

(Reporter: vporof, Assigned: vporof)

Tracking

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

Comment hidden (empty)
(Assignee)

Comment 1

3 years ago
Created attachment 8689627 [details] [diff] [review]
v1

Don't think we need a test yet.
Assignee: nobody → vporof
Status: NEW → ASSIGNED
Attachment #8689627 - Flags: review?(jsantell)
What's this for? Come to think of it, why is CallWatcher an actor anyway? We don't communicate with it via RDP, and it's always fully owned by another actor.
Attachment #8689627 - Flags: review?(jsantell)
(Assignee)

Comment 3

2 years ago
See https://github.com/victorporof/Restartless-Template/commit/c8b71e634084d8b1caa60298d3a93d72d52a7721. There have been requests to use the call-watcher by add-ons on the client-side. Currently, there's no way for this actor to be used standalone, because it doesn't surface events. This patch addresses this.
(Assignee)

Comment 4

2 years ago
Comment on attachment 8689627 [details] [diff] [review]
v1

Just want to get this landed so I get this out of my queue. See previous comment.
Attachment #8689627 - Flags: review?(jsantell)
Attachment #8689627 - Flags: review?(jsantell) → review+
(Assignee)

Updated

2 years ago
Status: ASSIGNED → RESOLVED
Last Resolved: 2 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.