Opened on 03/30/2015 at 01:42:54 PM

Closed on 04/01/2015 at 08:41:21 AM

Last modified on 05/20/2015 at 02:22:39 PM

#2232 closed change (fixed)

Get rid of conditional catch clauses

Reported by: sebastian Assignee: sebastian
Priority: P3 Milestone: Adblock-Plus-2.6.10-for-Firefox
Module: Adblock-Plus-for-Firefox Keywords:
Cc: trev Blocked By:
Blocking: #312, #2231 Platform: Firefox
Ready: yes Confidential: no
Tester: Verified working: no
Review URL(s):

Description (last modified by trev)


We currently use a conditional catch clause in lib/contentPolicy.js. However, conditional catch clauses are non-standard and JsDoc 3.3.0 is unable to parse them.

What to change

Catch the error unconditionally. For the factory registration failure proceed as before, all other errors should simply be reported.

Attachments (0)

Change History (6)

comment:1 Changed on 03/30/2015 at 01:43:17 PM by sebastian

  • Blocking 312 added; 321 removed

comment:2 Changed on 03/30/2015 at 01:51:14 PM by trev

  • Description modified (diff)
  • Platform changed from Unknown to Firefox/Firefox Mobile
  • Priority changed from Unknown to P3
  • Ready set

comment:3 Changed on 03/30/2015 at 04:57:16 PM by sebastian

  • Owner set to sebastian

comment:4 Changed on 03/30/2015 at 04:57:26 PM by sebastian

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

comment:5 Changed on 04/01/2015 at 08:41:21 AM by sebastian

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

comment:6 Changed on 05/20/2015 at 02:22:39 PM by philll

  • Platform changed from Firefox/Firefox Mobile to Firefox

Made Firefox and Firefox mobile available as seperate platforms.

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