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

Unified Diff: options.js

Issue 29339099: Noissue - Rename message type when propagating events to the content script (Closed)
Patch Set: Rename *.event to *.respond Created March 29, 2016, 11:11 a.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 | « messageResponder.js ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: options.js
===================================================================
--- a/options.js
+++ b/options.js
@@ -1079,7 +1079,7 @@
{
switch (message.type)
{
- case "app.listen":
+ case "app.respond":
if (message.action == "addSubscription")
{
var subscription = message.args[0];
@@ -1089,13 +1089,13 @@
openDialog("predefined");
}
break;
- case "filters.listen":
+ case "filters.respond":
onFilterMessage(message.action, message.args[0]);
break;
- case "prefs.listen":
+ case "prefs.respond":
onPrefMessage(message.action, message.args[0]);
break;
- case "subscriptions.listen":
+ case "subscriptions.respond":
onSubscriptionMessage(message.action, message.args[0]);
break;
}
« no previous file with comments | « messageResponder.js ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld