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

Unified Diff: src/plugin/PluginTabBase.cpp

Issue 4784041706389504: Issue 1183 - Fix Acceptable Ads hiding (Closed)
Patch Set: Addressed comments, reorganized the code Created Aug. 8, 2014, 4:37 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/PluginTabBase.cpp
===================================================================
--- a/src/plugin/PluginTabBase.cpp
+++ b/src/plugin/PluginTabBase.cpp
@@ -183,7 +183,8 @@
void CPluginTabBase::OnDownloadComplete(IWebBrowser2* browser)
{
#ifdef SUPPORT_DOM_TRAVERSER
- if (!CPluginClient::GetInstance()->IsWhitelistedUrl(std::wstring(GetDocumentUrl())))
+ CPluginClient* client = CPluginClient::GetInstance();
+ if (!client->IsWhitelistedUrl(std::wstring(GetDocumentUrl())) && !client->IsHidingWhitelistedOnDomain(std::wstring(GetDocumentUrl())))
Felix Dahlke 2014/08/08 16:41:45 std::wstring(GetDocumentUrl()) would like to be a
{
m_traverser->TraverseDocument(browser, GetDocumentDomain(), GetDocumentUrl());
}

Powered by Google App Engine
This is Rietveld