Rietveld Code Review Tool
Help | Bug tracker | Discussion group | Source code

Unified Diff: messageResponder.js

Issue 29339395: Noissue - Make the options page respond to subscription title and homepage changes (Closed)
Patch Set: Addressed comments Created April 6, 2016, 5:25 p.m.
Use n/p to move between diff chunks; N/P to move between comments.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « background.js ('k') | options.js » ('j') | options.js » ('J')
Expand Comments ('e') | Collapse Comments ('c') | Show Comments 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];
+ if (key in obj)
+ result[key] = obj[key];
+ }
return result;
}
« no previous file with comments | « background.js ('k') | options.js » ('j') | options.js » ('J')

Powered by Google App Engine
This is Rietveld