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

Unified Diff: src/plugin/AdblockPlusClient.h

Issue 5750789393874944: [IE] First round of ATL removal (Closed)
Patch Set: Created June 20, 2014, 9:22 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.h
===================================================================
--- a/src/plugin/AdblockPlusClient.h
+++ b/src/plugin/AdblockPlusClient.h
@@ -28,7 +28,7 @@
CComAutoCriticalSection m_criticalSectionFilter;
CComAutoCriticalSection m_criticalSectionCache;
- std::map<CString,bool> m_cacheBlockedSources;
+ std::map< std::wstring, bool > m_cacheBlockedSources;
std::shared_ptr<Communication::Pipe> enginePipe;
CriticalSection enginePipeLock;
@@ -49,10 +49,10 @@
// Removes the url from the list of whitelisted urls if present
// Only called from ui thread
- bool ShouldBlock(CString src, int contentType, const CString& domain, bool addDebug=false);
+ bool ShouldBlock( std::wstring src, int contentType, const std::wstring & domain, bool addDebug=false);
sergei 2014/07/08 11:58:34 `src` also should be `const std::wstring&`. It hap
- bool IsElementHidden(const CString& tag, IHTMLElement* pEl, const CString& domain, const CString& indent, CPluginFilter* filter);
- bool IsWhitelistedUrl(const std::wstring& url);
+ bool IsElementHidden(const std::wstring & tag, IHTMLElement* pEl, const std::wstring & domain, const std::wstring & indent, CPluginFilter* filter);
+ bool IsWhitelistedUrl(const std::wstring & url);
int GetIEVersion();

Powered by Google App Engine
This is Rietveld