Opened on 11/02/2015 at 08:14:07 PM

Closed on 01/06/2016 at 01:30:58 PM

Last modified on 01/18/2016 at 01:57:01 AM

#3259 closed change (fixed)

Store blockable item selection in the content process

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



With #3222 fixed the blockable items will still call RequestNotifier.storeSelection() and RequestNotifier.getSelection() on the DOM window - meaning that it still works via cross-process wrappers and might potentially lose information (WeakMap doesn't see to support these wrappers properly).

What to change

Make RequestNotifier.storeSelection() and RequestNotifier.getSelection() take outerWindowID as parameter and store the data in the child. Blockable items implementation will have to adjust slightly for RequestNotifier.getSelection() being async.

Attachments (0)

Change History (6)

comment:1 Changed on 12/02/2015 at 11:28:51 AM by trev

  • Blocking 3222 removed

comment:2 Changed on 01/04/2016 at 08:26:02 PM by trev

  • Owner set to trev

comment:3 Changed on 01/04/2016 at 08:26:23 PM by trev

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

comment:4 Changed on 01/06/2016 at 01:30:58 PM by trev

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

comment:5 Changed on 01/07/2016 at 07:59:13 PM by trev

  • Blocked By 3491 added

comment:6 Changed on 01/18/2016 at 01:57:01 AM by Ross

  • Tester changed from Unknown to Ross
  • Verified working set

Blockable items list appears to be working correctly.

Firefox 38 / 43 / 44 / 46.0a1 / Ubuntu 14.04 x64

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.