Unified Diff: background.js
Issue 29338595:
Issue 3829 - Merge add-subscriptions and subscriptions.add messages (Closed)
Patch Set:
Created March 17, 2016, 9:20 p.m.
Use n/p to move between diff chunks;
N/P to move between comments.
« no previous file with comments
|
« no previous file
|
messageResponder.js » ('j')
|
messageResponder.js » ('J')
Expand Comments ('e') | Collapse Comments ('c') | Hide Comments ('s')
Expand Comments ('e') | Collapse Comments ('c') | Hide Comments ('s')
| Index: background.js |
| =================================================================== |
| --- a/background.js |
| +++ b/background.js |
| @@ -337,7 +337,8 @@ |
| payload: { |
| title: "Custom subscription", |
| url: "http://example.com/custom.txt", |
| - type: "add-subscription" |
| + ask: true, |
| + type: "subscription.add" |
|
Thomas Greiner
2016/03/18 10:48:20
The message type is "subscriptions.add".
Sebastian Noack
2016/03/18 11:35:07
Argh, right.
|
| } |
| }, "*"); |
| }, 1000); |
« no previous file with comments
|
« no previous file
|
messageResponder.js » ('j')
|
messageResponder.js » ('J')
