Opened on 11/09/2016 at 08:58:54 AM

Closed on 11/09/2016 at 05:04:19 PM

Last modified on 11/18/2016 at 11:46:11 AM

#4624 closed defect (fixed)

Element hiding rules apply on whitelisted schemes (chrome://, file:/// etc.)

Reported by: trev Assignee: trev
Priority: P1 Milestone: Adblock-Plus-2.8.2-for-Firefox
Module: Adblock-Plus-for-Firefox Keywords:
Cc: Blocked By:
Blocking: Platform: Unknown / Cross platform
Ready: yes Confidential: no
Tester: Ross Verified working: yes
Review URL(s):

https://codereview.adblockplus.org/29362064/

Description

Environment

Adblock Plus 2.8.1.4224-beta

How to reproduce

  1. Paste the following text into a test.html file on your disk:
    <html>
      <body>
        <div id="div">
          This should not be hidden.
        </div>
      </body>
    </html>
    
  2. Add the filter ###div.
  3. Open that HTML page in your browser (location should be something like file:///foo/bar/test.html).

Observed behaviour

The text on the page is hidden.

Expected behaviour

The text should be visible - element hiding rules aren't supposed to apply to file:///.

Attachments (0)

Change History (4)

comment:1 Changed on 11/09/2016 at 09:01:46 AM by trev

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

comment:2 Changed on 11/09/2016 at 05:00:38 PM by abpbot

comment:3 Changed on 11/09/2016 at 05:04:19 PM by trev

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

comment:4 Changed on 11/18/2016 at 11:46:11 AM by Ross

  • Tester changed from Unknown to Ross
  • Verified working set

Fixed.

Firefox 38 / 49 / Windows 7
Firefox Nightly 53.0a1 (2016-11-17) / Windows 7

Add Comment

Modify Ticket

Change Properties
Action
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.