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

Unified Diff: test/FilterEngineStubs.cpp

Issue 10198022: Pass application data into libadblockplus (Closed)
Patch Set: Merged upstream, addressed issues, renamed AppInfo fields Created April 17, 2013, 2:47 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: test/FilterEngineStubs.cpp
===================================================================
--- a/test/FilterEngineStubs.cpp
+++ b/test/FilterEngineStubs.cpp
@@ -44,7 +44,8 @@
{
DummyFileSystem fileSystem;
DummyErrorCallback errorCallback;
- AdblockPlus::JsEngine jsEngine(&fileSystem, 0, &errorCallback);
+ AdblockPlus::JsEngine jsEngine(AdblockPlus::AppInfo(), &fileSystem, 0,
+ &errorCallback);
AdblockPlus::FilterEngine filterEngine(jsEngine);
AdblockPlus::FilterPtr filter1 = filterEngine.GetFilter("foo");
@@ -63,7 +64,8 @@
{
DummyFileSystem fileSystem;
DummyErrorCallback errorCallback;
- AdblockPlus::JsEngine jsEngine(&fileSystem, 0, &errorCallback);
+ AdblockPlus::JsEngine jsEngine(AdblockPlus::AppInfo(), &fileSystem, 0,
+ &errorCallback);
AdblockPlus::FilterEngine filterEngine(jsEngine);
AdblockPlus::FilterPtr filter = filterEngine.GetFilter("foo");
@@ -83,7 +85,8 @@
{
DummyFileSystem fileSystem;
DummyErrorCallback errorCallback;
- AdblockPlus::JsEngine jsEngine(&fileSystem, 0, &errorCallback);
+ AdblockPlus::JsEngine jsEngine(AdblockPlus::AppInfo(), &fileSystem, 0,
+ &errorCallback);
AdblockPlus::FilterEngine filterEngine(jsEngine);
ASSERT_EQ(0u, filterEngine.GetListedFilters().size());
AdblockPlus::FilterPtr filter = filterEngine.GetFilter("foo");
@@ -104,7 +107,8 @@
{
DummyFileSystem fileSystem;
DummyErrorCallback errorCallback;
- AdblockPlus::JsEngine jsEngine(&fileSystem, 0, &errorCallback);
+ AdblockPlus::JsEngine jsEngine(AdblockPlus::AppInfo(), &fileSystem, 0,
+ &errorCallback);
AdblockPlus::FilterEngine filterEngine(jsEngine);
AdblockPlus::SubscriptionPtr subscription = filterEngine.GetSubscription("foo");
@@ -124,7 +128,8 @@
{
DummyFileSystem fileSystem;
DummyErrorCallback errorCallback;
- AdblockPlus::JsEngine jsEngine(&fileSystem, 0, &errorCallback);
+ AdblockPlus::JsEngine jsEngine(AdblockPlus::AppInfo(), &fileSystem, 0,
+ &errorCallback);
AdblockPlus::FilterEngine filterEngine(jsEngine);
ASSERT_EQ(0u, filterEngine.GetListedSubscriptions().size());
AdblockPlus::SubscriptionPtr subscription = filterEngine.GetSubscription("foo");
@@ -145,7 +150,8 @@
{
DummyFileSystem fileSystem;
DummyErrorCallback errorCallback;
- AdblockPlus::JsEngine jsEngine(&fileSystem, 0, &errorCallback);
+ AdblockPlus::JsEngine jsEngine(AdblockPlus::AppInfo(), &fileSystem, 0,
+ &errorCallback);
AdblockPlus::FilterEngine filterEngine(jsEngine);
AdblockPlus::SubscriptionPtr subscription = filterEngine.GetSubscription("foo");
ASSERT_FALSE(subscription->IsUpdating());
@@ -158,7 +164,8 @@
{
DummyFileSystem fileSystem;
DummyErrorCallback errorCallback;
- AdblockPlus::JsEngine jsEngine(&fileSystem, 0, &errorCallback);
+ AdblockPlus::JsEngine jsEngine(AdblockPlus::AppInfo(), &fileSystem, 0,
+ &errorCallback);
AdblockPlus::FilterEngine filterEngine(jsEngine);
filterEngine.GetFilter("adbanner.gif")->AddToList();

Powered by Google App Engine
This is Rietveld