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

Unified Diff: src/plugin/PluginTabBase.cpp

Issue 11292028: Status bar menu added (Closed)
Patch Set: Comments addressed Created Aug. 5, 2013, 1:13 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
« no previous file with comments | « src/plugin/PluginSettings.cpp ('k') | src/plugin/PluginUtil.h » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: src/plugin/PluginTabBase.cpp
===================================================================
--- a/src/plugin/PluginTabBase.cpp
+++ b/src/plugin/PluginTabBase.cpp
@@ -25,18 +25,6 @@
int CPluginTabBase::s_configVersion = 0;
#endif
-namespace
-{
- CString ExtractDomain(const CString& url)
- {
- int pos = 0;
- if (url.Find('/', pos) >= 0)
- url.Tokenize(L"/", pos);
- CString domain = url.Tokenize(L"/", pos);
- domain.MakeLower();
- return domain;
- }
-}
CPluginTabBase::CPluginTabBase(CPluginClass* plugin) : m_plugin(plugin), m_isActivated(false)
{
@@ -115,7 +103,10 @@
void CPluginTabBase::OnDownloadComplete(IWebBrowser2* browser)
{
#ifdef SUPPORT_DOM_TRAVERSER
- m_traverser->TraverseDocument(browser, GetDocumentDomain(), GetDocumentUrl());
+ if (!CPluginClient::GetInstance()->IsWhitelistedUrl(std::wstring(GetDocumentUrl())))
+ {
+ m_traverser->TraverseDocument(browser, GetDocumentDomain(), GetDocumentUrl());
+ }
#endif // SUPPORT_DOM_TRAVERSER
}
« no previous file with comments | « src/plugin/PluginSettings.cpp ('k') | src/plugin/PluginUtil.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld