Make the listener an object

NEW
Unassigned

Status

P3
normal
4 years ago
2 months ago

People

(Reporter: ato, Unassigned)

Tracking

({pi-marionette-server})

Trunk
pi-marionette-server
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

(Reporter)

Description

4 years ago
This is a follow-up on bug 1153832 that introduced the new dispatching technique to the listener.  That patch was quite ugly because it had to explicitly assign listeners to functions so that it could add and remove them to the message manager.  We can avoid this if we make the listener an object that implements nsIMessageListener.

Turning it into an object would also make it more (unit) testable as it would enable us to make assumptions about the listener's current state.
(Reporter)

Updated

4 years ago
Assignee: nobody → ato
Status: NEW → ASSIGNED
Keywords: ateam-marionette-server
(Reporter)

Updated

10 months ago
Depends on: 1426154
(Reporter)

Updated

9 months ago
OS: Unspecified → All
Priority: -- → P3
Hardware: Unspecified → All
Version: unspecified → Trunk
(Reporter)

Comment 2

2 months ago
Not currently working on this, but the recent decision to remove
frame scripts from Gecko in favour of an ActorChild model may impact
the decision whether to do fix this.
Assignee: ato → nobody
Status: ASSIGNED → NEW
You need to log in before you can comment on or make changes to this bug.