Unified Diff: notification.js
Issue 29374674:
Issue 4864 - Start using ESLint for adblockpluschrome (Closed)
Patch Set: Use var for ext declarations again
Created Feb. 8, 2017, 9:02 a.m.
Use n/p to move between diff chunks;
N/P to move between comments.
« lib/io.js ('K')
|
« lib/whitelisting.js ('k')
|
options.js » ('j')
|
options.js » ('J')
Expand Comments ('e') | Collapse Comments ('c') | Hide Comments ('s')
Expand Comments ('e') | Collapse Comments ('c') | Hide Comments ('s')
| Index: notification.js |
| diff --git a/notification.js b/notification.js |
| index bcc5d8554573ed06eb53b139529884653efecc02..3328828a0fcbcebf42b25555ad75db91ec50d052 100644 |
| --- a/notification.js |
| +++ b/notification.js |
| @@ -99,6 +99,7 @@ window.addEventListener("load", () => |
| break; |
| case "notification-optout": |
| Notification.toggleIgnoreCategory("*", true); |
| + // Falls through. |
|
Sebastian Noack
2017/02/09 01:04:51
Interesting, this isn't a regular eslint comment,
kzar
2017/02/20 10:27:33
Yea, I thought this was kind of interesting too. I
|
| case "notification-hide": |
| notificationElement.hidden = true; |
| notification.onClicked(); |
