Unified Diff: messageResponder.js
Issue 29339395:
Noissue - Make the options page respond to subscription title and homepage changes (Closed)
Patch Set:
Created April 5, 2016, 7:05 p.m.
Use n/p to move between diff chunks;
N/P to move between comments.
« background.js ('K')
|
« background.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: messageResponder.js |
| =================================================================== |
| --- a/messageResponder.js |
| +++ b/messageResponder.js |
| @@ -60,7 +60,11 @@ |
| { |
| var result = {}; |
| for (var i = 0; i < keys.length; i++) |
| - result[keys[i]] = obj[keys[i]]; |
| + { |
| + var key = keys[i]; |
|
Sebastian Noack
2016/04/05 19:09:54
As per the comment in background.js we don't set t
|
| + if (key in obj) |
| + result[key] = obj[key]; |
| + } |
| return result; |
| } |
