Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(19)

Side by Side Diff: Source/core/css/parser/MediaQueryParser.cpp

Issue 217423005: Get Media Query parser to handle parens, brackets and braces blocks correctly (Closed) Base URL: https://chromium.googlesource.com/chromium/blink.git@master
Patch Set: Added more tests Created 6 years, 8 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View unified diff | Download patch
« no previous file with comments | « Source/core/css/parser/MediaQueryParser.h ('k') | Source/core/css/parser/MediaQueryToken.h » ('j') | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 // Copyright 2014 The Chromium Authors. All rights reserved. 1 // Copyright 2014 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #include "config.h" 5 #include "config.h"
6 #include "core/css/parser/MediaQueryParser.h" 6 #include "core/css/parser/MediaQueryParser.h"
7 7
8 #include "MediaTypeNames.h" 8 #include "MediaTypeNames.h"
9 #include "core/css/parser/CSSPropertyParser.h" 9 #include "core/css/parser/CSSPropertyParser.h"
10 #include "core/css/parser/MediaQueryTokenizer.h" 10 #include "core/css/parser/MediaQueryTokenizer.h"
11 11
12 namespace WebCore { 12 namespace WebCore {
13 13
14 PassRefPtrWillBeRawPtr<MediaQuerySet> MediaQueryParser::parse(const String& quer yString) 14 PassRefPtrWillBeRawPtr<MediaQuerySet> MediaQueryParser::parse(const String& quer yString)
15 { 15 {
16 return MediaQueryParser(queryString).parseImpl(); 16 return MediaQueryParser(queryString).parseImpl();
17 } 17 }
18 18
19 const MediaQueryParser::State MediaQueryParser::ReadRestrictor = &MediaQueryPars er::readRestrictor; 19 const MediaQueryParser::State MediaQueryParser::ReadRestrictor = &MediaQueryPars er::readRestrictor;
20 const MediaQueryParser::State MediaQueryParser::ReadMediaType = &MediaQueryParse r::readMediaType; 20 const MediaQueryParser::State MediaQueryParser::ReadMediaType = &MediaQueryParse r::readMediaType;
21 const MediaQueryParser::State MediaQueryParser::ReadAnd = &MediaQueryParser::rea dAnd; 21 const MediaQueryParser::State MediaQueryParser::ReadAnd = &MediaQueryParser::rea dAnd;
22 const MediaQueryParser::State MediaQueryParser::ReadFeatureStart = &MediaQueryPa rser::readFeatureStart; 22 const MediaQueryParser::State MediaQueryParser::ReadFeatureStart = &MediaQueryPa rser::readFeatureStart;
23 const MediaQueryParser::State MediaQueryParser::ReadFeature = &MediaQueryParser: :readFeature; 23 const MediaQueryParser::State MediaQueryParser::ReadFeature = &MediaQueryParser: :readFeature;
24 const MediaQueryParser::State MediaQueryParser::ReadFeatureColon = &MediaQueryPa rser::readFeatureColon; 24 const MediaQueryParser::State MediaQueryParser::ReadFeatureColon = &MediaQueryPa rser::readFeatureColon;
25 const MediaQueryParser::State MediaQueryParser::ReadFeatureValue = &MediaQueryPa rser::readFeatureValue; 25 const MediaQueryParser::State MediaQueryParser::ReadFeatureValue = &MediaQueryPa rser::readFeatureValue;
26 const MediaQueryParser::State MediaQueryParser::ReadFeatureEnd = &MediaQueryPars er::readFeatureEnd; 26 const MediaQueryParser::State MediaQueryParser::ReadFeatureEnd = &MediaQueryPars er::readFeatureEnd;
27 const MediaQueryParser::State MediaQueryParser::SkipUntilComma = &MediaQueryPars er::skipUntilComma; 27 const MediaQueryParser::State MediaQueryParser::SkipUntilComma = &MediaQueryPars er::skipUntilComma;
28 const MediaQueryParser::State MediaQueryParser::SkipUntilParenthesis = &MediaQue ryParser::skipUntilParenthesis; 28 const MediaQueryParser::State MediaQueryParser::SkipUntilBlockEnd = &MediaQueryP arser::skipUntilBlockEnd;
29 const MediaQueryParser::State MediaQueryParser::Done = &MediaQueryParser::done; 29 const MediaQueryParser::State MediaQueryParser::Done = &MediaQueryParser::done;
30 30
31 // FIXME: Replace the MediaQueryTokenizer with a generic CSSTokenizer, once ther e is one, 31 // FIXME: Replace the MediaQueryTokenizer with a generic CSSTokenizer, once ther e is one,
32 // or better yet, replace the MediaQueryParser with a generic thread-safe CSS pa rser. 32 // or better yet, replace the MediaQueryParser with a generic thread-safe CSS pa rser.
33 MediaQueryParser::MediaQueryParser(const String& queryString) 33 MediaQueryParser::MediaQueryParser(const String& queryString)
34 : m_state(&MediaQueryParser::readRestrictor) 34 : m_state(&MediaQueryParser::readRestrictor)
35 , m_querySet(MediaQuerySet::create()) 35 , m_querySet(MediaQuerySet::create())
36 { 36 {
37 MediaQueryTokenizer::tokenize(queryString, m_tokens); 37 MediaQueryTokenizer::tokenize(queryString, m_tokens);
38 } 38 }
(...skipping 65 matching lines...) Expand 10 before | Expand all | Expand 10 after
104 } 104 }
105 105
106 void MediaQueryParser::readFeatureColon(MediaQueryTokenType type, TokenIterator& token) 106 void MediaQueryParser::readFeatureColon(MediaQueryTokenType type, TokenIterator& token)
107 { 107 {
108 if (type == ColonToken) { 108 if (type == ColonToken) {
109 m_state = ReadFeatureValue; 109 m_state = ReadFeatureValue;
110 } else if (type == RightParenthesisToken || type == EOFToken) { 110 } else if (type == RightParenthesisToken || type == EOFToken) {
111 --token; 111 --token;
112 m_state = ReadFeatureEnd; 112 m_state = ReadFeatureEnd;
113 } else { 113 } else {
114 m_state = SkipUntilParenthesis; 114 m_state = SkipUntilBlockEnd;
115 } 115 }
116 } 116 }
117 117
118 void MediaQueryParser::readFeatureValue(MediaQueryTokenType type, TokenIterator& token) 118 void MediaQueryParser::readFeatureValue(MediaQueryTokenType type, TokenIterator& token)
119 { 119 {
120 if (type == DimensionToken && token->unitType() == CSSPrimitiveValue::CSS_UN KNOWN) { 120 if (type == DimensionToken && token->unitType() == CSSPrimitiveValue::CSS_UN KNOWN) {
121 m_state = SkipUntilComma; 121 m_state = SkipUntilComma;
122 } else { 122 } else {
123 m_mediaQueryData.addParserValue(type, *token); 123 m_mediaQueryData.addParserValue(type, *token);
124 m_state = ReadFeatureEnd; 124 m_state = ReadFeatureEnd;
125 } 125 }
126 } 126 }
127 127
128 void MediaQueryParser::readFeatureEnd(MediaQueryTokenType type, TokenIterator& t oken) 128 void MediaQueryParser::readFeatureEnd(MediaQueryTokenType type, TokenIterator& t oken)
129 { 129 {
130 if (type == RightParenthesisToken || type == EOFToken) { 130 if (type == RightParenthesisToken || type == EOFToken) {
131 if (m_mediaQueryData.addExpression()) 131 if (m_mediaQueryData.addExpression())
132 m_state = ReadAnd; 132 m_state = ReadAnd;
133 else 133 else
134 m_state = SkipUntilComma; 134 m_state = SkipUntilComma;
135 } else if (type == DelimiterToken && token->delimiter() == '/') { 135 } else if (type == DelimiterToken && token->delimiter() == '/') {
136 m_mediaQueryData.addParserValue(type, *token); 136 m_mediaQueryData.addParserValue(type, *token);
137 m_state = ReadFeatureValue; 137 m_state = ReadFeatureValue;
138 } else { 138 } else {
139 m_state = SkipUntilParenthesis; 139 m_state = SkipUntilBlockEnd;
140 } 140 }
141 } 141 }
142 142
143 void MediaQueryParser::skipUntilComma(MediaQueryTokenType type, TokenIterator& t oken) 143 void MediaQueryParser::skipUntilComma(MediaQueryTokenType type, TokenIterator& t oken)
144 { 144 {
145 if (type == CommaToken || type == EOFToken) { 145 if (type == CommaToken || type == EOFToken) {
146 m_state = ReadRestrictor; 146 m_state = ReadRestrictor;
147 m_mediaQueryData.clear(); 147 m_mediaQueryData.clear();
148 m_querySet->addMediaQuery(MediaQuery::createNotAll()); 148 m_querySet->addMediaQuery(MediaQuery::createNotAll());
149 } 149 }
150 } 150 }
151 151
152 void MediaQueryParser::skipUntilParenthesis(MediaQueryTokenType type, TokenItera tor& token) 152 void MediaQueryParser::skipUntilBlockEnd(MediaQueryTokenType type, TokenIterator & token)
153 { 153 {
154 if (type == RightParenthesisToken) 154 if (m_blockStack.isEmpty())
155 m_state = SkipUntilComma; 155 m_state = SkipUntilComma;
156 } 156 }
157 157
158 void MediaQueryParser::done(MediaQueryTokenType type, TokenIterator& token) { } 158 void MediaQueryParser::done(MediaQueryTokenType type, TokenIterator& token) { }
159 159
160 void MediaQueryParser::popIfBlockMatches(Vector<BlockType>& blockStack, BlockTyp e type)
161 {
162 if (!blockStack.isEmpty() && blockStack.last() == type)
163 blockStack.removeLast();
164 }
165
166 bool MediaQueryParser::observeBlock(BlockParameters& parameters, MediaQueryToken Type type)
167 {
168 if (type == parameters.leftToken) {
169 if (parameters.stateChange == ModifyState)
170 m_state = SkipUntilBlockEnd;
171 m_blockStack.append(parameters.blockType);
172 } else if (type == parameters.rightToken) {
173 popIfBlockMatches(m_blockStack, parameters.blockType);
174 } else {
175 return false;
176 }
177 return true;
178 }
179
180 void MediaQueryParser::observeBlocks(MediaQueryTokenType type)
181 {
182 const unsigned blockParametersNumber = 3;
183 BlockParameters blockParameterSet[blockParametersNumber] = {
184 { LeftParenthesisToken, RightParenthesisToken, ParenthesisBlock, DoNotMo difyState },
185 { LeftBracketToken, RightBracketToken, BracketsBlock, ModifyState },
186 { LeftBraceToken, RightBraceToken, BracesBlock, ModifyState }
187 };
188
189 for (unsigned i = 0; i < blockParametersNumber; ++i) {
190 if (observeBlock(blockParameterSet[i], type))
191 break;
192 }
193 }
194
160 void MediaQueryParser::processToken(TokenIterator& token) 195 void MediaQueryParser::processToken(TokenIterator& token)
161 { 196 {
162 MediaQueryTokenType type = token->type(); 197 MediaQueryTokenType type = token->type();
163 198
199 observeBlocks(type);
200
164 // Call the function that handles current state 201 // Call the function that handles current state
165 if (type != WhitespaceToken && type != CommentToken) 202 if (type != WhitespaceToken && type != CommentToken)
166 ((this)->*(m_state))(type, token); 203 ((this)->*(m_state))(type, token);
167 } 204 }
168 205
169 // The state machine loop 206 // The state machine loop
170 PassRefPtrWillBeRawPtr<MediaQuerySet> MediaQueryParser::parseImpl() 207 PassRefPtrWillBeRawPtr<MediaQuerySet> MediaQueryParser::parseImpl()
171 { 208 {
172 for (Vector<MediaQueryToken>::iterator token = m_tokens.begin(); token != m_ tokens.end(); ++token) 209 for (Vector<MediaQueryToken>::iterator token = m_tokens.begin(); token != m_ tokens.end(); ++token)
173 processToken(token); 210 processToken(token);
(...skipping 60 matching lines...) Expand 10 before | Expand all | Expand 10 after
234 m_valueList.addValue(value); 271 m_valueList.addValue(value);
235 } 272 }
236 273
237 void MediaQueryData::setMediaType(const String& mediaType) 274 void MediaQueryData::setMediaType(const String& mediaType)
238 { 275 {
239 m_mediaType = mediaType; 276 m_mediaType = mediaType;
240 m_mediaTypeSet = true; 277 m_mediaTypeSet = true;
241 } 278 }
242 279
243 } // namespace WebCore 280 } // namespace WebCore
OLDNEW
« no previous file with comments | « Source/core/css/parser/MediaQueryParser.h ('k') | Source/core/css/parser/MediaQueryToken.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698