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

Unified Diff: desktop-options.js

Issue 29536764: Issue 5587, 5748 - Use mobile options page on Firefox for Android (Closed) Base URL: https://hg.adblockplus.org/adblockpluschrome/
Patch Set: Merge lib/browserAction.js into lib/options.js Created Sept. 30, 2017, 1:48 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 | « desktop-options.html ('k') | ext/background.js » ('j') | lib/options.js » ('J')
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: desktop-options.js
===================================================================
copy from options.js
copy to desktop-options.js
--- a/options.js
+++ b/desktop-options.js
@@ -740,8 +740,19 @@
case "prefs.respond":
onPrefMessage(message.action, message.args[0]);
break;
case "subscriptions.respond":
onSubscriptionMessage(message.action, message.args[0]);
break;
}
});
+
+$(() =>
Sebastian Noack 2017/10/02 01:47:07 So I guess this code isn't necessary anymore?
Manish Jethani 2017/10/02 08:03:12 Sorry about that, done.
+{
+ ext.onMessage.addListener((message, sender, sendResponse) =>
+ {
+ if (message.type == "ping")
+ sendResponse(true);
+ });
+
+ ext.backgroundPage.sendMessage({type: "ping"});
+});
« no previous file with comments | « desktop-options.html ('k') | ext/background.js » ('j') | lib/options.js » ('J')

Powered by Google App Engine
This is Rietveld