Opened on 09/28/2018 at 11:08:40 AM

Closed on 09/28/2018 at 12:55:16 PM

Last modified on 10/08/2019 at 06:05:49 PM

#6998 closed change (fixed)

Remove usage of Shadow DOM v0 from the content script

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

Description (last modified by mjethani)


Shadow DOM v0 is deprecated and will be removed from Chrome in M73 (April 2019).

Since recent versions of both Chrome (see ticket:242#comment:58) and Firefox (see #5090) support user style sheets, and style sheets can be injected directly into the head element on older versions of Chrome and Firefox, as well as all versions of Edge (which does not support Shadow DOM v0 anyway), it is no longer necessary to use this feature.

What to change

Remove all code related to Shadow DOM v0 from the content script.

Hints for testers

Test that element hiding works on supported versions of Chrome both before and after (including) version 66 and those of Firefox both before and after (including) version 53. In other words, test all supported versions, but at least Chrome 65, Chrome 66, Firefox 52, and Firefox 53.

Given the following document:

<!DOCTYPE html>
    <div class="ad">Ad</div>

The filter should hide the element that says "Ad".

On Chrome 66+ and Firefox 53+, the filter should also work for the following document (but is not expected to work on older versions of these browsers):

<!DOCTYPE html>
    <div class="ad" style="display: block !important">Ad</div>

Attachments (0)

Change History (11)

comment:1 Changed on 09/28/2018 at 11:15:23 AM by mjethani

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

comment:2 Changed on 09/28/2018 at 12:53:57 PM by abpbot

A commit referencing this issue has landed:
Issue 6998 - Stop using Shadow DOM v0

comment:3 follow-up: Changed on 09/28/2018 at 12:55:16 PM by mjethani

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

comment:4 Changed on 09/28/2018 at 12:55:46 PM by mjethani

  • Milestone set to Adblock-Plus-for-Chrome-Opera-Firefox-next

comment:5 Changed on 09/28/2018 at 03:24:08 PM by sebastian

  • Priority changed from Unknown to P3
  • Ready set

comment:6 in reply to: ↑ 3 Changed on 02/21/2019 at 11:17:06 AM by ukacar

Replying to mjethani:is there anything for us to test regarding this change? :)

comment:7 Changed on 02/21/2019 at 01:04:19 PM by mjethani

  • Description modified (diff)

comment:8 Changed on 02/21/2019 at 01:04:59 PM by mjethani

  • Cc ukacar added

comment:9 Changed on 02/21/2019 at 01:05:25 PM by mjethani

@ukacar sorry about this, I have updated the ticket with some more information.

Last edited on 02/21/2019 at 01:05:33 PM by mjethani

comment:10 Changed on 03/07/2019 at 09:03:53 AM by ukacar

  • Verified working set

comment:11 Changed on 07/27/2019 at 10:00:00 AM by jha2125


Last edited on 10/08/2019 at 06:05:49 PM by kzar

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