Opened on 01/16/2017 at 10:34:10 AM

Closed on 02/28/2017 at 12:53:32 PM

Last modified on 03/13/2017 at 12:12:37 PM

#4804 closed defect (fixed)

Icon animations do not play properly for newly created tab

Reported by: kzar Assignee: kzar
Priority: P2 Milestone: Adblock-Plus-1.13-for-Chrome-Opera
Module: Platform Keywords:
Cc: sebastian Blocked By:
Blocking: Platform: Chrome
Ready: yes Confidential: no
Tester: Ross Verified working: yes
Review URL(s):



Chrome 55
Adblock Plus 1.12.4

How to reproduce

  1. Install / reload Adblock Plus
  2. After Adblock Plus has loaded open a new tab
  3. Type this in the extension's background console: require("icon").startIconAnimation("critical");.

Observed behaviour

The icon for the newly created tab does not change. If you switch to a different tab, then switch back to the newly created tab during the animation the icon gets stuck for the new tab.

Expected behaviour

The animation plays properly, even for the newly created tab.


This is a regression caused by Issue 4386 - Fixed determining document domain, particularly after being redirected. (See also #4598, #4599 and #4647.)

Attachments (0)

Change History (6)

comment:1 Changed on 01/16/2017 at 12:45:20 PM by kzar

It seems that this is caused by the fact that chrome.tabs.onUpdated is dispatched with the "loading" event before chrome.webNavigation.onCommitted.

Our chrome.tabs.onUpdated listener dispatches the ext.onLoading event, our listeners for that event add the new Page to their PageMaps. Shortly afterwards our chrome.webNavigation.onCommitted listener calls ext._removeFromAllPageMaps and the records of the page are trashed!

comment:2 Changed on 01/18/2017 at 08:12:42 AM by kzar

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

comment:3 Changed on 01/18/2017 at 08:13:48 AM by kzar

  • Priority changed from P3 to P2

(Increasing to P2 since this bug causes our PageMaps to be wrong half the time, which could well be causing other issues. Perhaps we should even mark this P1?)

comment:4 Changed on 02/24/2017 at 08:07:46 AM by kzar

  • Review URL(s) modified (diff)

comment:5 Changed on 02/28/2017 at 12:53:32 PM by kzar

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

A commit referencing this issue has landed:
Issue 4598,4599,4647,4804 - Work around onCommitted flaw

comment:6 Changed on 03/13/2017 at 12:12:37 PM by Ross

  • Tester changed from Unknown to Ross
  • Verified working set

Looks fixed. Icon animations continue to play correctly when started on different tabs and after opening/closing tabs as described.

Firefox 52 / Windows 10

Chrome 49 / 56 / Windows 10
Chrome 56 / OS X 10.12
Chrome 56 / Ubuntu 16.04
Opera 37 / 41 / Windows 7
Safari 10 / OS X 10.12

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