Opened 5 weeks ago

Closed 5 weeks ago

Last modified 4 weeks ago

#6473 closed change (fixed)

Remove WebSocket wrapper

Reported by: mjethani Assignee: mjethani
Priority: Unknown Milestone: Adblock-Plus-for-Chrome-Opera-Firefox-next
Module: Platform Keywords:
Cc: sebastian, kzar Blocked By:
Blocking: #6459 Platform: Unknown / Cross platform
Ready: no Confidential: no
Tester: Unknown Verified working: no
Review URL(s):

https://codereview.adblockplus.org/29721716/

Description (last modified by mjethani)

Background

All supported versions of Firefox (51 onwards) support WebSocket blocking, while Chrome added support in version 58 (released April 19, 2017). As of now the WebSocket wrapper in the content script exists only for very outdated versions of Chrome.

See discussion on #6469.

What to change

Remove the WebSocket wrapper from inject.preload.js along with any related code and comments.

Change History (8)

comment:1 Changed 5 weeks ago by mjethani

  • Description modified (diff)

comment:2 Changed 5 weeks ago by mjethani

  • Review URL(s) modified (diff)

comment:3 Changed 5 weeks ago by abpbot

A commit referencing this issue has landed:
Issue 6473 - Remove WebSocket wrapper

comment:4 Changed 5 weeks ago by mjethani

  • Owner set to mjethani

comment:5 Changed 5 weeks ago by mjethani

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

comment:6 Changed 4 weeks ago by kzar

Since we have a release coming up and this was (correctly) pushed to the next branch I wonder if assigning the Adblock-Plus-for-Chrome-Opera-Firefox-next is right? Really the milestone should be Adblock-Plus-for-Chrome-Opera-Firefox-next-next, but that doesn't exist.

comment:7 Changed 4 weeks ago by sebastian

That is correct. There is an Adblock-Plus-3.0.3-for Chrome-Opera-Firefox milestone with the changes for the upcoming release. The current Adblock-Plus-for-Chrome-Opera-Firefox-next milestone will eventually turn into the 3.0.4 release.

comment:8 Changed 4 weeks ago by kzar

Cool OK, didn't realise you already moved the milestone on. That makes sense thanks.

Note: See TracTickets for help on using tickets.