Replace RenderMeter::updateLogicalHeight to RenderMeter::computeLogicalHeight

https://bugs.webkit.org/show_bug.cgi?id=97475

Reviewed by Ojan Vafai.

Using RenderMeter::computeLogicalHeight is part of bug 96804.
Also fix some code to be vertical writing mode aware. This doesn't actually cause
a behavioral difference because we use percentage heights/widths which don't depend on
updateLogicalWidth or computeLogicalHeight.  You can still see bugs if you try to set
the min-width on a <meter> node in a vertical writing mode.

No new tests, no behavioral changes.

* rendering/RenderMeter.cpp:
(WebCore::RenderMeter::updateLogicalWidth): Make vertical writing mode aware.
(WebCore::RenderMeter::computeLogicalHeight): Switch from updateLogicalHeight and make vertical writing mode aware.
* rendering/RenderMeter.h:
(RenderMeter):

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

diff --git a/Source/WebCore/ChangeLog b/Source/WebCore/ChangeLog
index fa9ff41c55548994878f65fa83f92815cce76ece..c97bd77665d21d4f05e6ec647a0f9f2de689c19a 100644 (file)
--- a/Source/WebCore/ChangeLog
+++ b/Source/WebCore/ChangeLog
@@ -1,3 +1,24 @@
+2012-09-24  Tony Chang  <tony@chromium.org>
+
+        Replace RenderMeter::updateLogicalHeight to RenderMeter::computeLogicalHeight
+        https://bugs.webkit.org/show_bug.cgi?id=97475
+
+        Reviewed by Ojan Vafai.
+
+        Using RenderMeter::computeLogicalHeight is part of bug 96804.
+        Also fix some code to be vertical writing mode aware. This doesn't actually cause
+        a behavioral difference because we use percentage heights/widths which don't depend on
+        updateLogicalWidth or computeLogicalHeight.  You can still see bugs if you try to set
+        the min-width on a <meter> node in a vertical writing mode.
+
+        No new tests, no behavioral changes.
+
+        * rendering/RenderMeter.cpp:
+        (WebCore::RenderMeter::updateLogicalWidth): Make vertical writing mode aware.
+        (WebCore::RenderMeter::computeLogicalHeight): Switch from updateLogicalHeight and make vertical writing mode aware.
+        * rendering/RenderMeter.h:
+        (RenderMeter):
+
 2012-09-24  Dimitri Glazkov  <dglazkov@chromium.org>
 
         Remove unbaked support for :scope pseudo-class.

diff --git a/Source/WebCore/rendering/RenderMeter.cpp b/Source/WebCore/rendering/RenderMeter.cpp
index b539c0a3adfd2f083dea81faed238125d85fa552..a4b3eca3589c6e142d538ff93e2767e26376d039 100644 (file)
--- a/Source/WebCore/rendering/RenderMeter.cpp
+++ b/Source/WebCore/rendering/RenderMeter.cpp
@@ -55,13 +55,22 @@ HTMLMeterElement* RenderMeter::meterElement() const
 void RenderMeter::updateLogicalWidth()
 {
     RenderBox::updateLogicalWidth();
-    setWidth(theme()->meterSizeForBounds(this, pixelSnappedIntRect(frameRect())).width());
+
+    IntSize frameSize = theme()->meterSizeForBounds(this, pixelSnappedIntRect(frameRect()));
+    setLogicalWidth(isHorizontalWritingMode() ? frameSize.width() : frameSize.height());
 }
 
-void RenderMeter::updateLogicalHeight()
+void RenderMeter::computeLogicalHeight(LayoutUnit logicalHeight, LayoutUnit logicalTop, LogicalExtentComputedValues& computedValues) const
 {
-    RenderBox::updateLogicalHeight();
-    setHeight(theme()->meterSizeForBounds(this, pixelSnappedIntRect(frameRect())).height());
+    RenderBox::computeLogicalHeight(logicalHeight, logicalTop, computedValues);
+
+    LayoutRect frame = frameRect();
+    if (isHorizontalWritingMode())
+        frame.setHeight(computedValues.m_extent);
+    else
+        frame.setWidth(computedValues.m_extent);
+    IntSize frameSize = theme()->meterSizeForBounds(this, pixelSnappedIntRect(frame));
+    computedValues.m_extent = isHorizontalWritingMode() ? frameSize.height() : frameSize.width();
 }
 
 double RenderMeter::valueRatio() const

diff --git a/Source/WebCore/rendering/RenderMeter.h b/Source/WebCore/rendering/RenderMeter.h
index 9051951d158275691f73d8f9000afd9259bb1c95..b7c3ecee740b96c9784a0bbac266d4bd55ae8d6e 100644 (file)
--- a/Source/WebCore/rendering/RenderMeter.h
+++ b/Source/WebCore/rendering/RenderMeter.h
@@ -40,7 +40,7 @@ public:
 
 private:    
     virtual void updateLogicalWidth() OVERRIDE;
-    virtual void updateLogicalHeight() OVERRIDE;
+    virtual void computeLogicalHeight(LayoutUnit logicalHeight, LayoutUnit logicalTop, LogicalExtentComputedValues&) const OVERRIDE;
 
     virtual const char* renderName() const { return "RenderMeter"; }
     virtual bool isMeter() const { return true; }