Opened on 07/23/2014 at 11:10:00 AM

Closed on 08/20/2015 at 02:12:09 PM

Last modified on 09/02/2015 at 08:18:31 AM

#1109 closed change (fixed)

Support notifications

Reported by: fhd Assignee: sergz
Priority: P3 Milestone:
Module: Adblock-Plus-for-Internet-Explorer Keywords:
Cc: oleksandr Blocked By: #1557, #2810
Blocking: Platform: Internet Explorer
Ready: yes Confidential: no
Tester: Unknown Verified working: no
Review URL(s):

http://codereview.adblockplus.org/6505394822184960/

Description (last modified by sergz)

Background

#1107 will add support for notifications to libadblockplus.

What to change

Use the new API to show notifications. We have a range of options for how this can look in IE, something similar to how it looks in Firefox would probably be best.

Note for testers

In addition to the notification functionality please pay attention to the uninstalling (it also happens during the update procedure) of ABP for IE. The change in this issue can affect the termination of the engine (background ABP process, AdblockPlusEngine.exe).

To test the notification functionality one can use Fiddler and modify OnBeforeResponse handler by adding

if (oSession.uriContains("notification.json")) {
  var respBody = '{"notifications": [{"id": "myId", "title": "Update Adblock Plus", "message": "A new version of Adblock Plus is ready to be installed. Do you want to <a>install it now</a>?", "links": ["http://adblockplus.org"]}],"version": "201502031430" }';
   oSession.utilSetResponseBody(respBody);
 }

Take into account that the first download notification request is executed in about one minute if there is nothing about notifications in prefs.json.

While testing the notification please pay attention that

  • there is no any prompting dialog to launch AdblockPlusEngine.exe (Now it always runs at Meidum IL, one can ensure about that using some advanced task manager like ProcessExplorer from Windows Sysinternals)
  • the link in the notification is opened on mouse-click
  • the notification window is correctly displayed on different DPI settings
  • it works on the whole range of platforms Windows[XP-10] x IE[8-11].

Attachments (0)

Change History (7)

comment:1 Changed on 02/17/2015 at 11:56:40 PM by oleksandr

  • Blocked By 1557 added

Notification requests are now made by ABP for IE since #1557 is fixed now. We now need to implement some UI to display notifications to the user.

comment:2 Changed on 04/20/2015 at 09:08:14 AM by sergz

  • Owner set to sergz

comment:3 Changed on 05/11/2015 at 10:07:18 AM by sergz

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

comment:4 Changed on 07/22/2015 at 01:23:52 PM by sergz

  • Blocked By 2810 added

comment:5 Changed on 08/20/2015 at 02:11:45 PM by sergz

  • Blocked By 1107 removed
  • Tester set to Unknown

Remove #1107 because it's already not blocking and having it opened one cannot close this issue.

comment:6 Changed on 08/20/2015 at 02:12:09 PM by sergz

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

comment:7 Changed on 09/02/2015 at 08:18:31 AM by sergz

  • Description modified (diff)

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