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

Delta Between Two Patch Sets: lib/subscriptionInit.js

Issue 29664623: Issue 6403 - Updated adblockplusui dependency (Closed)
Left Patch Set: Made minor adjustments Created Jan. 25, 2018, 11:08 a.m.
Right Patch Set: Created Feb. 23, 2018, 11:51 a.m.
Left:
Right:
Use n/p to move between diff chunks; N/P to move between comments.
Jump to:
Left: Side by side diff | Download
Right: Side by side diff | Download
« no previous file with change/comment | « lib/prefs.js ('k') | metadata.chrome » ('j') | no next file with change/comment »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
LEFTRIGHT
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-present eyeo GmbH 3 * Copyright (C) 2006-present 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 10 matching lines...) Expand all
21 21
22 const {Subscription, DownloadableSubscription, 22 const {Subscription, DownloadableSubscription,
23 SpecialSubscription} = require("subscriptionClasses"); 23 SpecialSubscription} = require("subscriptionClasses");
24 const {FilterStorage} = require("filterStorage"); 24 const {FilterStorage} = require("filterStorage");
25 const {FilterNotifier} = require("filterNotifier"); 25 const {FilterNotifier} = require("filterNotifier");
26 const info = require("info"); 26 const info = require("info");
27 const {Prefs} = require("prefs"); 27 const {Prefs} = require("prefs");
28 const {Synchronizer} = require("synchronizer"); 28 const {Synchronizer} = require("synchronizer");
29 const {Utils} = require("utils"); 29 const {Utils} = require("utils");
30 const {initNotifications} = require("notificationHelper"); 30 const {initNotifications} = require("notificationHelper");
31 const {updatesVersion} = require("ui"); 31 const {updatesVersion} = require("../adblockplusui/lib/prefs");
32 32
33 let firstRun; 33 let firstRun;
34 let subscriptionsCallback = null; 34 let subscriptionsCallback = null;
35 35
36 /** 36 /**
37 * If there aren't any filters, the default subscriptions are added. 37 * If there aren't any filters, the default subscriptions are added.
38 * However, if patterns.ini already did exist and/or any preference 38 * However, if patterns.ini already did exist and/or any preference
39 * is set to a non-default value, this indicates that this isn't the 39 * is set to a non-default value, this indicates that this isn't the
40 * first run, but something went wrong. 40 * first run, but something went wrong.
41 * 41 *
(...skipping 167 matching lines...) Expand 10 before | Expand all | Expand 10 after
209 if (!Prefs.suppress_first_run_page) 209 if (!Prefs.suppress_first_run_page)
210 { 210 {
211 let page = null; 211 let page = null;
212 if (firstRun) 212 if (firstRun)
213 { 213 {
214 page = "firstRun.html"; 214 page = "firstRun.html";
215 } 215 }
216 // For now we're limiting the updates page to users of 216 // For now we're limiting the updates page to users of
217 // Chromium-based browsers to gage its impact 217 // Chromium-based browsers to gage its impact
218 else if (info.platform == "chromium" && 218 else if (info.platform == "chromium" &&
219 updatesVersion > Prefs.updates_version) 219 updatesVersion > Prefs.last_updates_page_displayed)
220 { 220 {
221 page = "updates.html"; 221 page = "updates.html";
222 } 222 }
223 223
224 if (page) 224 if (page)
225 { 225 {
226 browser.tabs.create({url: browser.extension.getURL(page)}); 226 browser.tabs.create({url: browser.extension.getURL(page)});
227 227
228 // Users should now be aware of the latest updates so we need to reflect 228 // For new users and users that have already seen this updates page we
229 // that in the preference to ensure they're not shown the updates page 229 // want to avoid showing it again for subsequent updates.
230 // multiple times 230 Prefs.last_updates_page_displayed = updatesVersion;
231 Prefs.updates_version = updatesVersion;
232 } 231 }
233 } 232 }
234 233
235 initNotifications(); 234 initNotifications();
236 } 235 }
237 236
238 Promise.all([FilterNotifier.once("load"), 237 Promise.all([FilterNotifier.once("load"),
239 Prefs.untilLoaded]).then(detectFirstRun) 238 Prefs.untilLoaded]).then(detectFirstRun)
240 .then(getSubscriptions) 239 .then(getSubscriptions)
241 .then(finishInitialization); 240 .then(finishInitialization);
(...skipping 11 matching lines...) Expand all
253 * Sets a callback that is called with an array of subscriptions to be added 252 * Sets a callback that is called with an array of subscriptions to be added
254 * during initialization. The callback must return an array of subscriptions 253 * during initialization. The callback must return an array of subscriptions
255 * that will effectively be added. 254 * that will effectively be added.
256 * 255 *
257 * @param {function} callback 256 * @param {function} callback
258 */ 257 */
259 exports.setSubscriptionsCallback = callback => 258 exports.setSubscriptionsCallback = callback =>
260 { 259 {
261 subscriptionsCallback = callback; 260 subscriptionsCallback = callback;
262 }; 261 };
LEFTRIGHT

Powered by Google App Engine
This is Rietveld