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

Side by Side Diff: lib/adblockplus.js

Issue 29348698: NoIssue - Use localStorage for storing filters in current Windows Store version of ABP for Edge (Closed)
Patch Set: Address the comments Created July 27, 2016, 4:32 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
« no previous file with comments | « no previous file | manifest.json » ('j') | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
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 342 matching lines...) Expand 10 before | Expand all | Expand 10 after
353 var keyPrefix = "file:"; 353 var keyPrefix = "file:";
354 354
355 function fileToKey(file) 355 function fileToKey(file)
356 { 356 {
357 return keyPrefix + (file instanceof FakeFile ? file.path : file.spec); 357 return keyPrefix + (file instanceof FakeFile ? file.path : file.spec);
358 } 358 }
359 359
360 function loadFile(file, successCallback, errorCallback) 360 function loadFile(file, successCallback, errorCallback)
361 { 361 {
362 var key = fileToKey(file); 362 var key = fileToKey(file);
363 ext.storage.get([key], function(items) 363 var entry = localStorage.getItem(key);
364 if (entry)
364 { 365 {
365 var entry = items[key]; 366 successCallback(JSON.parse(entry));
366 if (entry) 367 }
368 else
369 {
370 // Also check the chrome.storage.local
371 // We may have a init data there
372 ext.storage.get([key], function(items)
367 { 373 {
368 successCallback(entry); 374 var entry = items[key];
369 } 375 if (entry)
370 else 376 successCallback(entry);
371 { 377 else
372 errorCallback(new Error("File doesn't exist")); 378 errorCallback(new Error("File doesn't exist"));
373 } 379 });
374 }); 380 }
375 } 381 }
376
377 function saveFile(file, data, callback) 382 function saveFile(file, data, callback)
378 { 383 {
379 ext.storage.set(fileToKey(file), 384 try
380 { 385 {
381 content: data, 386 localStorage.setItem(fileToKey(file), JSON.stringify({
382 lastModified: Date.now() 387 content: data,
383 }, callback); 388 lastModified: Date.now()
389 }));
390 }
391 catch(error)
392 {
393 // QuotaExceededError can happen. Notify the user and ignore
394 var errorMessage = "Subscription storage is full. " +
395 "Please remove some subscriptions and try again.";
396 alert(errorMessage);
397 callback(new Error(errorMessage));
398 return;
399 }
400 callback();
384 } 401 }
385 exports.IO = { 402 exports.IO = {
386 resolveFilePath: function(path) 403 resolveFilePath: function(path)
387 { 404 {
388 return new FakeFile(path); 405 return new FakeFile(path);
389 }, 406 },
390 readFromFile: function(file, listener, callback) 407 readFromFile: function(file, listener, callback)
391 { 408 {
392 function onLoaded(entry) 409 function onLoaded(entry)
393 { 410 {
(...skipping 6143 matching lines...) Expand 10 before | Expand all | Expand 10 after
6537 search.push("notificationDownloadCount=" + encodeURIComponent(downlCount)); 6554 search.push("notificationDownloadCount=" + encodeURIComponent(downlCount));
6538 chrome.runtime.setUninstallURL(Utils.getDocLink("uninstalled") + "&" + searc h.join("&")); 6555 chrome.runtime.setUninstallURL(Utils.getDocLink("uninstalled") + "&" + searc h.join("&"));
6539 } 6556 }
6540 if ("setUninstallURL" in chrome.runtime) 6557 if ("setUninstallURL" in chrome.runtime)
6541 { 6558 {
6542 Prefs.untilLoaded.then(setUninstallURL); 6559 Prefs.untilLoaded.then(setUninstallURL);
6543 Prefs.on("notificationdata", setUninstallURL); 6560 Prefs.on("notificationdata", setUninstallURL);
6544 } 6561 }
6545 return exports; 6562 return exports;
6546 })(); 6563 })();
OLDNEW
« no previous file with comments | « no previous file | manifest.json » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld