[CSS Exclusions] ExclusionShape inlines should use isFlippedBlocksWritingMode()
authorcommit-queue@webkit.org <commit-queue@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Wed, 5 Dec 2012 08:03:58 +0000 (08:03 +0000)
committercommit-queue@webkit.org <commit-queue@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Wed, 5 Dec 2012 08:03:58 +0000 (08:03 +0000)
https://bugs.webkit.org/show_bug.cgi?id=103939

Patch by Hans Muller <hmuller@adobe.com> on 2012-12-05
Reviewed by Dirk Schulze.

Just a minor cleanup: the protected ExclusionShape inlines, like minYForLogicalLine(),
now use isFlippedBlocksWritingMode() instead of testing for RightToLeftWritingMode directly.
This changeimproves consistency with the rest of WebKit and will work correctly
if "horizontal-bt" writing-mode support is ever added.

No new tests were added since the existing tests cover these methods.

* rendering/ExclusionShape.h:
(WebCore::ExclusionShape::minYForLogicalLine):
(WebCore::ExclusionShape::maxYForLogicalLine):
(WebCore::ExclusionShape::internalToLogicalBoundingBox):

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

Source/WebCore/ChangeLog
Source/WebCore/rendering/ExclusionShape.h

index 2a01a31..652adbb 100644 (file)
@@ -1,3 +1,22 @@
+2012-12-05  Hans Muller  <hmuller@adobe.com>
+
+        [CSS Exclusions] ExclusionShape inlines should use isFlippedBlocksWritingMode()
+        https://bugs.webkit.org/show_bug.cgi?id=103939
+
+        Reviewed by Dirk Schulze.
+
+        Just a minor cleanup: the protected ExclusionShape inlines, like minYForLogicalLine(),
+        now use isFlippedBlocksWritingMode() instead of testing for RightToLeftWritingMode directly.
+        This changeimproves consistency with the rest of WebKit and will work correctly
+        if "horizontal-bt" writing-mode support is ever added.
+
+        No new tests were added since the existing tests cover these methods.
+
+        * rendering/ExclusionShape.h:
+        (WebCore::ExclusionShape::minYForLogicalLine):
+        (WebCore::ExclusionShape::maxYForLogicalLine):
+        (WebCore::ExclusionShape::internalToLogicalBoundingBox):
+
 2012-12-05  Bear Travis  <betravis@adobe.com>
 
         Absolutely positioned non-replaced elements should resolve vertical margins against
index 075c002..f43dbf8 100644 (file)
@@ -69,9 +69,9 @@ public:
     virtual void getExcludedIntervals(float logicalTop, float logicalHeight, SegmentList&) const = 0;
 
 protected:
-    float minYForLogicalLine(float logicalTop, float logicalHeight) const { return (m_writingMode == RightToLeftWritingMode) ? m_logicalBoxHeight - logicalTop - logicalHeight : logicalTop; }
-    float maxYForLogicalLine(float logicalTop, float logicalHeight) const { return (m_writingMode == RightToLeftWritingMode) ? m_logicalBoxHeight - logicalTop : logicalTop + logicalHeight; }
-    FloatRect internalToLogicalBoundingBox(FloatRect r) const { return (m_writingMode == RightToLeftWritingMode) ? FloatRect(r.x(), m_logicalBoxHeight - r.maxY(), r.width(), r.height()) : r; }
+    float minYForLogicalLine(float logicalTop, float logicalHeight) const { return isFlippedBlocksWritingMode(m_writingMode) ? m_logicalBoxHeight - logicalTop - logicalHeight : logicalTop; }
+    float maxYForLogicalLine(float logicalTop, float logicalHeight) const { return  isFlippedBlocksWritingMode(m_writingMode) ? m_logicalBoxHeight - logicalTop : logicalTop + logicalHeight; }
+    FloatRect internalToLogicalBoundingBox(const FloatRect& r) const { return isFlippedBlocksWritingMode(m_writingMode) ? FloatRect(r.x(), m_logicalBoxHeight - r.maxY(), r.width(), r.height()) : r; }
 
 private:
     WritingMode m_writingMode;