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

Unified Diff: include/AdblockPlus/FilterEngine.h

Issue 5653480979038208: Issue 2325 - Add a way to set settings in libadblockplus for FRP and automatic updates (Closed)
Patch Set: Created April 17, 2015, 3:54 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
« no previous file with comments | « no previous file | lib/init.js » ('j') | lib/init.js » ('J')
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: include/AdblockPlus/FilterEngine.h
===================================================================
--- a/include/AdblockPlus/FilterEngine.h
+++ b/include/AdblockPlus/FilterEngine.h
@@ -186,11 +186,14 @@
typedef std::tr1::function<void(const std::string&, const JsValuePtr)> FilterChangeCallback;
/**
- * Constructor.
+ * Constructor. Allows disabling FRP and auto updates
* @param jsEngine `JsEngine` instance used to run JavaScript code
* internally.
+ * @param disableAutoUpdates `bool` true if automatic updates should
+ * be disabled
+ * @param disableFRP `bool` true if first run page should be disabled
*/
- explicit FilterEngine(JsEnginePtr jsEngine);
+ explicit FilterEngine(JsEnginePtr jsEngine, bool firstRunDisabled = false, bool autoUpdatesDisabled = false);
Eric 2015/05/15 19:36:13 Even at this early stage, I think it would be bett
Felix Dahlke 2015/05/28 20:42:56 I agree - in fact, I had something more generic in
/**
* Retrieves the `JsEngine` instance associated with this `FilterEngine`
« no previous file with comments | « no previous file | lib/init.js » ('j') | lib/init.js » ('J')

Powered by Google App Engine
This is Rietveld