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

Unified Diff: lib/prefs.js

Issue 5653480979038208: Issue 2325 - Add a way to set settings in libadblockplus for FRP and automatic updates (Closed)
Patch Set: Simplify the setting of global property. Add more tests. Created June 12, 2015, 7:33 a.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: lib/prefs.js
===================================================================
--- a/lib/prefs.js
+++ b/lib/prefs.js
@@ -41,9 +41,13 @@
update_hard_expiration: 0,
currentVersion: "0.0",
notificationdata: {},
- notificationurl: "https://notification.adblockplus.org/notification.json"
+ notificationurl: "https://notification.adblockplus.org/notification.json",
+ suppress_first_run_page: false,
+ disable_auto_updates: false
};
+let preconfigurable = ["suppress_first_run_page", "disable_auto_updates"];
+
let values = Object.create(defaults);
let path = _fileSystem.resolve("prefs.json");
let listeners = [];
@@ -75,6 +79,15 @@
function load()
{
+ // Set default prefs based on _preconfiguredPrefs
+ for (let key in _preconfiguredPrefs)
+ {
+ if (preconfigurable.indexOf(key) != -1)
+ {
+ values[key] = _preconfiguredPrefs[key];
+ }
+ }
+
_fileSystem.read(path, function(result)
{
// prefs.json is expected to be missing, ignore errors reading file

Powered by Google App Engine
This is Rietveld