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

Unified Diff: src/plugin/AdblockPlusClient.cpp

Issue 10948032: Fix domain-based whitelisting (Closed)
Patch Set: Created June 17, 2013, 1:50 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
« no previous file with comments | « src/plugin/AdblockPlusClient.h ('k') | src/plugin/PluginClass.cpp » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: src/plugin/AdblockPlusClient.cpp
===================================================================
--- a/src/plugin/AdblockPlusClient.cpp
+++ b/src/plugin/AdblockPlusClient.cpp
@@ -197,11 +197,6 @@
isWhitelisted = m_filter.get() && m_filter->ShouldWhiteList(url);
}
m_criticalSectionFilter.Unlock();
-
- if (isWhitelisted)
- {
- CacheWhiteListedUrl(url, isWhitelisted);
- }
}
return isWhitelisted;
@@ -362,3 +357,18 @@
DEBUG_GENERAL(e.what());
}
}
+
+void CAdblockPlusClient::RemoveFilter(const std::string& text)
+{
+ Communication::OutputBuffer request;
+ request << Communication::PROC_REMOVE_FILTER << text;
+
+ try
+ {
+ CallAdblockPlusEngineProcedure(request);
+ }
+ catch (const std::exception& e)
+ {
+ DEBUG_GENERAL(e.what());
+ }
+}
« no previous file with comments | « src/plugin/AdblockPlusClient.h ('k') | src/plugin/PluginClass.cpp » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld