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

Unified Diff: src/plugin/AdblockPlusDomTraverser.cpp

Issue 29331669: Issue 3237 - Whitelisted AFS ad is not shown on netzwelt.de (Closed)
Patch Set: Move subdocument detection to the top. Created Dec. 15, 2015, 2:05 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 | « no previous file | src/plugin/PluginDomTraverserBase.h » ('j') | src/plugin/PluginWbPassThrough.cpp » ('J')
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: src/plugin/AdblockPlusDomTraverser.cpp
===================================================================
--- a/src/plugin/AdblockPlusDomTraverser.cpp
+++ b/src/plugin/AdblockPlusDomTraverser.cpp
@@ -34,7 +34,7 @@
// If src should be blocked, set style display:none on iframe
bool isBlocked = client->ShouldBlock(url,
- AdblockPlus::FilterEngine::ContentType::CONTENT_TYPE_SUBDOCUMENT, m_domain);
+ AdblockPlus::FilterEngine::ContentType::CONTENT_TYPE_SUBDOCUMENT, m_documentUrl);
if (isBlocked)
{
HideElement(pEl, L"iframe", url, true, indent);
@@ -72,7 +72,7 @@
// If src should be blocked, set style display:none on image
cache->m_isHidden = client->ShouldBlock(src,
- AdblockPlus::FilterEngine::ContentType::CONTENT_TYPE_IMAGE, m_domain);
+ AdblockPlus::FilterEngine::ContentType::CONTENT_TYPE_IMAGE, m_documentUrl);
if (cache->m_isHidden)
{
HideElement(pEl, L"image", src, true, indent);
« no previous file with comments | « no previous file | src/plugin/PluginDomTraverserBase.h » ('j') | src/plugin/PluginWbPassThrough.cpp » ('J')

Powered by Google App Engine
This is Rietveld