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

Unified Diff: src/plugin/PluginDomTraverserBase.h

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
Index: src/plugin/PluginDomTraverserBase.h
===================================================================
--- a/src/plugin/PluginDomTraverserBase.h
+++ b/src/plugin/PluginDomTraverserBase.h
@@ -62,7 +62,7 @@
CComAutoCriticalSection m_criticalSection;
std::wstring m_domain;
- std::wstring m_documentName;
+ std::wstring m_documentUrl;
bool m_isHeaderTraversed;
@@ -104,7 +104,7 @@
void CPluginDomTraverserBase<T>::TraverseDocument(IWebBrowser2* pBrowser, const std::wstring& domain, const std::wstring& documentUrl)
{
m_domain = domain;
-
+ m_documentUrl = documentUrl;
TraverseDocument(pBrowser, true, L"");
}
@@ -113,7 +113,7 @@
void CPluginDomTraverserBase<T>::TraverseSubdocument(IWebBrowser2* pBrowser, const std::wstring& domain, const std::wstring& documentUrl)
{
m_domain = domain;
-
+ m_documentUrl = documentUrl;
TraverseDocument(pBrowser, false, L"");
}
@@ -207,8 +207,8 @@
m_criticalSection.Lock();
{
- hasFrames = m_cacheDocumentHasFrames.find(m_documentName) != m_cacheDocumentHasFrames.end();
- hasIframes = m_cacheDocumentHasIframes.find(m_documentName) != m_cacheDocumentHasIframes.end();
+ hasFrames = m_cacheDocumentHasFrames.find(m_documentUrl) != m_cacheDocumentHasFrames.end();
+ hasIframes = m_cacheDocumentHasIframes.find(m_documentUrl) != m_cacheDocumentHasIframes.end();
}
m_criticalSection.Unlock();
@@ -396,7 +396,7 @@
{
m_criticalSection.Lock();
{
- m_cacheDocumentHasIframes.insert(m_documentName);
+ m_cacheDocumentHasIframes.insert(m_documentUrl);
}
m_criticalSection.Unlock();
}
@@ -404,7 +404,7 @@
{
m_criticalSection.Lock();
{
- m_cacheDocumentHasFrames.insert(m_documentName);
+ m_cacheDocumentHasFrames.insert(m_documentUrl);
}
m_criticalSection.Unlock();
}

Powered by Google App Engine
This is Rietveld