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