Opened on 03/17/2016 at 02:01:10 PM

Closed on 03/23/2016 at 01:36:58 PM

Last modified on 05/25/2016 at 02:02:29 PM

#3823 closed change (fixed)

Split up message responder code

Reported by: sebastian Assignee: sebastian
Priority: P3 Milestone: Adblock-Plus-1.12-for-Chrome-Opera-Safari
Module: Platform Keywords:
Cc: Blocked By:
Blocking: Platform: Unknown / Cross platform
Ready: yes Confidential: no
Tester: Ross Verified working: yes
Review URL(s):



In background.js we currently have one huge ext.onMessage listener that handles messages for each part of the extension. However, some of that code would rather belong into their respective modules.

Moreover, ext.onMessage is just a thin wrapper mimicking chrome.runtime.onMessage which has some issues that got addressed with the new messaging API we going to use on Firefox, see #3499.

What to change

  • Implement the subset of #3499 supporting Port.on and
  • Migrate the messaging code in background.js to the new API.
  • Move message responders out to their respective module where applicable.

Attachments (0)

Change History (5)

comment:1 Changed on 03/17/2016 at 02:04:23 PM by sebastian

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

comment:2 Changed on 03/19/2016 at 07:42:21 PM by sebastian

  • Review URL(s) modified (diff)

comment:3 Changed on 03/23/2016 at 01:33:18 PM by abpbot

comment:4 Changed on 03/23/2016 at 01:36:58 PM by sebastian

  • Milestone set to Adblock-Plus-for-Chrome-Opera-Safari-next
  • Resolution set to fixed
  • Status changed from reviewing to closed

comment:5 Changed on 05/25/2016 at 02:02:29 PM by Ross

  • Tester changed from Unknown to Ross
  • Verified working set

Change doesn't look to have caused any obvious regressions. Context menu, subscriptions, state changes, icon, devtools panel are working as expected still.

Chrome 31, 45, 50 / Windows 7
Chrome 32, 49 / OS X 10.11
Chrome 49 / Ubuntu 14.04

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 sebastian.
Note: See TracTickets for help on using tickets.