1585902 - Type `paused` event via protocol.js specification and use emit() instead of sendActorEvent

Status: RESOLVED FIXED

(DevTools :: Debugger, task, P3)

Description

[Alexandre Poirot [:ochameau]]

The thread actor is still using old API, pre-protcotol.js to emit a few of its events: sendActorEvent:
https://searchfox.org/mozilla-central/search?q=sendActorEvent(&case=true&regexp=false&path=thread.js

Doing this:

• prevents easily passing actor instances in the payload of these events
• make the thread actor special compare to all other one

Instead it should be using emit and type each attribute of the event in the thread specification:
https://searchfox.org/mozilla-central/source/devtools/shared/specs/thread.js#27-35
It looks like the specification has already been covered when implementing ThreadFront,
so it should be mostly about replacing sendActorEvent by emit.

Comment 1

[Logan Smyth [:loganfsmyth]]

Attachment: Bug 1585902 - Use the protocoljs framework for emitting events. r=ochameau

Comment 2

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/c7777eb19c6e
Use the protocoljs framework for emitting events. r=ochameau

Comment 3

[Stefan Hindli [:stefan_hindli]]

https://hg.mozilla.org/mozilla-central/rev/c7777eb19c6e