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

Side by Side Diff: lib/uninstall.js

Issue 29338534: Issue 3826 - Filter preference change events (Closed)
Patch Set: Created March 17, 2016, 4:30 p.m.
Left:
Right:
Use n/p to move between diff chunks; N/P to move between comments.
Jump to:
View unified diff | Download patch
OLDNEW
1 /* 1 /*
2 * This file is part of Adblock Plus <https://adblockplus.org/>, 2 * This file is part of Adblock Plus <https://adblockplus.org/>,
3 * Copyright (C) 2006-2016 Eyeo GmbH 3 * Copyright (C) 2006-2016 Eyeo GmbH
4 * 4 *
5 * Adblock Plus is free software: you can redistribute it and/or modify 5 * Adblock Plus is free software: you can redistribute it and/or modify
6 * it under the terms of the GNU General Public License version 3 as 6 * it under the terms of the GNU General Public License version 3 as
7 * published by the Free Software Foundation. 7 * published by the Free Software Foundation.
8 * 8 *
9 * Adblock Plus is distributed in the hope that it will be useful, 9 * Adblock Plus is distributed in the hope that it will be useful,
10 * but WITHOUT ANY WARRANTY; without even the implied warranty of 10 * but WITHOUT ANY WARRANTY; without even the implied warranty of
(...skipping 39 matching lines...) Expand 10 before | Expand all | Expand 10 after
50 chrome.runtime.setUninstallURL(Utils.getDocLink("uninstalled") + "&" + 50 chrome.runtime.setUninstallURL(Utils.getDocLink("uninstalled") + "&" +
51 search.join("&")); 51 search.join("&"));
52 } 52 }
53 53
54 // The uninstall URL contains the notification download count as a parameter, 54 // The uninstall URL contains the notification download count as a parameter,
55 // therefore we must wait for preferences to be loaded before generating the 55 // therefore we must wait for preferences to be loaded before generating the
56 // URL and we need to re-generate it each time the notification data changes. 56 // URL and we need to re-generate it each time the notification data changes.
57 if ("setUninstallURL" in chrome.runtime) 57 if ("setUninstallURL" in chrome.runtime)
58 { 58 {
59 Prefs.untilLoaded.then(setUninstallURL); 59 Prefs.untilLoaded.then(setUninstallURL);
60 Prefs.onChanged.addListener(function(name) 60 Prefs.on("notificationdata", setUninstallURL);
61 {
62 if (name == "notificationdata")
63 setUninstallURL();
64 });
65 } 61 }
OLDNEW

Powered by Google App Engine
This is Rietveld