Title: [205292] trunk/Source/WebCore
Revision
205292
Author
jfernan...@igalia.com
Date
2016-09-01 09:51:08 -0700 (Thu, 01 Sep 2016)

Log Message

Unreviewed build fix after r205102.

The change made in r205102 caused compilation failures on
builds with the CSS_GRID_LAYOUT compilation flag disabled.


* css/CSSComputedStyleDeclaration.cpp:
(WebCore::ComputedStyleExtractor::propertyValue):
(WebCore::resolveJustifySelfAuto): Deleted.
* css/CSSPropertyNames.in:
* css/parser/CSSParser.cpp:
(WebCore::isKeywordPropertyID):
(WebCore::CSSParser::parseValue):

Modified Paths

Diff

Modified: trunk/Source/WebCore/ChangeLog (205291 => 205292)


--- trunk/Source/WebCore/ChangeLog	2016-09-01 14:54:55 UTC (rev 205291)
+++ trunk/Source/WebCore/ChangeLog	2016-09-01 16:51:08 UTC (rev 205292)
@@ -1,3 +1,18 @@
+2016-09-01  Javier Fernandez  <jfernan...@igalia.com>
+
+        Unreviewed build fix after r205102.
+
+        The change made in r205102 caused compilation failures on
+        builds with the CSS_GRID_LAYOUT compilation flag disabled.
+
+        * css/CSSComputedStyleDeclaration.cpp:
+        (WebCore::ComputedStyleExtractor::propertyValue):
+        (WebCore::resolveJustifySelfAuto): Deleted.
+        * css/CSSPropertyNames.in:
+        * css/parser/CSSParser.cpp:
+        (WebCore::isKeywordPropertyID):
+        (WebCore::CSSParser::parseValue):
+
 2016-09-01  Fujii Hironori  <hironori.fu...@sony.com>
 
         Fix the Mac cmake clean build issue

Modified: trunk/Source/WebCore/css/CSSComputedStyleDeclaration.cpp (205291 => 205292)


--- trunk/Source/WebCore/css/CSSComputedStyleDeclaration.cpp	2016-09-01 14:54:55 UTC (rev 205291)
+++ trunk/Source/WebCore/css/CSSComputedStyleDeclaration.cpp	2016-09-01 16:51:08 UTC (rev 205292)
@@ -312,8 +312,10 @@
     CSSPropertyFlexDirection,
     CSSPropertyFlexWrap,
     CSSPropertyJustifyContent,
+#if ENABLE(CSS_GRID_LAYOUT)
     CSSPropertyJustifySelf,
     CSSPropertyJustifyItems,
+#endif
 #if ENABLE(FILTERS_LEVEL_2)
     CSSPropertyWebkitBackdropFilter,
 #endif
@@ -2291,6 +2293,7 @@
     return data;
 }
 
+#if ENABLE(CSS_GRID_LAYOUT)
 static StyleSelfAlignmentData resolveJustifyItemsAuto(const StyleSelfAlignmentData& data, Node* parent)
 {
     if (data.position() != ItemPositionAuto)
@@ -2315,6 +2318,7 @@
         return { ItemPositionNormal, OverflowAlignmentDefault };
     return resolveLegacyJustifyItems(resolveJustifyItemsAuto(parent->computedStyle()->justifyItems(), parent->parentNode()));
 }
+#endif
 
 static StyleSelfAlignmentData resolveAlignSelfAuto(const StyleSelfAlignmentData& data, Node* parent)
 {
@@ -2830,10 +2834,12 @@
             return cssValuePool.createValue(style->flexWrap());
         case CSSPropertyJustifyContent:
             return valueForContentPositionAndDistributionWithOverflowAlignment(style->justifyContent());
+#if ENABLE(CSS_GRID_LAYOUT)
         case CSSPropertyJustifyItems:
             return valueForItemPositionWithOverflowAlignment(resolveJustifyItemsAuto(style->justifyItems(), styledNode->parentNode()));
         case CSSPropertyJustifySelf:
             return valueForItemPositionWithOverflowAlignment(resolveJustifySelfAuto(style->justifySelf(), styledNode->parentNode()));
+#endif
         case CSSPropertyOrder:
             return cssValuePool.createValue(style->order(), CSSPrimitiveValue::CSS_NUMBER);
         case CSSPropertyFloat:

Modified: trunk/Source/WebCore/css/CSSPropertyNames.in (205291 => 205292)


--- trunk/Source/WebCore/css/CSSPropertyNames.in	2016-09-01 14:54:55 UTC (rev 205291)
+++ trunk/Source/WebCore/css/CSSPropertyNames.in	2016-09-01 16:51:08 UTC (rev 205292)
@@ -491,11 +491,11 @@
 #if defined(ENABLE_FILTERS_LEVEL_2) && ENABLE_FILTERS_LEVEL_2
 -webkit-backdrop-filter [ConditionalConverter=FilterOperations]
 #endif
+-webkit-font-size-delta [SkipBuilder]
+#if defined(ENABLE_CSS_GRID_LAYOUT) && ENABLE_CSS_GRID_LAYOUT
 justify-self [Initial=initialSelfAlignment, Converter=SelfOrDefaultAlignmentData]
--webkit-font-size-delta [SkipBuilder]
 justify-items [Initial=initialSelfAlignment, Converter=SelfOrDefaultAlignmentData]
 -webkit-justify-items = justify-items
-#if defined(ENABLE_CSS_GRID_LAYOUT) && ENABLE_CSS_GRID_LAYOUT
 grid [Longhands=grid-template-rows|grid-template-columns|grid-template-areas|grid-auto-flow|grid-auto-rows|grid-auto-columns|grid-column-gap|grid-row-gap]
 grid-area [Longhands=grid-row-start|grid-column-start|grid-row-end|grid-column-end]
 grid-auto-columns [Converter=GridTrackSizeList]

Modified: trunk/Source/WebCore/css/parser/CSSParser.cpp (205291 => 205292)


--- trunk/Source/WebCore/css/parser/CSSParser.cpp	2016-09-01 14:54:55 UTC (rev 205291)
+++ trunk/Source/WebCore/css/parser/CSSParser.cpp	2016-09-01 16:51:08 UTC (rev 205292)
@@ -1157,7 +1157,11 @@
         return true;
     case CSSPropertyAlignItems:
     case CSSPropertyAlignSelf:
+#if ENABLE(CSS_GRID_LAYOUT)
         return !RuntimeEnabledFeatures::sharedFeatures().isCSSGridLayoutEnabled();
+#else
+        return true;
+#endif
     default:
         return false;
     }
@@ -2737,16 +2741,18 @@
     case CSSPropertyJustifyContent:
         parsedValue = parseContentDistributionOverflowPosition();
         break;
+#if ENABLE(CSS_GRID_LAYOUT)
     case CSSPropertyJustifySelf:
-        ASSERT(RuntimeEnabledFeatures::sharedFeatures().isCSSGridLayoutEnabled());
+        if (!isCSSGridLayoutEnabled())
+            return false;
         return parseItemPositionOverflowPosition(propId, important);
     case CSSPropertyJustifyItems:
-        ASSERT(RuntimeEnabledFeatures::sharedFeatures().isCSSGridLayoutEnabled());
+        if (!isCSSGridLayoutEnabled())
+            return false;
         if (parseLegacyPosition(propId, important))
             return true;
         m_valueList->setCurrentIndex(0);
         return parseItemPositionOverflowPosition(propId, important);
-#if ENABLE(CSS_GRID_LAYOUT)
     case CSSPropertyGridAutoColumns:
     case CSSPropertyGridAutoRows:
         if (!isCSSGridLayoutEnabled())
@@ -3159,6 +3165,7 @@
     case CSSPropertyAlignContent:
         parsedValue = parseContentDistributionOverflowPosition();
         break;
+#if ENABLE(CSS_GRID_LAYOUT)
     case CSSPropertyAlignSelf:
         ASSERT(RuntimeEnabledFeatures::sharedFeatures().isCSSGridLayoutEnabled());
         return parseItemPositionOverflowPosition(propId, important);
@@ -3166,6 +3173,7 @@
     case CSSPropertyAlignItems:
         ASSERT(RuntimeEnabledFeatures::sharedFeatures().isCSSGridLayoutEnabled());
         return parseItemPositionOverflowPosition(propId, important);
+#endif
     case CSSPropertyBorderBottomStyle:
     case CSSPropertyBorderCollapse:
     case CSSPropertyBorderLeftStyle:
_______________________________________________
webkit-changes mailing list
webkit-changes@lists.webkit.org
https://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to