Andreas Tolfsen ❲:ato❳ Reporter
	Description • 9 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.

	Andreas Tolfsen ❲:ato❳ Reporter
	Comment 1 • 9 years ago

try: https://treeherder.mozilla.org/#/jobs?repo=try&revision=146a07b0eb27

	Andreas Tolfsen ❲:ato❳ Reporter
	Comment 2 • 6 years 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.