Open Bug 1499988 Opened 6 years ago Updated 2 years ago

Fix Protocol.js documentation

Categories

(DevTools :: General, enhancement, P2)

enhancement

Tracking

(Not tracked)

People

(Reporter: yulia, Unassigned)

References

(Blocks 1 open bug)

Details

Our protocol.js documentation is out of date. It recommends the following: https://searchfox.org/mozilla-central/source/devtools/docs/backend/protocol.js.md#75-76 ``` const HelloFront = protocol.FrontClassWithSpec(helloWorldSpec, { initialize: function (client, form) { protocol.Front.prototype.initialize.call(this, client, form); // This call may not be required but it's a good idea. It will // guarantee that your instance is managed in the pool. this.manage(this); } }); ``` This results in the entire form object being passed into the front method, and if it is the form that we get with the target getFront method, then the `actor` field will always be `frameTarget`. We need to update this documentation to either not pass in the form at all and set the actor ID manually (since the form parameter should go away soon: https://searchfox.org/mozilla-central/source/devtools/shared/protocol.js#1289-1292 or, we should change the documentation to follow this pattern: https://searchfox.org/mozilla-central/source/devtools/shared/fronts/reflow.js#20
Severity: normal → enhancement
Priority: -- → P2
Severity: normal → S3
You need to log in before you can comment on or make changes to this bug.