Fix <input type="month"> tests
authortony@chromium.org <tony@chromium.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Thu, 4 Oct 2012 17:46:43 +0000 (17:46 +0000)
committertony@chromium.org <tony@chromium.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Thu, 4 Oct 2012 17:46:43 +0000 (17:46 +0000)
https://bugs.webkit.org/show_bug.cgi?id=98426

Reviewed by Ojan Vafai.

The month picker uses flexbox and assumes that it will be aligned with
the last line box. This looks visually correct. I will follow up on
www-style to see what the expected behavior is.

Tests: fast/forms/month-multiple-fields tests cover this.

* rendering/RenderFlexibleBox.cpp:
* rendering/RenderFlexibleBox.h:

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

Source/WebCore/ChangeLog
Source/WebCore/rendering/RenderFlexibleBox.cpp
Source/WebCore/rendering/RenderFlexibleBox.h

index 5d8492e..eef9b3d 100644 (file)
@@ -1,3 +1,19 @@
+2012-10-04  Tony Chang  <tony@chromium.org>
+
+        Fix <input type="month"> tests
+        https://bugs.webkit.org/show_bug.cgi?id=98426
+
+        Reviewed by Ojan Vafai.
+
+        The month picker uses flexbox and assumes that it will be aligned with
+        the last line box. This looks visually correct. I will follow up on
+        www-style to see what the expected behavior is.
+
+        Tests: fast/forms/month-multiple-fields tests cover this.
+
+        * rendering/RenderFlexibleBox.cpp:
+        * rendering/RenderFlexibleBox.h:
+
 2012-10-04  Sheriff Bot  <webkit.review.bot@gmail.com>
 
         Unreviewed, rolling out r130396.
index 3420572..fc019ff 100644 (file)
@@ -246,12 +246,6 @@ LayoutUnit RenderFlexibleBox::baselinePosition(FontBaseline baselineType, bool f
     return RenderBox::baselinePosition(baselineType, firstLine, direction, linePositionMode);
 }
 
-LayoutUnit RenderFlexibleBox::lastLineBoxBaseline() const
-{
-    // Similar to tables, if we're in an inline-block, align with the bottom of our box.
-    return -1;
-}
-
 LayoutUnit RenderFlexibleBox::firstLineBoxBaseline() const
 {
     ASSERT(m_orderIterator);
index 8e9454e..7c69175 100644 (file)
@@ -51,7 +51,6 @@ public:
 
     virtual LayoutUnit baselinePosition(FontBaseline, bool firstLine, LineDirectionMode, LinePositionMode = PositionOnContainingLine) const OVERRIDE;
     virtual LayoutUnit firstLineBoxBaseline() const OVERRIDE;
-    virtual LayoutUnit lastLineBoxBaseline() const OVERRIDE;
 
     virtual void paintChildren(PaintInfo& forSelf, const LayoutPoint&, PaintInfo& forChild, bool usePrintRect) OVERRIDE;