Opened 4 years ago

Last modified 4 years ago

#4783 closed change

Switch message_responder.js to use the messaging API — at Version 2

Reported by: kzar Assignee: kzar
Priority: Unknown Milestone:
Module: User-Interface Keywords:
Cc: sebastian, greiner Blocked By:
Blocking: #4579 Platform: Unknown / Cross platform
Ready: no Confidential: no
Tester: Unknown Verified working: no
Review URL(s):

Description (last modified by kzar)


messageResponder.js contains a rather monolithic message listener, still using the ext API which is planned to go away.

What to change

Break the message listener down into separate listeners which use the Port API. (See adblockpluschrome/lib/messaging.js.)

While at it we might as well update the file to use more modern JavaScript syntax like let instead of var, arrow functions and for of.

Change History (2)

comment:1 Changed 4 years ago by kzar

  • Description modified (diff)

comment:2 Changed 4 years ago by kzar

  • Review URL(s) modified (diff)
  • Status changed from new to reviewing
Note: See TracTickets for help on using tickets.