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

Unified Diff: src/plugin/PluginFilter.cpp

Issue 6505394822184960: Issue 1109 - Support notifications (Closed)
Patch Set: fix naming Created July 29, 2015, 11:18 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/PluginFilter.cpp
diff --git a/src/plugin/PluginFilter.cpp b/src/plugin/PluginFilter.cpp
index 546766cc26e7bfc03f3e3ca89cbd5f5ea9812059..edcdbe607d5d5732ee900da8db3ab8349cf74f60 100644
--- a/src/plugin/PluginFilter.cpp
+++ b/src/plugin/PluginFilter.cpp
@@ -26,55 +26,12 @@
#include "mlang.h"
#include "..\shared\CriticalSection.h"
#include "..\shared\Utils.h"
+#include "..\shared\MsHTMLUtils.h"
// The filters are described at http://adblockplus.org/en/filters
static CriticalSection s_criticalSectionFilterMap;
-namespace
-{
- struct GetHtmlElementAttributeResult
- {
- GetHtmlElementAttributeResult() : isAttributeFound(false)
- {
- }
- std::wstring attributeValue;
- bool isAttributeFound;
- };
-
- GetHtmlElementAttributeResult GetHtmlElementAttribute(IHTMLElement& htmlElement,
- const ATL::CComBSTR& attributeName)
- {
- GetHtmlElementAttributeResult retValue;
- ATL::CComVariant vAttr;
- ATL::CComPtr<IHTMLElement4> htmlElement4;
- if (FAILED(htmlElement.QueryInterface(&htmlElement4)) || !htmlElement4)
- {
- return retValue;
- }
- ATL::CComPtr<IHTMLDOMAttribute> attributeNode;
- if (FAILED(htmlElement4->getAttributeNode(attributeName, &attributeNode)) || !attributeNode)
- {
- return retValue;
- }
- // we set that attribute found but it's not necessary that we can retrieve its value
- retValue.isAttributeFound = true;
- if (FAILED(attributeNode->get_nodeValue(&vAttr)))
- {
- return retValue;
- }
- if (vAttr.vt == VT_BSTR && vAttr.bstrVal)
- {
- retValue.attributeValue = vAttr.bstrVal;
- }
- else if (vAttr.vt == VT_I4)
- {
- retValue.attributeValue = std::to_wstring(vAttr.iVal);
- }
- return retValue;
- }
-}
-
// ============================================================================
// CFilterElementHideAttrSelector
// ============================================================================

Powered by Google App Engine
This is Rietveld