Opened on 04/20/2016 at 10:26:25 AM

Closed on 04/20/2016 at 02:38:44 PM

Last modified on 04/21/2016 at 04:46:17 AM

#3960 closed defect (fixed)

Console errors from contentPolicy and cssProperties

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

https://codereview.adblockplus.org/29340656/

Description

Environment

ABP 2.7.2.4163

Reproduced with:
Firefox 43 / 45 / Windows 8

How to reproduce

  1. Launch Firefox open the Firefox console (CTRL+SHIFT+J).
  2. Install the latest ABP dev build.
  3. Perform a Google search (any query) from the search field in Firefox (top right).
  4. Observe errors in the console.

Observed behaviour

The following errors appear in the console:

whitelistMatch is not defined contentPolicy.js 277:0
whitelistMatch is not defined contentPolicy.js 277:0
TypeError: this.patterns is undefined cssProperties.js 140:1

Expected behaviour

No errors to occur.

Attachments (0)

Change History (5)

comment:1 Changed on 04/20/2016 at 02:04:16 PM by trev

  • Blocking 2401 added
  • Owner set to trev
  • Priority changed from Unknown to P1
  • Ready set

comment:2 Changed on 04/20/2016 at 02:05:37 PM by trev

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

comment:3 Changed on 04/20/2016 at 02:38:20 PM by abpbot

A commit referencing this issue has landed:
https://hg.adblockplus.org/adblockplus/rev/b755cde55566

comment:4 Changed on 04/20/2016 at 02:38:44 PM by trev

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

comment:5 Changed on 04/21/2016 at 04:46:17 AM by Ross

  • Tester changed from Unknown to Ross
  • Verified working set

ABP 2.7.2.4165
Firefox 43 / 45 / Windows 8

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.