Unified Diff: subscriptionLink.postload.js
Issue 29374674:
Issue 4864 - Start using ESLint for adblockpluschrome (Closed)
Patch Set: Reduce inline eslint-disable comments
Created Feb. 8, 2017, 7:27 a.m.
Use n/p to move between diff chunks;
N/P to move between comments.
« no previous file with comments
|
« stats.js ('k')
|
utils.js » ('j')
|
no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Hide Comments ('s')
Expand Comments ('e') | Collapse Comments ('c') | Hide Comments ('s')
| Index: subscriptionLink.postload.js |
| diff --git a/subscriptionLink.postload.js b/subscriptionLink.postload.js |
| index 0aaf68c78dd12e4f82afa3f37b3138529207893f..9f2fc26b5d3476e15a2e18c94944047e8d6cbf70 100644 |
| --- a/subscriptionLink.postload.js |
| +++ b/subscriptionLink.postload.js |
| @@ -45,7 +45,9 @@ if (document instanceof HTMLDocument) |
| return; |
| } |
| else if (!/^abp:\/*subscribe\/*\?/i.test(link.href)) |
| + { |
| return; |
| + } |
| // This is our link - make sure the browser doesn't handle it |
| event.preventDefault(); |
| @@ -84,8 +86,8 @@ if (document instanceof HTMLDocument) |
| ext.backgroundPage.sendMessage({ |
| type: "subscriptions.add", |
| - title: title, |
| - url: url, |
| + title, |
| + url, |
| confirm: true |
| }); |
| }, true); |
« no previous file with comments
|
« stats.js ('k')
|
utils.js » ('j')
|
no next file with comments »
