Add FIXMEs for vertical writing mode and override sizes.

* rendering/RenderBox.cpp:
(WebCore::RenderBox::overrideLogicalContentWidth):
(WebCore::RenderBox::overrideLogicalContentHeight):

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

diff --git a/Source/WebCore/ChangeLog b/Source/WebCore/ChangeLog
index f593223..88741a2 100644 (file)
--- a/Source/WebCore/ChangeLog
+++ b/Source/WebCore/ChangeLog
@@ -1,3 +1,10 @@
+2012-06-29  Ojan Vafai  <ojan@chromium.org>
+
+        Add FIXMEs for vertical writing mode and override sizes.
+        * rendering/RenderBox.cpp:
+        (WebCore::RenderBox::overrideLogicalContentWidth):
+        (WebCore::RenderBox::overrideLogicalContentHeight):
+
 2012-06-29  James Robinson  <jamesr@chromium.org>
 
         [chromium] Use CCThread::Task in compositor's RateLimiter instead of Timer

diff --git a/Source/WebCore/rendering/RenderBox.cpp b/Source/WebCore/rendering/RenderBox.cpp
index 524082e..9a8cc50 100644 (file)
--- a/Source/WebCore/rendering/RenderBox.cpp
+++ b/Source/WebCore/rendering/RenderBox.cpp
@@ -702,11 +702,13 @@ void RenderBox::clearOverrideSize()
 
 LayoutUnit RenderBox::overrideLogicalContentWidth() const
 {
+    // FIXME: This should probably be returning contentLogicalWidth instead of contentWidth.
     return hasOverrideWidth() ? gOverrideWidthMap->get(this) : contentWidth();
 }
 
 LayoutUnit RenderBox::overrideLogicalContentHeight() const
 {
+    // FIXME: This should probably be returning contentLogicalHeight instead of contentHeight.
     return hasOverrideHeight() ? gOverrideHeightMap->get(this) : contentHeight();
 }