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

Unified Diff: lib/synchronizer.js

Issue 6305806509146112: Issue 427 - Remove non-standard function and getter syntax (Closed)
Patch Set: Convert shorthand function expressions to arrow functions, this of course isn't possible for getter… Created May 10, 2014, 12:43 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
Index: lib/synchronizer.js
===================================================================
--- a/lib/synchronizer.js
+++ b/lib/synchronizer.js
@@ -306,17 +306,17 @@ let Synchronizer = exports.Synchronizer
if (!(subscription.url in FilterStorage.knownSubscriptions))
return;
let match = /^(\d+)(?:\s+(\S+))?$/.exec(request.responseText);
if (match && match[1] == "301" && match[2] && /^https?:\/\//i.test(match[2])) // Moved permanently
redirectCallback(match[2]);
else if (match && match[1] == "410") // Gone
{
- let data = "[Adblock]\n" + subscription.filters.map(function(f) f.text).join("\n");
+ let data = "[Adblock]\n" + subscription.filters.map((f) => f.text).join("\n");
redirectCallback("data:text/plain," + encodeURIComponent(data));
}
}, false);
request.send(null);
}
}
},
};

Powered by Google App Engine
This is Rietveld