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

Unified Diff: Shared/AdblockPlusClient.cpp

Issue 10338036: Fix errors, pass AppInfo (Closed)
Patch Set: Created April 19, 2013, 3:11 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 | « Shared/AdblockPlusClient.h ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: Shared/AdblockPlusClient.cpp
===================================================================
--- a/Shared/AdblockPlusClient.cpp
+++ b/Shared/AdblockPlusClient.cpp
@@ -18,11 +18,12 @@
CAdblockPlusClient::CAdblockPlusClient() : CPluginClientBase()
{
m_filter = std::auto_ptr<CPluginFilter>(new CPluginFilter());
- fileSystem = std::auto_ptr<AdblockPlus::DefaultFileSystem>(new AdblockPlus::DefaultFileSystem());
- webRequest = std::auto_ptr<AdblockPlus::DefaultWebRequest>(new AdblockPlus::DefaultWebRequest());
- errorCallback = std::auto_ptr<CerrErrorCallback>(new CerrErrorCallback());
- jsEngine = std::auto_ptr<AdblockPlus::JsEngine>(new AdblockPlus::JsEngine(fileSystem.get(), webRequest.get(), errorCallback.get()));
- filterEngine = std::auto_ptr<AdblockPlus::FilterEngine>(new AdblockPlus::FilterEngine(*jsEngine));
+ AdblockPlus::AppInfo appInfo;
+ appInfo.name = "Adblock Plus for Internet Explorer";
Wladimir Palant 2013/04/19 06:35:59 This isn't a string displayed to the user - rather
+ appInfo.version = CT2CA(_T(IEPLUGIN_VERSION), CP_UTF8);
+ appInfo.platform = "Internet Explorer";
Wladimir Palant 2013/04/19 06:35:59 Same here, it's not a user-visible string - rather
+ JsEnginePtr jsEngine(AdblockPlus::JsEngine::New(appInfo));
+ filterEngine = std::auto_ptr<AdblockPlus::FilterEngine>(new AdblockPlus::FilterEngine(jsEngine));
}
CAdblockPlusClient::~CAdblockPlusClient()
{
« no previous file with comments | « Shared/AdblockPlusClient.h ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld