Unreviewed, rolling out r104972.
authorojan@chromium.org <ojan@chromium.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Fri, 13 Jan 2012 22:00:29 +0000 (22:00 +0000)
committerojan@chromium.org <ojan@chromium.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Fri, 13 Jan 2012 22:00:29 +0000 (22:00 +0000)
http://trac.webkit.org/changeset/104972
https://bugs.webkit.org/show_bug.cgi?id=75782

Broke some tests

Source/WebCore:

* css/CSSComputedStyleDeclaration.cpp:
(WebCore::CSSComputedStyleDeclaration::getPropertyCSSValue):
* css/CSSParser.cpp:
(WebCore::CSSParser::parseValue):
* css/CSSProperty.cpp:
(WebCore::CSSProperty::isInheritedProperty):
* css/CSSPropertyNames.in:
* css/CSSStyleApplyProperty.cpp:
(WebCore::CSSStyleApplyProperty::CSSStyleApplyProperty):
* css/CSSStyleSelector.cpp:
(WebCore::CSSStyleSelector::applyProperty):
* rendering/RenderBox.cpp:
(WebCore::RenderBox::sizesToIntrinsicLogicalWidth):
* rendering/RenderFlexibleBox.cpp:
(WebCore::RenderFlexibleBox::layoutAndPlaceChildren):
(WebCore::RenderFlexibleBox::alignChildren):
* rendering/style/RenderStyle.h:
(WebCore::RenderStyleBitfields::initialFlexItemAlign):
* rendering/style/StyleFlexibleBoxData.cpp:
(WebCore::StyleFlexibleBoxData::StyleFlexibleBoxData):
(WebCore::StyleFlexibleBoxData::operator==):
* rendering/style/StyleFlexibleBoxData.h:

LayoutTests:

* css3/flexbox/css-properties-expected.txt:
* css3/flexbox/flex-align-expected.txt:
* css3/flexbox/flex-align.html:
* css3/flexbox/script-tests/css-properties.js:
* fast/css/getComputedStyle/computed-style-expected.txt:
* platform/chromium-mac-snowleopard/svg/css/getComputedStyle-basic-expected.txt:
* platform/chromium-win/fast/css/getComputedStyle/computed-style-expected.txt:
* platform/chromium-win/fast/css/getComputedStyle/computed-style-without-renderer-expected.txt:
* platform/chromium-win/svg/css/getComputedStyle-basic-expected.txt:
* platform/gtk/fast/css/getComputedStyle/computed-style-without-renderer-expected.txt:
* platform/mac/fast/css/getComputedStyle/computed-style-without-renderer-expected.txt:
* platform/qt/fast/css/getComputedStyle/computed-style-without-renderer-expected.txt:
* svg/css/getComputedStyle-basic-expected.txt:

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@104985 268f45cc-cd09-0410-ab3c-d52691b4dbfc

26 files changed:
LayoutTests/ChangeLog
LayoutTests/css3/flexbox/css-properties-expected.txt
LayoutTests/css3/flexbox/flex-align-expected.txt
LayoutTests/css3/flexbox/flex-align.html
LayoutTests/css3/flexbox/script-tests/css-properties.js
LayoutTests/fast/css/getComputedStyle/computed-style-expected.txt
LayoutTests/platform/chromium-mac-snowleopard/svg/css/getComputedStyle-basic-expected.txt
LayoutTests/platform/chromium-win/fast/css/getComputedStyle/computed-style-expected.txt
LayoutTests/platform/chromium-win/fast/css/getComputedStyle/computed-style-without-renderer-expected.txt
LayoutTests/platform/chromium-win/svg/css/getComputedStyle-basic-expected.txt
LayoutTests/platform/gtk/fast/css/getComputedStyle/computed-style-without-renderer-expected.txt
LayoutTests/platform/mac/fast/css/getComputedStyle/computed-style-without-renderer-expected.txt
LayoutTests/platform/qt/fast/css/getComputedStyle/computed-style-without-renderer-expected.txt
LayoutTests/svg/css/getComputedStyle-basic-expected.txt
Source/WebCore/ChangeLog
Source/WebCore/css/CSSComputedStyleDeclaration.cpp
Source/WebCore/css/CSSParser.cpp
Source/WebCore/css/CSSProperty.cpp
Source/WebCore/css/CSSPropertyNames.in
Source/WebCore/css/CSSStyleApplyProperty.cpp
Source/WebCore/css/CSSStyleSelector.cpp
Source/WebCore/rendering/RenderBox.cpp
Source/WebCore/rendering/RenderFlexibleBox.cpp
Source/WebCore/rendering/style/RenderStyle.h
Source/WebCore/rendering/style/StyleFlexibleBoxData.cpp
Source/WebCore/rendering/style/StyleFlexibleBoxData.h

index 1037d0a..241db14 100644 (file)
@@ -1,3 +1,25 @@
+2012-01-13  Ojan Vafai  <ojan@chromium.org>
+
+        Unreviewed, rolling out r104972.
+        http://trac.webkit.org/changeset/104972
+        https://bugs.webkit.org/show_bug.cgi?id=75782
+
+        Broke some tests
+
+        * css3/flexbox/css-properties-expected.txt:
+        * css3/flexbox/flex-align-expected.txt:
+        * css3/flexbox/flex-align.html:
+        * css3/flexbox/script-tests/css-properties.js:
+        * fast/css/getComputedStyle/computed-style-expected.txt:
+        * platform/chromium-mac-snowleopard/svg/css/getComputedStyle-basic-expected.txt:
+        * platform/chromium-win/fast/css/getComputedStyle/computed-style-expected.txt:
+        * platform/chromium-win/fast/css/getComputedStyle/computed-style-without-renderer-expected.txt:
+        * platform/chromium-win/svg/css/getComputedStyle-basic-expected.txt:
+        * platform/gtk/fast/css/getComputedStyle/computed-style-without-renderer-expected.txt:
+        * platform/mac/fast/css/getComputedStyle/computed-style-without-renderer-expected.txt:
+        * platform/qt/fast/css/getComputedStyle/computed-style-without-renderer-expected.txt:
+        * svg/css/getComputedStyle-basic-expected.txt:
+
 2012-01-13  Simon Fraser  <simon.fraser@apple.com>
 
         Add new baselines for various compositing tests which didn't have Mac results.
index b71813d..069dec0 100644 (file)
@@ -31,8 +31,6 @@ PASS window.getComputedStyle(flexbox, null).webkitFlexPack is "start"
 PASS flexbox.style.webkitFlexItemAlign is ""
 PASS window.getComputedStyle(flexbox, null).webkitFlexItemAlign is "stretch"
 PASS flexbox.style.webkitFlexItemAlign is ""
-PASS flexbox.style.webkitFlexItemAlign is "auto"
-PASS window.getComputedStyle(flexbox, null).webkitFlexItemAlign is "stretch"
 PASS flexbox.style.webkitFlexItemAlign is "start"
 PASS window.getComputedStyle(flexbox, null).webkitFlexItemAlign is "start"
 PASS flexbox.style.webkitFlexItemAlign is "end"
@@ -45,36 +43,6 @@ PASS flexbox.style.webkitFlexItemAlign is "baseline"
 PASS window.getComputedStyle(flexbox, null).webkitFlexItemAlign is "baseline"
 PASS flexbox.style.webkitFlexItemAlign is ""
 PASS window.getComputedStyle(flexbox, null).webkitFlexItemAlign is "stretch"
-PASS flexbox.style.webkitFlexAlign is ""
-PASS window.getComputedStyle(flexbox, null).webkitFlexAlign is "stretch"
-PASS window.getComputedStyle(flexitem, null).webkitFlexItemAlign is "stretch"
-PASS flexbox.style.webkitFlexAlign is ""
-PASS flexbox.style.webkitFlexAlign is ""
-PASS window.getComputedStyle(flexbox, null).webkitFlexAlign is "stretch"
-PASS window.getComputedStyle(flexitem, null).webkitFlexItemAlign is "stretch"
-PASS flexbox.style.webkitFlexAlign is "start"
-PASS window.getComputedStyle(flexbox, null).webkitFlexAlign is "start"
-PASS window.getComputedStyle(flexitem, null).webkitFlexItemAlign is "start"
-PASS flexbox.style.webkitFlexAlign is "end"
-PASS window.getComputedStyle(flexbox, null).webkitFlexAlign is "end"
-PASS window.getComputedStyle(flexitem, null).webkitFlexItemAlign is "end"
-PASS flexbox.style.webkitFlexAlign is "center"
-PASS window.getComputedStyle(flexbox, null).webkitFlexAlign is "center"
-PASS window.getComputedStyle(flexitem, null).webkitFlexItemAlign is "center"
-PASS flexbox.style.webkitFlexAlign is "stretch"
-PASS window.getComputedStyle(flexbox, null).webkitFlexAlign is "stretch"
-PASS window.getComputedStyle(flexitem, null).webkitFlexItemAlign is "stretch"
-PASS flexbox.style.webkitFlexAlign is "baseline"
-PASS window.getComputedStyle(flexbox, null).webkitFlexAlign is "baseline"
-PASS window.getComputedStyle(flexitem, null).webkitFlexItemAlign is "baseline"
-PASS flexbox.style.webkitFlexAlign is ""
-PASS window.getComputedStyle(flexbox, null).webkitFlexAlign is "stretch"
-PASS window.getComputedStyle(flexitem, null).webkitFlexItemAlign is "stretch"
-PASS flexbox.style.webkitFlexAlign is ""
-PASS window.getComputedStyle(flexbox, null).webkitFlexAlign is "stretch"
-PASS window.getComputedStyle(flexitem, null).webkitFlexItemAlign is "stretch"
-PASS window.getComputedStyle(detachedFlexbox, null).webkitFlexItemAlign is ""
-PASS window.getComputedStyle(detachedFlexItem, null).webkitFlexItemAlign is ""
 PASS flexbox.style.webkitFlexDirection is ""
 PASS window.getComputedStyle(flexbox, null).webkitFlexDirection is "row"
 PASS flexbox.style.webkitFlexDirection is ""
index 8ead7bc..5fc9496 100644 (file)
@@ -9,16 +9,6 @@ PASS
 PASS
 PASS
 PASS
-PASS
-PASS
-PASS
-PASS
-PASS
-PASS
-PASS
-PASS
-PASS
-PASS
 ahem
 
 
index 7a494ca..2c67d2a 100644 (file)
@@ -10,7 +10,7 @@ body {
     background-color: #aaa;
     position: relative;
 }
-.flexbox div {
+.flexbox div {
     border: 0;
 }
 
@@ -67,86 +67,6 @@ if (window.layoutTestController)
   </div>
 </div>
 
-<div class="flexbox" style="-webkit-flex-align: start">
-    <div data-expected-height="0" data-offset-y="0" style="width: -webkit-flex(1 0 0);"></div>
-    <div data-expected-height="100" data-offset-y="0" style="width: -webkit-flex(1 0 0); height: 100px;"></div>
-</div>
-
-<div class="flexbox" style="-webkit-flex-align: end">
-    <div data-expected-height="0" data-offset-y="100" style="width: -webkit-flex(1 0 0);"></div>
-    <div data-expected-height="100" data-offset-y="0" style="width: -webkit-flex(1 0 0); height: 100px;"></div>
-</div>
-
-<div class="flexbox" style="-webkit-flex-align: center">
-    <div data-expected-height="0" data-offset-y="50" style="width: -webkit-flex(1 0 0);"></div>
-    <div data-expected-height="100" data-offset-y="0" style="width: -webkit-flex(1 0 0); height: 100px;"></div>
-</div>
-
-<div class="flexbox" style="-webkit-flex-align: baseline">
-    <div data-expected-height="10" data-offset-y="90" style="width: -webkit-flex(1 0 0); line-height: 10px;"><div style="display:inline-block;"></div></div>
-    <div data-expected-height="100" data-offset-y="0" style="width: -webkit-flex(1 0 0); height: 100px;"></div>
-</div>
-
-<div class="flexbox" style="-webkit-flex-align: stretch">
-    <div data-expected-height="100" data-offset-y="0" style="width: -webkit-flex(1 0 0);"></div>
-    <div data-expected-height="100" data-offset-y="0" style="width: -webkit-flex(1 0 0); height: 100px;"></div>
-</div>
-
-<div class="flexbox" style="-webkit-flex-align: start">
-    <div data-expected-height="100" data-offset-y="0" style="-webkit-flex-item-align: auto; width: -webkit-flex(1 0 0); height: 100px;"></div>
-    <div data-expected-height="0" data-offset-y="0" style="-webkit-flex-item-align: auto; width: -webkit-flex(1 0 0);"></div>
-    <div data-expected-height="0" data-offset-y="0" style="-webkit-flex-item-align: start; width: -webkit-flex(1 0 0);"></div>
-    <div data-expected-height="0" data-offset-y="100" style="-webkit-flex-item-align: end; width: -webkit-flex(1 0 0);"></div>
-    <div data-expected-height="0" data-offset-y="50" style="-webkit-flex-item-align: center; width: -webkit-flex(1 0 0);"></div>
-    <div data-expected-height="10" data-offset-y="20" style="-webkit-flex-item-align: baseline; width: -webkit-flex(1 0 0); line-height: 10px;"><div style="display:inline-block;"></div></div>
-    <div data-expected-height="10" data-offset-y="20" style="-webkit-flex-item-align: baseline; width: -webkit-flex(1 0 0); margin: 20px; line-height: 10px;"><div style="display:inline-block;"></div></div>
-    <div data-expected-height="100" data-offset-y="0" style="-webkit-flex-item-align: stretch; width: -webkit-flex(1 0 0);"></div>
-</div>
-
-<div class="flexbox" style="-webkit-flex-align: end">
-    <div data-expected-height="100" data-offset-y="0" style="-webkit-flex-item-align: auto; width: -webkit-flex(1 0 0); height: 100px;"></div>
-    <div data-expected-height="0" data-offset-y="100" style="-webkit-flex-item-align: auto; width: -webkit-flex(1 0 0);"></div>
-    <div data-expected-height="0" data-offset-y="0" style="-webkit-flex-item-align: start; width: -webkit-flex(1 0 0);"></div>
-    <div data-expected-height="0" data-offset-y="100" style="-webkit-flex-item-align: end; width: -webkit-flex(1 0 0);"></div>
-    <div data-expected-height="0" data-offset-y="50" style="-webkit-flex-item-align: center; width: -webkit-flex(1 0 0);"></div>
-    <div data-expected-height="10" data-offset-y="20" style="-webkit-flex-item-align: baseline; width: -webkit-flex(1 0 0); line-height: 10px;"><div style="display:inline-block;"></div></div>
-    <div data-expected-height="10" data-offset-y="20" style="-webkit-flex-item-align: baseline; width: -webkit-flex(1 0 0); margin: 20px; line-height: 10px;"><div style="display:inline-block;"></div></div>
-    <div data-expected-height="100" data-offset-y="0" style="-webkit-flex-item-align: stretch; width: -webkit-flex(1 0 0);"></div>
-</div>
-
-<div class="flexbox" style="-webkit-flex-align: center">
-    <div data-expected-height="100" data-offset-y="0" style="-webkit-flex-item-align: auto; width: -webkit-flex(1 0 0); height: 100px;"></div>
-    <div data-expected-height="0" data-offset-y="50" style="-webkit-flex-item-align: auto; width: -webkit-flex(1 0 0);"></div>
-    <div data-expected-height="0" data-offset-y="0" style="-webkit-flex-item-align: start; width: -webkit-flex(1 0 0);"></div>
-    <div data-expected-height="0" data-offset-y="100" style="-webkit-flex-item-align: end; width: -webkit-flex(1 0 0);"></div>
-    <div data-expected-height="0" data-offset-y="50" style="-webkit-flex-item-align: center; width: -webkit-flex(1 0 0);"></div>
-    <div data-expected-height="10" data-offset-y="20" style="-webkit-flex-item-align: baseline; width: -webkit-flex(1 0 0); line-height: 10px;"><div style="display:inline-block;"></div></div>
-    <div data-expected-height="10" data-offset-y="20" style="-webkit-flex-item-align: baseline; width: -webkit-flex(1 0 0); margin: 20px; line-height: 10px;"><div style="display:inline-block;"></div></div>
-    <div data-expected-height="100" data-offset-y="0" style="-webkit-flex-item-align: stretch; width: -webkit-flex(1 0 0);"></div>
-</div>
-
-<div class="flexbox" style="-webkit-flex-align: baseline">
-    <div data-expected-height="100" data-offset-y="0" style="-webkit-flex-item-align: start; width: -webkit-flex(1 0 0); height: 100px;"></div>
-    <div data-expected-height="10" data-offset-y="20" style="-webkit-flex-item-align: auto; width: -webkit-flex(1 0 0); line-height: 10px;"><div style="display:inline-block;"></div></div>
-    <div data-expected-height="0" data-offset-y="0" style="-webkit-flex-item-align: start; width: -webkit-flex(1 0 0);"></div>
-    <div data-expected-height="0" data-offset-y="100" style="-webkit-flex-item-align: end; width: -webkit-flex(1 0 0);"></div>
-    <div data-expected-height="0" data-offset-y="50" style="-webkit-flex-item-align: center; width: -webkit-flex(1 0 0);"></div>
-    <div data-expected-height="10" data-offset-y="20" style="-webkit-flex-item-align: baseline; width: -webkit-flex(1 0 0); line-height: 10px;"><div style="display:inline-block;"></div></div>
-    <div data-expected-height="10" data-offset-y="20" style="-webkit-flex-item-align: baseline; width: -webkit-flex(1 0 0); margin: 20px; line-height: 10px;"><div style="display:inline-block;"></div></div>
-    <div data-expected-height="100" data-offset-y="0" style="-webkit-flex-item-align: stretch; width: -webkit-flex(1 0 0);"></div>
-</div>
-
-<div class="flexbox" style="-webkit-flex-align: stretch">
-    <div data-expected-height="100" data-offset-y="0" style="-webkit-flex-item-align: auto; width: -webkit-flex(1 0 0); height: 100px;"></div>
-    <div data-expected-height="100" data-offset-y="0" style="-webkit-flex-item-align: auto; width: -webkit-flex(1 0 0);"></div>
-    <div data-expected-height="0" data-offset-y="0" style="-webkit-flex-item-align: start; width: -webkit-flex(1 0 0);"></div>
-    <div data-expected-height="0" data-offset-y="100" style="-webkit-flex-item-align: end; width: -webkit-flex(1 0 0);"></div>
-    <div data-expected-height="0" data-offset-y="50" style="-webkit-flex-item-align: center; width: -webkit-flex(1 0 0);"></div>
-    <div data-expected-height="10" data-offset-y="20" style="-webkit-flex-item-align: baseline; width: -webkit-flex(1 0 0); line-height: 10px;"><div style="display:inline-block;"></div></div>
-    <div data-expected-height="10" data-offset-y="20" style="-webkit-flex-item-align: baseline; width: -webkit-flex(1 0 0); margin: 20px; line-height: 10px;"><div style="display:inline-block;"></div></div>
-    <div data-expected-height="100" data-offset-y="0" style="-webkit-flex-item-align: stretch; width: -webkit-flex(1 0 0);"></div>
-</div>
-
 <div class="flexbox" style="height: 100px;">
   <div data-expected-height="50"  data-offset-y="0" style="width: -webkit-flex(1 0 0); height: 50px;"></div>
   <div data-expected-height="100" data-offset-y="0" style="width: -webkit-flex(1 0 0);"></div>
@@ -216,7 +136,6 @@ if (window.layoutTestController)
 
 <div class="flexbox">
   <div data-expected-height="20" data-offset-y="20" style="width: -webkit-flex(1 0 0); -webkit-flex-item-align: baseline; height: 20px;"></div>
-  <!-- FIXME: This looks like it might be positioned 10 pixels too high due the margin-bottom. All the boxes should have their bottoms aligned I think. -->
   <div data-expected-height="10" data-offset-y="20" style="width: -webkit-flex(1 0 0); -webkit-flex-item-align: baseline; height: 10px; margin: 20px 0 10px;"></div>
   <div data-expected-height="10" data-offset-y="30" style="width: -webkit-flex(1 0 0); -webkit-flex-item-align: baseline; height: 10px; margin: 20px 0 0;"></div>
   <div data-expected-height="100" data-offset-y="0" style="width: -webkit-flex(1 0 0); height: 100px;"></div>
index b1e7914..d427af9 100644 (file)
@@ -76,10 +76,6 @@ shouldBeEqualToString('window.getComputedStyle(flexbox, null).webkitFlexItemAlig
 flexbox.style.webkitFlexItemAlign = 'foo';
 shouldBeEqualToString('flexbox.style.webkitFlexItemAlign', '');
 
-flexbox.style.webkitFlexItemAlign = 'auto';
-shouldBeEqualToString('flexbox.style.webkitFlexItemAlign', 'auto');
-shouldBeEqualToString('window.getComputedStyle(flexbox, null).webkitFlexItemAlign', 'stretch');
-
 flexbox.style.webkitFlexItemAlign = 'start';
 shouldBeEqualToString('flexbox.style.webkitFlexItemAlign', 'start');
 shouldBeEqualToString('window.getComputedStyle(flexbox, null).webkitFlexItemAlign', 'start');
@@ -104,64 +100,6 @@ flexbox.style.webkitFlexItemAlign = '';
 shouldBeEqualToString('flexbox.style.webkitFlexItemAlign', '');
 shouldBeEqualToString('window.getComputedStyle(flexbox, null).webkitFlexItemAlign', 'stretch');
 
-shouldBeEqualToString('flexbox.style.webkitFlexAlign', '');
-// The initial value is 'stretch'.
-shouldBeEqualToString('window.getComputedStyle(flexbox, null).webkitFlexAlign', 'stretch');
-shouldBeEqualToString('window.getComputedStyle(flexitem, null).webkitFlexItemAlign', 'stretch');
-
-flexbox.style.webkitFlexAlign = 'foo';
-shouldBeEqualToString('flexbox.style.webkitFlexAlign', '');
-
-flexbox.style.webkitFlexAlign = 'auto';
-shouldBeEqualToString('flexbox.style.webkitFlexAlign', '');
-shouldBeEqualToString('window.getComputedStyle(flexbox, null).webkitFlexAlign', 'stretch');
-shouldBeEqualToString('window.getComputedStyle(flexitem, null).webkitFlexItemAlign', 'stretch');
-
-flexbox.style.webkitFlexAlign = 'start';
-shouldBeEqualToString('flexbox.style.webkitFlexAlign', 'start');
-shouldBeEqualToString('window.getComputedStyle(flexbox, null).webkitFlexAlign', 'start');
-shouldBeEqualToString('window.getComputedStyle(flexitem, null).webkitFlexItemAlign', 'start');
-
-flexbox.style.webkitFlexAlign = 'end';
-shouldBeEqualToString('flexbox.style.webkitFlexAlign', 'end');
-shouldBeEqualToString('window.getComputedStyle(flexbox, null).webkitFlexAlign', 'end');
-shouldBeEqualToString('window.getComputedStyle(flexitem, null).webkitFlexItemAlign', 'end');
-
-flexbox.style.webkitFlexAlign = 'center';
-shouldBeEqualToString('flexbox.style.webkitFlexAlign', 'center');
-shouldBeEqualToString('window.getComputedStyle(flexbox, null).webkitFlexAlign', 'center');
-shouldBeEqualToString('window.getComputedStyle(flexitem, null).webkitFlexItemAlign', 'center');
-
-flexbox.style.webkitFlexAlign = 'stretch';
-shouldBeEqualToString('flexbox.style.webkitFlexAlign', 'stretch');
-shouldBeEqualToString('window.getComputedStyle(flexbox, null).webkitFlexAlign', 'stretch');
-shouldBeEqualToString('window.getComputedStyle(flexitem, null).webkitFlexItemAlign', 'stretch');
-
-flexbox.style.webkitFlexAlign = 'baseline';
-shouldBeEqualToString('flexbox.style.webkitFlexAlign', 'baseline');
-shouldBeEqualToString('window.getComputedStyle(flexbox, null).webkitFlexAlign', 'baseline');
-shouldBeEqualToString('window.getComputedStyle(flexitem, null).webkitFlexItemAlign', 'baseline');
-
-flexbox.style.webkitFlexAlign = '';
-shouldBeEqualToString('flexbox.style.webkitFlexAlign', '');
-shouldBeEqualToString('window.getComputedStyle(flexbox, null).webkitFlexAlign', 'stretch');
-shouldBeEqualToString('window.getComputedStyle(flexitem, null).webkitFlexItemAlign', 'stretch');
-
-flexbox.style.display = 'none';
-shouldBeEqualToString('flexbox.style.webkitFlexAlign', '');
-shouldBeEqualToString('window.getComputedStyle(flexbox, null).webkitFlexAlign', 'stretch');
-shouldBeEqualToString('window.getComputedStyle(flexitem, null).webkitFlexItemAlign', 'stretch');
-flexbox.style.display = '';
-
-
-// FIXME: This should probably return stretch. See https://bugs.webkit.org/show_bug.cgi?id=14563.
-var detachedFlexbox = document.createElement('div');
-var detachedFlexItem = document.createElement('div');
-detachedFlexbox.appendChild(detachedFlexItem);
-shouldBeEqualToString('window.getComputedStyle(detachedFlexbox, null).webkitFlexItemAlign', '');
-shouldBeEqualToString('window.getComputedStyle(detachedFlexItem, null).webkitFlexItemAlign', '');
-
-
 shouldBeEqualToString('flexbox.style.webkitFlexDirection', '');
 shouldBeEqualToString('window.getComputedStyle(flexbox, null).webkitFlexDirection', 'row');
 
index 6c6bc57..37c596c 100644 (file)
@@ -144,7 +144,6 @@ zoom: 1;
 -webkit-column-width: auto;
 -webkit-flex-order: 0;
 -webkit-flex-pack: start;
--webkit-flex-align: stretch;
 -webkit-flex-item-align: stretch;
 -webkit-flex-direction: row;
 -webkit-flex-flow: row nowrap;
index 6a99ef2..e9904b4 100644 (file)
@@ -286,8 +286,6 @@ rect: style.getPropertyValue(-webkit-flex-order) : 0
 rect: style.getPropertyCSSValue(-webkit-flex-order) : [object CSSPrimitiveValue]
 rect: style.getPropertyValue(-webkit-flex-pack) : start
 rect: style.getPropertyCSSValue(-webkit-flex-pack) : [object CSSPrimitiveValue]
-rect: style.getPropertyValue(-webkit-flex-align) : stretch
-rect: style.getPropertyCSSValue(-webkit-flex-align) : [object CSSPrimitiveValue]
 rect: style.getPropertyValue(-webkit-flex-item-align) : stretch
 rect: style.getPropertyCSSValue(-webkit-flex-item-align) : [object CSSPrimitiveValue]
 rect: style.getPropertyValue(-webkit-flex-direction) : row
@@ -800,8 +798,6 @@ g: style.getPropertyValue(-webkit-flex-order) : 0
 g: style.getPropertyCSSValue(-webkit-flex-order) : [object CSSPrimitiveValue]
 g: style.getPropertyValue(-webkit-flex-pack) : start
 g: style.getPropertyCSSValue(-webkit-flex-pack) : [object CSSPrimitiveValue]
-g: style.getPropertyValue(-webkit-flex-align) : stretch
-g: style.getPropertyCSSValue(-webkit-flex-align) : [object CSSPrimitiveValue]
 g: style.getPropertyValue(-webkit-flex-item-align) : stretch
 g: style.getPropertyCSSValue(-webkit-flex-item-align) : [object CSSPrimitiveValue]
 g: style.getPropertyValue(-webkit-flex-direction) : row
index 6f5af50..8fa00ba 100644 (file)
@@ -144,7 +144,6 @@ zoom: 1;
 -webkit-column-width: auto;
 -webkit-flex-order: 0;
 -webkit-flex-pack: start;
--webkit-flex-align: stretch;
 -webkit-flex-item-align: stretch;
 -webkit-flex-direction: row;
 -webkit-flex-flow: row nowrap;
index a9b2c2a..27ad6a0 100644 (file)
@@ -143,7 +143,6 @@ Computed style of an element whose parent's 'display' value is 'none':
     -webkit-column-width: auto
     -webkit-flex-order: 0
     -webkit-flex-pack: start
-    -webkit-flex-align: stretch
     -webkit-flex-item-align: stretch
     -webkit-flex-direction: row
     -webkit-flex-flow: row nowrap
index 83e2e9d..3f96dab 100644 (file)
@@ -286,8 +286,6 @@ rect: style.getPropertyValue(-webkit-flex-order) : 0
 rect: style.getPropertyCSSValue(-webkit-flex-order) : [object CSSPrimitiveValue]
 rect: style.getPropertyValue(-webkit-flex-pack) : start
 rect: style.getPropertyCSSValue(-webkit-flex-pack) : [object CSSPrimitiveValue]
-rect: style.getPropertyValue(-webkit-flex-align) : stretch
-rect: style.getPropertyCSSValue(-webkit-flex-align) : [object CSSPrimitiveValue]
 rect: style.getPropertyValue(-webkit-flex-item-align) : stretch
 rect: style.getPropertyCSSValue(-webkit-flex-item-align) : [object CSSPrimitiveValue]
 rect: style.getPropertyValue(-webkit-flex-direction) : row
@@ -800,8 +798,6 @@ g: style.getPropertyValue(-webkit-flex-order) : 0
 g: style.getPropertyCSSValue(-webkit-flex-order) : [object CSSPrimitiveValue]
 g: style.getPropertyValue(-webkit-flex-pack) : start
 g: style.getPropertyCSSValue(-webkit-flex-pack) : [object CSSPrimitiveValue]
-g: style.getPropertyValue(-webkit-flex-align) : stretch
-g: style.getPropertyCSSValue(-webkit-flex-align) : [object CSSPrimitiveValue]
 g: style.getPropertyValue(-webkit-flex-item-align) : stretch
 g: style.getPropertyCSSValue(-webkit-flex-item-align) : [object CSSPrimitiveValue]
 g: style.getPropertyValue(-webkit-flex-direction) : row
index 69cfa13..e4b509d 100644 (file)
@@ -143,7 +143,6 @@ Computed style of an element whose parent's 'display' value is 'none':
     -webkit-column-width: auto
     -webkit-flex-order: 0
     -webkit-flex-pack: start
-    -webkit-flex-align: stretch
     -webkit-flex-item-align: stretch
     -webkit-flex-direction: row
     -webkit-flex-flow: row nowrap
index d4807b0..87c33d3 100644 (file)
@@ -143,7 +143,6 @@ Computed style of an element whose parent's 'display' value is 'none':
     -webkit-column-width: auto
     -webkit-flex-order: 0
     -webkit-flex-pack: start
-    -webkit-flex-align: stretch
     -webkit-flex-item-align: stretch
     -webkit-flex-direction: row
     -webkit-flex-flow: row nowrap
index 69cfa13..e4b509d 100644 (file)
@@ -143,7 +143,6 @@ Computed style of an element whose parent's 'display' value is 'none':
     -webkit-column-width: auto
     -webkit-flex-order: 0
     -webkit-flex-pack: start
-    -webkit-flex-align: stretch
     -webkit-flex-item-align: stretch
     -webkit-flex-direction: row
     -webkit-flex-flow: row nowrap
index 6a99ef2..e9904b4 100644 (file)
@@ -286,8 +286,6 @@ rect: style.getPropertyValue(-webkit-flex-order) : 0
 rect: style.getPropertyCSSValue(-webkit-flex-order) : [object CSSPrimitiveValue]
 rect: style.getPropertyValue(-webkit-flex-pack) : start
 rect: style.getPropertyCSSValue(-webkit-flex-pack) : [object CSSPrimitiveValue]
-rect: style.getPropertyValue(-webkit-flex-align) : stretch
-rect: style.getPropertyCSSValue(-webkit-flex-align) : [object CSSPrimitiveValue]
 rect: style.getPropertyValue(-webkit-flex-item-align) : stretch
 rect: style.getPropertyCSSValue(-webkit-flex-item-align) : [object CSSPrimitiveValue]
 rect: style.getPropertyValue(-webkit-flex-direction) : row
@@ -800,8 +798,6 @@ g: style.getPropertyValue(-webkit-flex-order) : 0
 g: style.getPropertyCSSValue(-webkit-flex-order) : [object CSSPrimitiveValue]
 g: style.getPropertyValue(-webkit-flex-pack) : start
 g: style.getPropertyCSSValue(-webkit-flex-pack) : [object CSSPrimitiveValue]
-g: style.getPropertyValue(-webkit-flex-align) : stretch
-g: style.getPropertyCSSValue(-webkit-flex-align) : [object CSSPrimitiveValue]
 g: style.getPropertyValue(-webkit-flex-item-align) : stretch
 g: style.getPropertyCSSValue(-webkit-flex-item-align) : [object CSSPrimitiveValue]
 g: style.getPropertyValue(-webkit-flex-direction) : row
index d851ff5..2c212f8 100644 (file)
@@ -1,3 +1,34 @@
+2012-01-13  Ojan Vafai  <ojan@chromium.org>
+
+        Unreviewed, rolling out r104972.
+        http://trac.webkit.org/changeset/104972
+        https://bugs.webkit.org/show_bug.cgi?id=75782
+
+        Broke some tests
+
+        * css/CSSComputedStyleDeclaration.cpp:
+        (WebCore::CSSComputedStyleDeclaration::getPropertyCSSValue):
+        * css/CSSParser.cpp:
+        (WebCore::CSSParser::parseValue):
+        * css/CSSProperty.cpp:
+        (WebCore::CSSProperty::isInheritedProperty):
+        * css/CSSPropertyNames.in:
+        * css/CSSStyleApplyProperty.cpp:
+        (WebCore::CSSStyleApplyProperty::CSSStyleApplyProperty):
+        * css/CSSStyleSelector.cpp:
+        (WebCore::CSSStyleSelector::applyProperty):
+        * rendering/RenderBox.cpp:
+        (WebCore::RenderBox::sizesToIntrinsicLogicalWidth):
+        * rendering/RenderFlexibleBox.cpp:
+        (WebCore::RenderFlexibleBox::layoutAndPlaceChildren):
+        (WebCore::RenderFlexibleBox::alignChildren):
+        * rendering/style/RenderStyle.h:
+        (WebCore::RenderStyleBitfields::initialFlexItemAlign):
+        * rendering/style/StyleFlexibleBoxData.cpp:
+        (WebCore::StyleFlexibleBoxData::StyleFlexibleBoxData):
+        (WebCore::StyleFlexibleBoxData::operator==):
+        * rendering/style/StyleFlexibleBoxData.h:
+
 2012-01-13  Anders Carlsson  <andersca@apple.com>
 
         -[WebTileCacheLayer setNeedsDisplay] doesn't trigger invalidation
index 8150df3..9b9abd5 100644 (file)
@@ -218,7 +218,6 @@ static const int computedProperties[] = {
 #endif
     CSSPropertyWebkitFlexOrder,
     CSSPropertyWebkitFlexPack,
-    CSSPropertyWebkitFlexAlign,
     CSSPropertyWebkitFlexItemAlign,
     CSSPropertyWebkitFlexDirection,
     CSSPropertyWebkitFlexFlow,
@@ -1527,11 +1526,7 @@ PassRefPtr<CSSValue> CSSComputedStyleDeclaration::getPropertyCSSValue(int proper
             return cssValuePool->createValue(style->flexOrder(), CSSPrimitiveValue::CSS_NUMBER);
         case CSSPropertyWebkitFlexPack:
             return cssValuePool->createValue(style->flexPack());
-        case CSSPropertyWebkitFlexAlign:
-            return cssValuePool->createValue(style->flexAlign());
         case CSSPropertyWebkitFlexItemAlign:
-            if (m_node && m_node->parentNode() && style->flexItemAlign() == AlignAuto)
-                return cssValuePool->createValue(m_node->parentNode()->computedStyle()->flexAlign());
             return cssValuePool->createValue(style->flexItemAlign());
         case CSSPropertyWebkitFlexDirection:
             return cssValuePool->createValue(style->flexDirection());
index 9a44ff2..46b3044 100644 (file)
@@ -1637,9 +1637,6 @@ bool CSSParser::parseValue(int propId, bool important)
     case CSSPropertyWebkitFlexPack:
         validPrimitive = id == CSSValueStart || id == CSSValueEnd || id == CSSValueCenter || id == CSSValueJustify;
         break;
-    case CSSPropertyWebkitFlexAlign:
-        validPrimitive = id == CSSValueStart || id == CSSValueEnd || id == CSSValueCenter || id == CSSValueBaseline || id == CSSValueStretch;
-        break;
     case CSSPropertyWebkitFlexItemAlign:
         validPrimitive = id == CSSValueAuto || id == CSSValueStart || id == CSSValueEnd || id == CSSValueCenter || id == CSSValueBaseline || id == CSSValueStretch;
         break;
index 1071bbe..7b2d201 100644 (file)
@@ -560,7 +560,6 @@ bool CSSProperty::isInheritedProperty(unsigned propertyID)
 #endif
     case CSSPropertyWebkitFlexOrder:
     case CSSPropertyWebkitFlexPack:
-    case CSSPropertyWebkitFlexAlign:
     case CSSPropertyWebkitFlexItemAlign:
     case CSSPropertyWebkitFlexDirection:
     case CSSPropertyWebkitFlexFlow:
index ceb2798..501447f 100644 (file)
@@ -255,7 +255,6 @@ z-index
 #if defined(ENABLE_CSS_FILTERS) && ENABLE_CSS_FILTERS
 -webkit-filter
 #endif
--webkit-flex-align
 -webkit-flex-direction
 -webkit-flex-flow
 -webkit-flex-item-align
index 45a6349..e4d8eec 100644 (file)
@@ -1756,7 +1756,6 @@ CSSStyleApplyProperty::CSSStyleApplyProperty()
 
     setPropertyHandler(CSSPropertyWebkitFlexOrder, ApplyPropertyDefault<int, &RenderStyle::flexOrder, int, &RenderStyle::setFlexOrder, int, &RenderStyle::initialFlexOrder>::createHandler());
     setPropertyHandler(CSSPropertyWebkitFlexPack, ApplyPropertyDefault<EFlexPack, &RenderStyle::flexPack, EFlexPack, &RenderStyle::setFlexPack, EFlexPack, &RenderStyle::initialFlexPack>::createHandler());
-    setPropertyHandler(CSSPropertyWebkitFlexAlign, ApplyPropertyDefault<EFlexAlign, &RenderStyle::flexAlign, EFlexAlign, &RenderStyle::setFlexAlign, EFlexAlign, &RenderStyle::initialFlexAlign>::createHandler());
     setPropertyHandler(CSSPropertyWebkitFlexItemAlign, ApplyPropertyDefault<EFlexAlign, &RenderStyle::flexItemAlign, EFlexAlign, &RenderStyle::setFlexItemAlign, EFlexAlign, &RenderStyle::initialFlexItemAlign>::createHandler());
     setPropertyHandler(CSSPropertyWebkitFlexDirection, ApplyPropertyDefault<EFlexDirection, &RenderStyle::flexDirection, EFlexDirection, &RenderStyle::setFlexDirection, EFlexDirection, &RenderStyle::initialFlexDirection>::createHandler());
     setPropertyHandler(CSSPropertyWebkitFlexWrap, ApplyPropertyDefault<EFlexWrap, &RenderStyle::flexWrap, EFlexWrap, &RenderStyle::setFlexWrap, EFlexWrap, &RenderStyle::initialFlexWrap>::createHandler());
index b81c8c5..a94adb8 100644 (file)
@@ -3786,7 +3786,6 @@ void CSSStyleSelector::applyProperty(int id, CSSValue *value)
     case CSSPropertyWordSpacing:
     case CSSPropertyWebkitFlexOrder:
     case CSSPropertyWebkitFlexPack:
-    case CSSPropertyWebkitFlexAlign:
     case CSSPropertyWebkitFlexItemAlign:
     case CSSPropertyWebkitFlexDirection:
     case CSSPropertyWebkitFlexFlow:
index 4069f32..befd5a2 100644 (file)
@@ -1793,13 +1793,8 @@ bool RenderBox::sizesToIntrinsicLogicalWidth(LogicalWidthType widthType) const
     // Flexible box items should shrink wrap, so we lay them out at their intrinsic widths.
     // In the case of columns that have a stretch alignment, we go ahead and layout at the
     // stretched size to avoid an extra layout when applying alignment.
-    if (parent()->isFlexibleBox()) {
-        if (!parent()->style()->isColumnFlexDirection())
-            return true;
-        EFlexAlign itemAlign = style()->flexItemAlign();
-        if (itemAlign != AlignStretch && (itemAlign != AlignAuto || parent()->style()->flexAlign() != AlignStretch))
-            return true;
-    }
+    if (parent()->isFlexibleBox() && (!parent()->style()->isColumnFlexDirection() || style()->flexItemAlign() != AlignStretch))
+        return true;
 
     // Flexible horizontal boxes lay out children at their intrinsic widths.  Also vertical boxes
     // that don't stretch their kids lay out their children at their intrinsic widths.
index 53997a1..0a646d9 100644 (file)
@@ -618,14 +618,6 @@ void RenderFlexibleBox::prepareChildForPositionedLayout(RenderBox* child, Layout
     }
 }
 
-static EFlexAlign flexAlignForChild(RenderBox* child)
-{
-    EFlexAlign align = child->style()->flexItemAlign();
-    if (align == AlignAuto)
-        return child->parent()->style()->flexAlign();
-    return align;
-}
-
 void RenderFlexibleBox::layoutAndPlaceChildren(FlexOrderIterator& iterator, const WTF::Vector<LayoutUnit>& childSizes, LayoutUnit availableFreeSpace, float totalPositiveFlexibility)
 {
     LayoutUnit mainAxisOffset = flowAwareBorderStart() + flowAwarePaddingStart();
@@ -647,7 +639,7 @@ void RenderFlexibleBox::layoutAndPlaceChildren(FlexOrderIterator& iterator, cons
         child->setChildNeedsLayout(true);
         child->layoutIfNeeded();
 
-        if (flexAlignForChild(child) == AlignBaseline) {
+        if (child->style()->flexItemAlign() == AlignBaseline) {
             LayoutUnit ascent = marginBoxAscent(child);
             LayoutUnit descent = (crossAxisMarginExtentForChild(child) + crossAxisExtentForChild(child)) - ascent;
 
@@ -740,9 +732,9 @@ void RenderFlexibleBox::alignChildren(FlexOrderIterator& iterator, LayoutUnit ma
         }
 
         // FIXME: Make sure this does the right thing with column flows.
-        switch (flexAlignForChild(child)) {
+        switch (child->style()->flexItemAlign()) {
         case AlignAuto:
-            ASSERT_NOT_REACHED();
+            // FIXME: Handle this once we add flex-align.
             break;
         case AlignStretch: {
             if (!isColumnFlow() && child->style()->logicalHeight().isAuto()) {
index 52d2887..88f950b 100644 (file)
@@ -801,7 +801,6 @@ public:
     float flexboxHeightNegativeFlex() const { return rareNonInheritedData->m_flexibleBox->m_heightNegativeFlex; }
     int flexOrder() const { return rareNonInheritedData->m_flexibleBox->m_flexOrder; }
     EFlexPack flexPack() const { return static_cast<EFlexPack>(rareNonInheritedData->m_flexibleBox->m_flexPack); }
-    EFlexAlign flexAlign() const { return static_cast<EFlexAlign>(rareNonInheritedData->m_flexibleBox->m_flexAlign); }
     EFlexAlign flexItemAlign() const { return static_cast<EFlexAlign>(rareNonInheritedData->m_flexibleBox->m_flexItemAlign); }
     EFlexDirection flexDirection() const { return static_cast<EFlexDirection>(rareNonInheritedData->m_flexibleBox->m_flexDirection); }
     bool isColumnFlexDirection() const { return flexDirection() == FlowColumn || flexDirection() == FlowColumnReverse; }
@@ -1222,7 +1221,6 @@ public:
     void setFlexboxHeightNegativeFlex(float f) { SET_VAR(rareNonInheritedData.access()->m_flexibleBox, m_heightNegativeFlex, f); }
     void setFlexOrder(int o) { SET_VAR(rareNonInheritedData.access()->m_flexibleBox, m_flexOrder, o); }
     void setFlexPack(EFlexPack p) { SET_VAR(rareNonInheritedData.access()->m_flexibleBox, m_flexPack, p); }
-    void setFlexAlign(EFlexAlign a) { SET_VAR(rareNonInheritedData.access()->m_flexibleBox, m_flexAlign, a); }
     void setFlexItemAlign(EFlexAlign a) { SET_VAR(rareNonInheritedData.access()->m_flexibleBox, m_flexItemAlign, a); }
     void setFlexDirection(EFlexDirection direction) { SET_VAR(rareNonInheritedData.access()->m_flexibleBox, m_flexDirection, direction); }
     void setFlexWrap(EFlexWrap w) { SET_VAR(rareNonInheritedData.access()->m_flexibleBox, m_flexWrap, w); }
@@ -1547,8 +1545,8 @@ public:
     static float initialFlexboxHeightNegativeFlex() { return 0; }
     static int initialFlexOrder() { return 0; }
     static EFlexPack initialFlexPack() { return PackStart; }
-    static EFlexAlign initialFlexAlign() { return AlignStretch; }
-    static EFlexAlign initialFlexItemAlign() { return AlignAuto; }
+    // FIXME: When we add in flex-align, default flex-item-align to AlignAuto.
+    static EFlexAlign initialFlexItemAlign() { return AlignStretch; }
     static EFlexDirection initialFlexDirection() { return FlowRow; }
     static EFlexWrap initialFlexWrap() { return FlexNoWrap; }
     static int initialMarqueeLoopCount() { return -1; }
index de6fabd..5ca0edb 100644 (file)
@@ -37,7 +37,6 @@ StyleFlexibleBoxData::StyleFlexibleBoxData()
     , m_heightNegativeFlex(RenderStyle::initialFlexboxHeightNegativeFlex())
     , m_flexOrder(RenderStyle::initialFlexOrder())
     , m_flexPack(RenderStyle::initialFlexPack())
-    , m_flexAlign(RenderStyle::initialFlexAlign())
     , m_flexItemAlign(RenderStyle::initialFlexItemAlign())
     , m_flexDirection(RenderStyle::initialFlexDirection())
     , m_flexWrap(RenderStyle::initialFlexWrap())
@@ -52,7 +51,6 @@ StyleFlexibleBoxData::StyleFlexibleBoxData(const StyleFlexibleBoxData& o)
     , m_heightNegativeFlex(o.m_heightNegativeFlex)
     , m_flexOrder(o.m_flexOrder)
     , m_flexPack(o.m_flexPack)
-    , m_flexAlign(o.m_flexAlign)
     , m_flexItemAlign(o.m_flexItemAlign)
     , m_flexDirection(o.m_flexDirection)
     , m_flexWrap(o.m_flexWrap)
@@ -63,8 +61,8 @@ bool StyleFlexibleBoxData::operator==(const StyleFlexibleBoxData& o) const
 {
     return m_widthPositiveFlex == o.m_widthPositiveFlex && m_widthNegativeFlex == o.m_widthNegativeFlex
         && m_heightPositiveFlex == o.m_heightPositiveFlex && m_heightNegativeFlex == o.m_heightNegativeFlex
-        && m_flexOrder == o.m_flexOrder && m_flexPack == o.m_flexPack && m_flexAlign == o.m_flexAlign
-        && m_flexItemAlign == o.m_flexItemAlign && m_flexDirection == o.m_flexDirection && m_flexWrap == o.m_flexWrap;
+        && m_flexOrder == o.m_flexOrder && m_flexPack == o.m_flexPack && m_flexItemAlign == o.m_flexItemAlign
+        && m_flexDirection == o.m_flexDirection && m_flexWrap == o.m_flexWrap;
 }
 
 }
index 840549b..4a90fbe 100644 (file)
@@ -50,7 +50,6 @@ public:
     int m_flexOrder;
 
     unsigned m_flexPack : 2; // EFlexPack
-    unsigned m_flexAlign : 3; // EFlexAlign
     unsigned m_flexItemAlign : 3; // EFlexAlign
     unsigned m_flexDirection : 2; // EFlexDirection
     unsigned m_flexWrap : 2; // EFlexWrap