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

Unified Diff: src/plugin/AdblockPlusClient.cpp

Issue 6567422169448448: Issue 119 - Switch to injecting CSS for element hiding (Closed)
Patch Set: rebase, improve memory handling (OnQuit) and improve code organizing Created April 27, 2016, 1:10 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: src/plugin/AdblockPlusClient.cpp
diff --git a/src/plugin/AdblockPlusClient.cpp b/src/plugin/AdblockPlusClient.cpp
index 656bcfe255bcb1b54ed222369f2d2538df0dcdf3..30d16bbe5349c6b21b1bd64411bcf35eeaa2b362 100644
--- a/src/plugin/AdblockPlusClient.cpp
+++ b/src/plugin/AdblockPlusClient.cpp
@@ -244,17 +244,6 @@ bool CAdblockPlusClient::ShouldBlock(const std::wstring& src, AdblockPlus::Filte
return isBlocked;
}
-bool CAdblockPlusClient::IsElementHidden(const std::wstring& tag, IHTMLElement* pEl, const std::wstring& domain, const std::wstring& indent, CPluginFilter* filter)
-{
- bool isHidden;
- m_criticalSectionFilter.Lock();
- {
- isHidden = filter && filter->IsElementHidden(tag, pEl, domain, indent);
- }
- m_criticalSectionFilter.Unlock();
- return isHidden;
-}
-
bool CAdblockPlusClient::IsWhitelistedUrl(const std::wstring& url, const std::vector<std::string>& frameHierarchy)
{
return !GetWhitelistingFilter(url, frameHierarchy).empty();

Powered by Google App Engine
This is Rietveld