Unified Diff: src/plugin/PluginFilter.cpp
Issue 10955040:
Fix domain whitelisting and remove unused code (Closed)
Patch Set: Addressed all issues
Created June 25, 2013, 2:19 p.m.
Use n/p to move between diff chunks;
N/P to move between comments.
« src/plugin/AdblockPlusClient.cpp ('K')
|
« src/plugin/PluginFilter.h ('k')
|
src/plugin/PluginSettings.h » ('j')
|
src/plugin/PluginTabBase.cpp » ('J')
Expand Comments ('e') | Collapse Comments ('c') | Hide Comments ('s')
Expand Comments ('e') | Collapse Comments ('c') | Hide Comments ('s')
| Index: src/plugin/PluginFilter.cpp |
| =================================================================== |
| --- a/src/plugin/PluginFilter.cpp |
| +++ b/src/plugin/PluginFilter.cpp |
| @@ -682,21 +682,6 @@ |
| } |
| } |
| - |
| -bool CPluginFilter::ShouldWhiteList(CString src) const |
| -{ |
| - // We should not block the empty string, so all filtering does not make sense |
| - // Therefore we just return |
| - if (src.Trim().IsEmpty()) |
| - { |
| - return false; |
| - } |
| - |
| - //TODO: Implement whitelisting check from libadblockplus here. Probably not needed anymore |
| - return false; |
| -} |
| - |
| - |
| bool CPluginFilter::ShouldBlock(CString src, int contentType, const CString& domain, bool addDebug) const |
| { |
| // We should not block the empty string, so all filtering does not make sense |
« src/plugin/AdblockPlusClient.cpp ('K')
|
« src/plugin/PluginFilter.h ('k')
|
src/plugin/PluginSettings.h » ('j')
|
src/plugin/PluginTabBase.cpp » ('J')
