Opened on 03/26/2014 at 11:55:57 AM

Closed on 03/27/2014 at 08:41:47 AM

Last modified on 04/13/2014 at 06:27:04 PM

#216 closed defect (fixed)

Firefox - First run page opens on every update

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

http://codereview.adblockplus.org/5689385303408640

Description (last modified by trev)

Environment

Adblock Plus 2.5.1.3786 in Firefox 31.0a1 nightly on OS X.

How to reproduce

  1. Install Adblock Plus 2.5.1.3785 from https://downloads.adblockplus.org/devbuilds/adblockplus/.
  2. Force extension update to have it update to Adblock Plus 2.5.1.3786.

Observed behaviour

First run page opens on update, filter subscriptions don't change.

Expected behaviour

No first run on update unless there is really no filter subscription installed.

Attachments (0)

Change History (6)

comment:1 Changed on 03/26/2014 at 11:56:22 AM by trev

  • Description modified (diff)

comment:2 Changed on 03/26/2014 at 12:06:19 PM by trev

Apparently, the issue is that FilterStorage._loading is being set too late. Timing changes caused by #153 are triggering that issue now, before that IO.statFile() was synchronous and this issue didn't matter.

comment:3 Changed on 03/26/2014 at 12:36:33 PM by trev

  • Blocking 153 added
  • in_progress changed from 0 to 1
  • Owner set to trev
  • Review URL(s) modified (diff)
  • Status changed from new to assigned

comment:4 Changed on 03/26/2014 at 12:36:39 PM by trev

  • Status changed from assigned to reviewing

comment:5 Changed on 03/27/2014 at 08:41:47 AM by trev

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

comment:6 Changed on 04/13/2014 at 06:27:04 PM by trev

  • Milestone set to Adblock-Plus-for-Firefox-next

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.