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

Unified Diff: test/BaseJsTest.cpp

Issue 29508555: Issue 5450 - move FilterEngine into Platform (Closed) Base URL: https://github.com/adblockplus/libadblockplus.git
Patch Set: Created Aug. 7, 2017, 8:28 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: test/BaseJsTest.cpp
diff --git a/test/BaseJsTest.cpp b/test/BaseJsTest.cpp
index a9720545b183141d5fb7b6986fb3d75c0e5c9dcd..f45e2b9274adba9aeb986de73635a226e07ffc45 100644
--- a/test/BaseJsTest.cpp
+++ b/test/BaseJsTest.cpp
@@ -36,7 +36,7 @@ void DelayedTimer::ProcessImmediateTimers(DelayedTimer::SharedTasks& timerTasks)
}
FilterEnginePtr CreateFilterEngine(LazyFileSystem& fileSystem,
- const JsEnginePtr& jsEngine,
+ Platform& paltform,
hub 2017/08/07 12:05:56 typo here too.
sergei 2017/08/07 13:10:50 Good catch!
const FilterEngine::CreationParameters& creationParams)
{
std::list<LazyFileSystem::Task> fileSystemTasks;
@@ -45,11 +45,11 @@ FilterEnginePtr CreateFilterEngine(LazyFileSystem& fileSystem,
fileSystemTasks.emplace_back(task);
};
FilterEnginePtr retValue;
- FilterEngine::CreateAsync(jsEngine, [&retValue, &fileSystem](const FilterEnginePtr& filterEngine)
+ paltform.CreateFilterEngineAsync(creationParams, [&retValue, &fileSystem](const FilterEnginePtr& filterEngine)
hub 2017/08/07 12:05:56 and here.
{
retValue = filterEngine;
fileSystem.scheduler = LazyFileSystem::ExecuteImmediately;
- }, creationParams);
+ });
while (!retValue && !fileSystemTasks.empty())
{
(*fileSystemTasks.begin())();
« test/BaseJsTest.h ('K') | « test/BaseJsTest.h ('k') | test/FilterEngine.cpp » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld