Changeset 106704 in webkit
- Timestamp:
- Feb 3, 2012 4:10:18 PM (12 years ago)
- Location:
- trunk
- Files:
-
- 2 added
- 3 edited
Legend:
- Unmodified
- Added
- Removed
-
trunk/LayoutTests/ChangeLog
r106694 r106704 1 2012-02-03 Tony Chang <tony@chromium.org> 2 3 positive and negative flex values are not being cleared on style changes 4 https://bugs.webkit.org/show_bug.cgi?id=77771 5 6 Reviewed by Ojan Vafai. 7 8 * css3/flexbox/flex-no-flex-expected.txt: Added. 9 * css3/flexbox/flex-no-flex.html: Added. 10 1 11 2012-02-03 Mihnea Ovidenie <mihnea@adobe.com> 2 12 -
trunk/Source/WebCore/ChangeLog
r106700 r106704 1 2012-02-03 Tony Chang <tony@chromium.org> 2 3 positive and negative flex values are not being cleared on style changes 4 https://bugs.webkit.org/show_bug.cgi?id=77771 5 6 Reviewed by Ojan Vafai. 7 8 If the width or height was a flex() value, but is no longer a flex 9 value, we weren't clearing the positive and negative flex values in 10 RenderStyle. 11 12 Test: css3/flexbox/flex-no-flex.html 13 14 * css/CSSStyleApplyProperty.cpp: 15 (WebCore::ApplyPropertyLength::applyValue): 16 1 17 2012-02-03 James Robinson <jamesr@chromium.org> 2 18 -
trunk/Source/WebCore/css/CSSStyleApplyProperty.cpp
r105901 r106704 350 350 static void applyValue(CSSStyleSelector* selector, CSSValue* value) 351 351 { 352 float positiveFlex = 0; 353 float negativeFlex = 0; 352 354 if (!value->isPrimitiveValue()) { 353 355 if (!flexDirection || !value->isFlexValue()) … … 357 359 value = flexValue->preferredSize(); 358 360 359 if (flexDirection == FlexWidth) { 360 selector->style()->setFlexboxWidthPositiveFlex(flexValue->positiveFlex()); 361 selector->style()->setFlexboxWidthNegativeFlex(flexValue->negativeFlex()); 362 } else if (flexDirection == FlexHeight) { 363 selector->style()->setFlexboxHeightPositiveFlex(flexValue->positiveFlex()); 364 selector->style()->setFlexboxHeightNegativeFlex(flexValue->negativeFlex()); 365 } 361 positiveFlex = flexValue->positiveFlex(); 362 negativeFlex = flexValue->negativeFlex(); 363 } 364 365 if (flexDirection == FlexWidth) { 366 selector->style()->setFlexboxWidthPositiveFlex(positiveFlex); 367 selector->style()->setFlexboxWidthNegativeFlex(negativeFlex); 368 } else if (flexDirection == FlexHeight) { 369 selector->style()->setFlexboxHeightPositiveFlex(positiveFlex); 370 selector->style()->setFlexboxHeightNegativeFlex(negativeFlex); 366 371 } 367 372
Note: See TracChangeset
for help on using the changeset viewer.