J. Ryan Stinnett [:jryans] (Use needinfo, replies may be slow) Reporter
	Description • 6 years ago

In bug 1256397, Eddy landed a few parts of converting ThreadActor to protocol.js, but there's still more work that remains:

* `attach` and potentially other request handlers may need special treatment in protocol.js due to the nested event loops used in the debugger
  * https://bugzilla.mozilla.org/attachment.cgi?id=8730334&action=diff from bug 1256397 is an attempt at this that was never finished
* The spec[1] should actually define the methods
  * File should also be renamed to `thread` to match the actor
* The old-style request handlers[2] should be removed and actor methods renamed, just like other conversions of actors to p.js

Additional conversion tips copied from :ochameau:

* `actorPrefix` becomes `typeName` on the spec object,
* `grip()` function becomes `form()` on the actor,
* We can remove `from` attributes in all in returned responses and only keep actual data you want to send to the client,
* remember calling `protocol.Actor.prototype.initialize.call(this, conn);` from your actor's `initialize` method to get events to work as expected and `protocol.Actor.prototype.destroy.call(this);` if you overload `destroy` method, in order to cleans things up correctly.

[1]: https://searchfox.org/mozilla-central/rev/57bbc1ac58816dc054df242948f3ecf75e12df5f/devtools/shared/specs/script.js#11
[2]: https://searchfox.org/mozilla-central/rev/57bbc1ac58816dc054df242948f3ecf75e12df5f/devtools/server/actors/thread.js#1730

	Yulia Startsev [:yulia] | Back in Oct. Assignee
	Comment 1 • 6 years ago

As mentioned in 1473511 -- we should also rename the destroy and exit methods, as they are a bit misleading in what they are doing. Destroy is doing a cleanup of the actor, and is called on detach and onReleaseMany, as well as from exit. The real "destroy" happens in `exit`

	Pulsebot
	Comment 4 • 5 years ago

Pushed by ystartsev@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/dd5b92f89835
Complete method set in the Spec for Thread; r=jdescottes,ochameau
https://hg.mozilla.org/integration/autoland/rev/7e6b874b870a
rename script spec to thread.js; r=jdescottes