 Chromium Code Reviews
 Chromium Code Reviews Issue 191353002:
  [CSS Shapes] Omit polygon default fill-rule  (Closed) 
  Base URL: https://chromium.googlesource.com/chromium/blink.git@master
    
  
    Issue 191353002:
  [CSS Shapes] Omit polygon default fill-rule  (Closed) 
  Base URL: https://chromium.googlesource.com/chromium/blink.git@master| Index: Source/core/css/CSSBasicShapes.cpp | 
| diff --git a/Source/core/css/CSSBasicShapes.cpp b/Source/core/css/CSSBasicShapes.cpp | 
| index e7f03f5c31abc281179e60dc69cf76fcbe92c7c4..fb998fac776c594b31b6b3e6d5e88f782258c1f7 100644 | 
| --- a/Source/core/css/CSSBasicShapes.cpp | 
| +++ b/Source/core/css/CSSBasicShapes.cpp | 
| @@ -291,9 +291,8 @@ static String buildPolygonString(const WindRule& windRule, const Vector<String>& | 
| StringBuilder result; | 
| const char evenOddOpening[] = "polygon(evenodd, "; | 
| - const char nonZeroOpening[] = "polygon(nonzero, "; | 
| + const char nonZeroOpening[] = "polygon("; | 
| const char commaSeparator[] = ", "; | 
| - COMPILE_ASSERT(sizeof(evenOddOpening) == sizeof(nonZeroOpening), polygon_string_openings_have_same_length); | 
| 
leviw_travelin_and_unemployed
2014/03/11 23:07:13
Would it make sense to assert that the evenOddOpen
 | 
| // Compute the required capacity in advance to reduce allocations. | 
| size_t length = sizeof(evenOddOpening) - 1; |