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

Unified Diff: compiled/ElemHideBase.cpp

Issue 29333474: Issue 4125 - [emscripten] Convert filter classes to C++ (Closed)
Patch Set: Reworked JS binding generation Created Feb. 1, 2016, 9:14 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: compiled/ElemHideBase.cpp
===================================================================
new file mode 100644
--- /dev/null
+++ b/compiled/ElemHideBase.cpp
@@ -0,0 +1,165 @@
+#include "ElemHideBase.h"
+#include "ElemHideFilter.h"
+#include "ElemHideException.h"
+#include "CSSPropertyFilter.h"
+#include "InvalidFilter.h"
+#include "StringScanner.h"
+
+ElemHideBase::ElemHideBase(const String& text,
+ String::size_type domainsEnd, String::size_type selectorStart)
+ : ActiveFilter(text, false),
+ mSelector(String(mText, selectorStart))
+{
+ if (domainsEnd)
+ ParseDomains(String(mText, 0, domainsEnd), u',');
+}
+
+Filter::Type ElemHideBase::Parse(const String& text,
+ String::size_type* domainsEnd,
+ String::size_type* selectorStart,
+ String::size_type* prefixEnd,
+ String::size_type* regexpStart,
+ String::size_type* regexpEnd,
+ String::size_type* suffixStart)
+{
+ StringScanner scanner(text);
+
+ // Domains part
+ loop:
René Jeschke 2016/02/02 11:11:22 Nit: this can be removed
Wladimir Palant 2016/02/02 17:55:18 Done.
+ while (!scanner.done())
+ {
+ String::value_type next = scanner.next();
+ if (next == u'#')
+ {
+ *domainsEnd = scanner.position();
+ break;
+ }
+
+ switch (next)
+ {
+ case u'/':
+ case u'*':
+ case u'|':
+ case u'@':
+ case u'"':
+ case u'!':
+ return Type::UNKNOWN;
+ }
+ }
+
+ bool exception = false;
+ String::value_type next = scanner.next();
+ if (next == u'@')
+ {
+ exception = true;
+ next = scanner.next();
+ }
+
+ if (next != u'#')
+ return Type::UNKNOWN;
+
+ // Selector part
+
+ // Selector shouldn't be empty
+ if (scanner.done())
+ return Type::UNKNOWN;
+
+ *selectorStart = scanner.position() + 1;
+ while (!scanner.done())
+ {
+ switch (scanner.next())
+ {
+ case u'{':
+ case u'}':
+ return Type::UNKNOWN;
+ }
+ }
+
+ if (exception)
+ return Type::ELEMHIDEEXCEPTION;
+
+ do
+ {
+ // Is this a CSS property rule maybe?
+ if (!prefixEnd)
+ {
+ // Caller doesn't care
+ break;
+ }
+
+ String searchString(u"[-abp-properties="_str);
+ *prefixEnd = text.find(searchString, *selectorStart);
+ if (*prefixEnd == text.npos ||
+ *prefixEnd + searchString.length() + 1 >= text.length())
+ {
+ break;
+ }
+
+ *regexpStart = *prefixEnd + searchString.length() + 1;
+ char16_t quotation = text[*regexpStart - 1];
+ if (quotation != u'\'' && quotation != u'"')
+ break;
+
+ *regexpEnd = text.find(quotation, *regexpStart);
+ if (*regexpEnd == text.npos || *regexpEnd + 1 >= text.length() ||
+ text[*regexpEnd + 1] != u']')
+ {
+ break;
+ }
+
+ *suffixStart = *regexpEnd + 2;
+ return Type::CSSPROPERTY;
+ } while (false);
+
+ return Type::ELEMHIDE;
+}
+
+Filter* ElemHideBase::Create(const String& text)
+{
+ size_t domainsEnd;
+ size_t selectorStart;
+ size_t prefixEnd;
+ size_t regexpStart;
+ size_t regexpEnd;
+ size_t suffixStart;
+ Type type = Parse(text, &domainsEnd, &selectorStart, &prefixEnd, &regexpStart,
+ &regexpEnd, &suffixStart);
+ if (type == Type::UNKNOWN)
+ return nullptr;
+
+ try
+ {
+ if (type == Type::ELEMHIDEEXCEPTION)
+ return new ElemHideException(text, domainsEnd, selectorStart);
+ else if (type == Type::CSSPROPERTY)
+ {
+ return new CSSPropertyFilter(text, domainsEnd, selectorStart, prefixEnd,
+ regexpStart, regexpEnd, suffixStart);
+ }
+ else
+ return new ElemHideFilter(text, domainsEnd, selectorStart);
+ }
+ catch (const String& e)
+ {
+ return new InvalidFilter(text, e);
+ }
+}
+
+String ElemHideBase::GetSelectorDomain() const
+{
+ /* TODO this is inefficient */
+ String result;
+ if (mDomains)
+ {
+ for (auto it = mDomains->begin(); it != mDomains->end(); ++it)
+ {
+ if (it->second && !it->first.empty())
+ {
+ if (!result.empty())
+ result.append(u',');
+ result.append(it->first);
+ }
+ }
+ }
+ return std::move(result);
+}
« no previous file with comments | « compiled/ElemHideBase.h ('k') | compiled/ElemHideException.h » ('j') | compiled/StringMap.h » ('J')

Powered by Google App Engine
This is Rietveld