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

Unified Diff: test/Prefs.cpp

Issue 29361562: Issue 3594 - remove circular references JsEngine-JsValue-JsEngine (Closed)
Patch Set: temporary workaround for race condition Created Dec. 1, 2016, 10:26 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
« src/JsValue.cpp ('K') | « test/JsEngine.cpp ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: test/Prefs.cpp
diff --git a/test/Prefs.cpp b/test/Prefs.cpp
index ce217979fe96b6e9692df5682bfc8e6140493e7c..9d0a43c0ca07e83bf64671ac47ecdc3dbbd523d3 100644
--- a/test/Prefs.cpp
+++ b/test/Prefs.cpp
@@ -74,6 +74,7 @@ namespace
{
fileSystem = new TestFileSystem();
fileSystemPtr.reset(fileSystem);
+ jsEngine = CreateJsEngine();
Reset();
}
@@ -81,7 +82,6 @@ namespace
void Reset(const AdblockPlus::FilterEngine::Prefs& preconfiguredPrefs =
AdblockPlus::FilterEngine::Prefs())
{
- jsEngine = CreateJsEngine();
jsEngine->SetLogSystem(AdblockPlus::LogSystemPtr(new LazyLogSystem));
jsEngine->SetFileSystem(fileSystemPtr);
jsEngine->SetWebRequest(AdblockPlus::WebRequestPtr(new LazyWebRequest));
« src/JsValue.cpp ('K') | « test/JsEngine.cpp ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld