Radical sign drawn incorrectly due to refactoring typo

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

Patch by David Barton <dbarton@mathscribe.com> on 2011-12-17
Reviewed by Dan Bernstein.

LayoutTests/mathml/presentation/roots.xhtml shows the bug, but apparently its .png
checksum is ok so run-webkit-tests can't detect the problem.

* rendering/mathml/RenderMathMLRoot.cpp:
(WebCore::RenderMathMLRoot::paint):
In revision 88250, one paintOffset should have been adjustedPaintOffset.

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

diff --git a/Source/WebCore/ChangeLog b/Source/WebCore/ChangeLog
index d5589734780694627368b4ef1dd7c6ebfd09622a..b222ed20d2191a995984697e9fce25b6604f350e 100644 (file)
--- a/Source/WebCore/ChangeLog
+++ b/Source/WebCore/ChangeLog
@@ -1,3 +1,17 @@
+2011-12-17  David Barton  <dbarton@mathscribe.com>
+
+        Radical sign drawn incorrectly due to refactoring typo
+        https://bugs.webkit.org/show_bug.cgi?id=74780
+
+        Reviewed by Dan Bernstein.
+
+        LayoutTests/mathml/presentation/roots.xhtml shows the bug, but apparently its .png
+        checksum is ok so run-webkit-tests can't detect the problem.
+
+        * rendering/mathml/RenderMathMLRoot.cpp:
+        (WebCore::RenderMathMLRoot::paint):
+        In revision 88250, one paintOffset should have been adjustedPaintOffset.
+
 2011-12-17  Adrienne Walker  <enne@google.com>
 
         [chromium] Reverting r103011, r103135 due to Aura test failures

diff --git a/Source/WebCore/rendering/mathml/RenderMathMLRoot.cpp b/Source/WebCore/rendering/mathml/RenderMathMLRoot.cpp
index 2d38a204071c09c51beae62ee1d1dab87fae1f16..dbfe5e22d57792204dcdc375256febf59587e52b 100644 (file)
--- a/Source/WebCore/rendering/mathml/RenderMathMLRoot.cpp
+++ b/Source/WebCore/rendering/mathml/RenderMathMLRoot.cpp
@@ -142,7 +142,7 @@ void RenderMathMLRoot::paint(PaintInfo& info, const LayoutPoint& paintOffset)
     LayoutUnit start = adjustedPaintOffset.x() + indexWidth + gRadicalLeftMargin + style()->paddingLeft().value() - rootPad;
     adjustedPaintOffset.setY(adjustedPaintOffset.y() + style()->paddingTop().value() - rootPad);
     
-    FloatPoint topStart(start - topStartShift, paintOffset.y());
+    FloatPoint topStart(start - topStartShift, adjustedPaintOffset.y());
     FloatPoint bottomLeft(start - gRadicalBottomPointXPos * frontWidth , adjustedPaintOffset.y() + maxHeight + gRadicalBasePad);
     FloatPoint topLeft(start - gRadicalTopLeftPointXPos * frontWidth , adjustedPaintOffset.y() + gRadicalTopLeftPointYPos * maxHeight);
     FloatPoint leftEnd(start - frontWidth , topLeft.y() + gRadicalLeftEndYShift * style()->fontSize());