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

Delta Between Two Patch Sets: compiled/CSSPropertyFilter.h

Issue 29333474: Issue 4125 - [emscripten] Convert filter classes to C++ (Closed)
Left Patch Set: Addressed comments from Patch Set 24 Created Dec. 6, 2016, 10:43 a.m.
Right Patch Set: Addressed comments from Patch Set 28 Created March 21, 2017, 10:04 a.m.
Left:
Right:
Use n/p to move between diff chunks; N/P to move between comments.
Jump to:
Left: Side by side diff | Download
Right: Side by side diff | Download
« no previous file with change/comment | « compiled/BlockingFilter.cpp ('k') | compiled/CSSPropertyFilter.cpp » ('j') | no next file with change/comment »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
LEFTRIGHT
1 #pragma once 1 #pragma once
2 2
3 #include <cstddef> 3 #include <cstddef>
4 4
5 #include "Filter.h" 5 #include "Filter.h"
6 #include "ElemHideBase.h" 6 #include "ElemHideBase.h"
7 7
8 struct CSSPropertyFilterData 8 struct CSSPropertyFilterData
9 { 9 {
10 String::size_type mPrefixEnd; 10 String::size_type mPrefixEnd;
(...skipping 20 matching lines...) Expand all
31 31
32 struct ElemHideData : ElemHideBaseData, CSSPropertyFilterData 32 struct ElemHideData : ElemHideBaseData, CSSPropertyFilterData
33 { 33 {
34 }; 34 };
35 35
36 class CSSPropertyFilter: public ElemHideBase 36 class CSSPropertyFilter: public ElemHideBase
37 { 37 {
38 protected: 38 protected:
39 CSSPropertyFilterData mPropertyData; 39 CSSPropertyFilterData mPropertyData;
40 public: 40 public:
41 CSSPropertyFilter(const String& text, const ElemHideData& data); 41 explicit CSSPropertyFilter(const String& text, const ElemHideData& data);
42 EMSCRIPTEN_KEEPALIVE OwnedString GetRegExpString() const; 42 EMSCRIPTEN_KEEPALIVE OwnedString GetRegExpString() const;
43 EMSCRIPTEN_KEEPALIVE const DependentString GetSelectorPrefix() const 43 EMSCRIPTEN_KEEPALIVE const DependentString GetSelectorPrefix() const
44 { 44 {
45 return mPropertyData.GetSelectorPrefix(mText, mData.mSelectorStart); 45 return mPropertyData.GetSelectorPrefix(mText, mData.mSelectorStart);
46 } 46 }
47 EMSCRIPTEN_KEEPALIVE const DependentString GetSelectorSuffix() const 47 EMSCRIPTEN_KEEPALIVE const DependentString GetSelectorSuffix() const
48 { 48 {
49 return mPropertyData.GetSelectorSuffix(mText); 49 return mPropertyData.GetSelectorSuffix(mText);
50 } 50 }
51 }; 51 };
LEFTRIGHT

Powered by Google App Engine
This is Rietveld