Open Bug 1450150 Opened 3 years ago Updated 5 months ago

[meta] Convert protocol clients to fronts using protocol.js

Categories

(DevTools :: General, enhancement, P5)

enhancement

Tracking

(Fission Milestone:M7)

Fission Milestone M7

People

(Reporter: jryans, Unassigned)

References

(Depends on 2 open bugs, Blocks 2 open bugs)

Details

(Keywords: meta, Whiteboard: dt-fission)

Various protocol clients from before the time of protocol.js should be converted into fronts so that we can have a single technology and lifetime model for RDP communication.

These include:

* devtools/shared/client/*-client.js
* tools relying on free-form RDP events[1], such as the console

[1]: https://searchfox.org/mozilla-central/source/devtools/shared/client/constants.js#22
Product: Firefox → DevTools
Depends on: 1485660
Depends on: 1485661
Depends on: 1485664
On thing I learned while working on bug 1485660, TabClient:
  First look for all usages of callback style arguments used for your client's requests.

protocol.js returns promises for each request method. It doesn't support passing a callback as last argument, whereas old fashion client were often supporting both styles...
See bug 1494305 for example of such convertion from callback to promises.
Once all clients are converted we should empty that list of unsollicited messages:
  https://searchfox.org/mozilla-central/source/devtools/shared/client/constants.js#22-53
It may be hard to reduce it while working on TabClient and RootClient as some message names list workerListChanged are used by both actors...
Depends on: 1494632
Depends on: 1494796
Depends on: 1495657
Depends on: 1497467
Depends on: 1497470
Whiteboard: dt-fission
Depends on: 1571704

Tracking Fission DevTools bugs for Fission Nightly (M6)

Fission Milestone: --- → M6
Depends on: 1618015

dt-fission-reserve bugs do not need to block Fission Nightly (M6), but these bugs' summaries mention the word "Fission", so let's track them for Fission riding the trains to Beta (M7). We'll revisit these bugs before we ship Fission.

Fission Milestone: M6 → M7
You need to log in before you can comment on or make changes to this bug.