Opened on 01/08/2015 at 03:11:16 PM

Closed on 01/08/2015 at 03:44:53 PM

Last modified on 05/20/2015 at 02:22:39 PM

#1765 closed change (fixed)

Align Firefox messaging implementation with Safari

Reported by: trev Assignee: trev
Priority: P3 Milestone: Adblock-Plus-2.6.7-for-Firefox
Module: Adblock-Plus-for-Firefox Keywords:
Cc: Blocked By:
Blocking: Platform: Firefox
Ready: yes Confidential: no
Tester: Verified working:
Review URL(s):

Description (last modified by trev)


Abstraction layer for messaging was introduced to Firefox in #1706. The implementation was very similar to Safari's, yet some of the changes introduced in #1724 made the two implementation diverge (with good reason).

What to change

Implement the relevant changes in Firefox as well:

  • If there is no response, don't set the payload property of the message instead of sending responseSent: false.
  • Use a local sent variable instead of the this._responseSent property to save whether the message was handled.
  • Return a list with results from all handlers in EventTarget._dispatch instead of making assumptions about the nature of the result.

Hints for testers

This change again affects all functionality of the first-run page.

Attachments (0)

Change History (4)

comment:1 Changed on 01/08/2015 at 03:14:50 PM by trev

  • Review URL(s) modified (diff)
  • Status changed from new to reviewing

comment:2 Changed on 01/08/2015 at 03:44:53 PM by trev

  • Resolution set to fixed
  • Status changed from reviewing to closed

comment:3 Changed on 01/08/2015 at 04:30:47 PM by trev

  • Description modified (diff)
  • Milestone set to Adblock-Plus-for-Firefox-next

comment:4 Changed on 05/20/2015 at 02:22:39 PM by philll

  • Platform changed from Firefox/Firefox Mobile to Firefox

Made Firefox and Firefox mobile available as seperate platforms.

Add Comment

Modify Ticket

Change Properties
as closed .
The resolution will be deleted. Next status will be 'reopened'.
to The owner will be changed from trev.
Note: See TracTickets for help on using tickets.