Opened on 11/02/2015 at 11:10:47 AM

Closed on 11/02/2015 at 01:36:08 PM

Last modified on 11/16/2015 at 11:50:38 AM

#3253 closed change (fixed)

Adblock Warning List opt in message should not be triggered by frames (Firefox)

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



Adblock Plus for Firefox has an issue similar to #3238 - anti-adblock warning can be triggered by frames. However, the test case from #3238 doesn't work for me because the logic determining document URL is flawed (subject.location.href cannot be expected to be set in content-document-global-created observer notification).

What to change

Fix document URL retrieval in ui module's content-document-global-created handler. Make sure that it only applies to top frames.

Attachments (0)

Change History (4)

comment:1 Changed on 11/02/2015 at 11:41:08 AM by trev

  • Owner set to trev

comment:2 Changed on 11/02/2015 at 11:42:13 AM by trev

  • Review URL(s) modified (diff)

comment:3 Changed on 11/02/2015 at 01:36:08 PM by trev

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

comment:4 Changed on 11/16/2015 at 11:50:38 AM by trev

  • Milestone changed from Adblock-Plus-for-Firefox-next to Adblock-Plus-2.6.12-for-Firefox

Adblock Plus 2.6.12 is being released from branch, this change is part of it.

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