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

Unified Diff: lib/uninstall.js

Issue 29332492: Issue 3269 - Display uninstallation page when uninstalled (Closed)
Patch Set: Addressed more feedback Created Dec. 12, 2015, 2:11 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 | « no previous file | metadata.common » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: lib/uninstall.js
diff --git a/lib/uninstall.js b/lib/uninstall.js
new file mode 100644
index 0000000000000000000000000000000000000000..d100a5c53d75359d6f979ed717100ddba02369c8
--- /dev/null
+++ b/lib/uninstall.js
@@ -0,0 +1,53 @@
+/*
+ * This file is part of Adblock Plus <https://adblockplus.org/>,
+ * Copyright (C) 2006-2015 Eyeo GmbH
+ *
+ * Adblock Plus is free software: you can redistribute it and/or modify
+ * it under the terms of the GNU General Public License version 3 as
+ * published by the Free Software Foundation.
+ *
+ * Adblock Plus is distributed in the hope that it will be useful,
+ * but WITHOUT ANY WARRANTY; without even the implied warranty of
+ * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
+ * GNU General Public License for more details.
+ *
+ * You should have received a copy of the GNU General Public License
+ * along with Adblock Plus. If not, see <http://www.gnu.org/licenses/>.
+ */
+
+/** @module uninstall */
+
+let info = require("info");
+let {Prefs} = require("prefs");
+let {Utils} = require("utils");
+
+function setUninstallURL()
+{
+ let search = [];
+ let keys = ["addonName", "addonVersion", "application", "applicationVersion",
+ "platform", "platformVersion"];
+ for (let key of keys)
+ search.push(key + "=" + encodeURIComponent(info[key]));
+
+ let downlCount = Prefs.notificationdata.downloadCount || 0;
+ search.push("notificationDownloadCount=" + encodeURIComponent(downlCount));
+
+ chrome.runtime.setUninstallURL(Utils.getDocLink("uninstalled") + "&" +
Sebastian Noack 2015/12/15 06:41:18 How I could I miss that? Of course, we have to che
kzar 2015/12/15 11:34:16 Done.
+ search.join("&"));
+}
+
+function onPrefsLoaded()
+{
+ Prefs.onLoaded.removeListener(onPrefsLoaded);
+ setUninstallURL();
+}
+
+// The uninstall URL contains the notification download count as a parameter,
+// therefore we must wait for preferences to be loaded before generating the
+// URL and we need to re-generate it each time the notification data changes.
+Prefs.onLoaded.addListener(onPrefsLoaded);
+Prefs.onChanged.addListener(function(name)
+{
+ if (name == "notificationdata")
+ setUninstallURL();
+});
« no previous file with comments | « no previous file | metadata.common » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld