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

Unified Diff: adblockplus/issue-6108.patch

Issue 29863604: Issue 6865 - Update ABP dependency to version 3.2 (Closed)
Patch Set: Adjusting build script and removing extension first run page Created Jan. 8, 2019, 2:54 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
Index: adblockplus/issue-6108.patch
===================================================================
deleted file mode 100644
--- a/adblockplus/issue-6108.patch
+++ /dev/null
@@ -1,42 +0,0 @@
-Issue 6108 - No filter list is selected after a migration failure
-
-diff --git a/lib/filterStorage.js b/lib/filterStorage.js
---- a/lib/filterStorage.js
-+++ b/lib/filterStorage.js
-@@ -43,16 +43,18 @@ let formatVersion = 5;
- let FilterStorage = exports.FilterStorage =
- {
- /**
- * Will be set to true after the initial loadFromDisk() call completes.
- * @type {boolean}
- */
- initialized: false,
-
-+ loadFromDiskFailed: false,
-+
- /**
- * Version number of the patterns.ini format used.
- * @type {number}
- */
- get formatVersion()
- {
- return formatVersion;
- },
-@@ -398,16 +400,17 @@ let FilterStorage = exports.FilterStorag
- {
- return this.restoreBackup(backupIndex, true).then(() =>
- {
- if (this.subscriptions.length == 0)
- return tryBackup(backupIndex + 1);
- }).catch(error =>
- {
- // Give up
-+ this.loadFromDiskFailed = true;
- });
- };
-
- return IO.statFile(this.sourceFile).then(statData =>
- {
- if (!statData.exists)
- {
- this.firstRun = true;

Powered by Google App Engine
This is Rietveld