Opened on 12/22/2017 at 10:47:06 PM

Closed on 01/08/2018 at 12:01:44 AM

#6217 closed change (fixed)

Correct behavior of whitelisting a domain

Reported by: dzhang Assignee: dzhang
Priority: Unknown Milestone: Adblock-Plus-for-iOS-next
Module: Adblock-Plus-for-iOS/macOS Keywords:
Cc: Blocked By:
Blocking: Platform: iOS
Ready: yes Confidential: no
Tester: Unknown Verified working: no
Review URL(s):

Description (last modified by dzhang)


It has been observed that the time it takes for whitelisting to take effect is inconsistent when performing whitelisting using the Safari action extension.

This has been traced to an implementation error where the content blocker was incorrectly being reloaded at the wrong time.

It appears that calling reload on the content blocker more than once concurrently can cause unexpected effects such as a crash during content blocker access or code execution being halted.

Removing extraneous content blocker reload calls appears to greatly improve the consistency of whitelisting by causing the whitelisted site to be recognized almost immediately after whitelisting.

A future improvement can involve a notification to let the user know that when the whitelisting has been completed.

What to change

Prevent extraneous reloads of the content blocker so that the reloads do not interefere with whitelisting operations.

Attachments (0)

Change History (4)

comment:1 Changed on 12/22/2017 at 11:16:20 PM by dzhang

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

comment:2 Changed on 12/26/2017 at 10:01:27 PM by dzhang

  • Description modified (diff)

comment:3 Changed on 12/27/2017 at 12:24:28 AM by dzhang

When testing on the simulator, there are Main Thread Checker errors reporting during the first whitelisting of a website using the Safari action extension. These are due to KVO observers, and their subsequent UI operations, being in effect in AdblockPlusControllerBase. The success of the whitelisting operation is not affected by these errors reported in Xcode 9.

Last edited on 12/27/2017 at 12:25:47 AM by dzhang

comment:4 Changed on 01/08/2018 at 12:01:44 AM by dzhang

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

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