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

Unified Diff: Source/core/css/parser/CSSGrammar.y

Issue 471893002: Support consecutive slash operators for border-image value (Closed) Base URL: https://chromium.googlesource.com/chromium/blink.git@master
Patch Set: Update TC with shouldBeEqualToString Created 6 years 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 side-by-side diff with in-line comments
Download patch
Index: Source/core/css/parser/CSSGrammar.y
diff --git a/Source/core/css/parser/CSSGrammar.y b/Source/core/css/parser/CSSGrammar.y
index 4a583d96006cccc4eacd8905e582492ac0bb2d4e..bae5ec963e45570750addbac74c29a9f004a5369 100644
--- a/Source/core/css/parser/CSSGrammar.y
+++ b/Source/core/css/parser/CSSGrammar.y
@@ -349,6 +349,7 @@ inline static CSSParserValue makeIdentValue(CSSParserString string)
%type <integer> unary_operator
%type <integer> maybe_unary_operator
%type <character> operator
+%type <character> slash_operator
%type <valueList> expr
%type <value> term
@@ -1596,6 +1597,12 @@ expr:
$$ = $1;
$$->addValue(parser->sinkFloatingValue($2));
}
+ | expr slash_operator slash_operator term {
+ $$ = $1;
+ $$->addValue(makeOperatorValue($2));
+ $$->addValue(makeOperatorValue($3));
+ $$->addValue(parser->sinkFloatingValue($4));
+ }
;
expr_recovery:
@@ -1604,10 +1611,14 @@ expr_recovery:
}
;
+slash_operator:
+ '/' maybe_space {
+ $$ = '/';
+ }
+ ;
+
operator:
- '/' maybe_space {
- $$ = '/';
- }
+ slash_operator
| ',' maybe_space {
$$ = ',';
}
« no previous file with comments | « LayoutTests/fast/css/border-image-value-grammar-expected.txt ('k') | Source/core/css/parser/CSSPropertyParser.cpp » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698