Newly added special subscriptions shown in list

[greiner]

Environment

Nexus One (emulator with API level 25)
Firefox Nightly 57.0a1
Adblock Plus 2.99.0.1826 (gecko-webext)

How to reproduce

1. Install extension
2. Open mobile options page
3. Dispatch "showPageOptions" message in the context of the options page to show page-specific options

   ext.onMessage._dispatch({
     type: "app.respond",
     action: "showPageOptions",
     args: [{host: "foo.com", whitelisted: false}]
   })
   

4. Click on enable toggle to whitelist the current domain

Observed behaviour

Special subscription (e.g. ~user~123456) shows up in subscriptions list

Expected behaviour

No special subscriptions shown in subscriptions list

Further information

No "filter.added" event is dispatched by FilterListener but instead "subscription.added". The options page doesn't expect to get any special subscriptions though which is why it displays it the same way as any downloadable subscription.

The same may is also true for "subscription.removed" in which case we're not receiving individual "filter.removed" events. Therefore a generic solution would be preferrable to avoid having to deal with all sorts of edge cases.

How to test using test environment

For the sake of testing all different variants I've used the following instructions using the local test environment.

Code to use

window.postMessage({
  type: "message",
  messageId: -1,
  payload: {
    type: "subscriptions.respond",
    action: "added",
    args: [
      {
        url: "~user~1234",
        filters: [{text: "@@||example.com^$document", disabled: true}]
      }
    ]
  }
}, "*");

Steps to reproduce

1. Open mobile-options.html?showPageOptions=true
2. Execute code with action=added and disabled=true
3. Execute code with action=removed and disabled=true
4. Execute code with action=added and disabled=false
5. Execute code with action=removed and disabled=false

Observed behavior

• Initial
  • Custom subscription is not shown
  • Whitelist toggle is true
• After 2:
  • Custom subscription is shown
  • Whitelist toggle is true
• After 3:
  • Custom subscription is not shown
  • Whitelist toggle is true
• After 4:
  • Custom subscription is shown
  • Whitelist toggle is true
• After 5:
  • Custom subscription does not exist
  • Whitelist toggle is true

Expected behavior

• Initial
  • Custom subscription is not shown
  • Whitelist toggle is true
• After 2:
  • Custom subscription is not shown
  • Whitelist toggle is true
• After 3:
  • Custom subscription is not shown
  • Whitelist toggle is true
• After 4:
  • Custom subscription is not shown
  • Whitelist toggle is false
• After 5:
  • Custom subscription is not shown
  • Whitelist toggle is true

[greiner]

@saroyanm FYI: This issue likely also affects the desktop version of the new options page.

[greiner]

The toggle button not changing was due to the changes I had to make in adblockpluschrome to be able to test it. Therefore I removed this part from the ticket description since it does work as expected.

[abpbot]

A commit referencing this issue has landed:
​Issue 5598 - Fixed: Consider filter changes due to subscription change