Consider text alignment and direction when computing the left offset for horizontal writing modes.

Source/core/rendering/RenderBox.cpp

 /*
  * Copyright (C) 1999 Lars Knoll (knoll@kde.org)
  *           (C) 1999 Antti Koivisto (koivisto@kde.org)
  *           (C) 2005 Allan Sandfeld Jensen (kde@carewolf.com)
  *           (C) 2005, 2006 Samuel Weinig (sam.weinig@gmail.com)
  * Copyright (C) 2005, 2006, 2007, 2008, 2009, 2010 Apple Inc. All rights reserved.
  * Copyright (C) 2013 Adobe Systems Incorporated. All rights reserved.
  *
  * This library is free software; you can redistribute it and/or
  * modify it under the terms of the GNU Library General Public
@@ skipping 3430 matching lines @@
     }
 }
 
 static void computeLogicalLeftPositionedOffset(LayoutUnit& logicalLeftPos, const RenderBox* child, LayoutUnit logicalWidthValue, const RenderBoxModelObject* containerBlock, LayoutUnit containerLogicalWidth)
 {
     // Deal with differing writing modes here.  Our offset needs to be in the containing block's coordinate space. If the containing block is flipped
     // along this axis, then we need to flip the coordinate.  This can only happen if the containing block is both a flipped mode and perpendicular to us.
     if (containerBlock->isHorizontalWritingMode() != child->isHorizontalWritingMode() && containerBlock->style()->isFlippedBlocksWritingMode()) {
         logicalLeftPos = containerLogicalWidth - logicalWidthValue - logicalLeftPos;
         logicalLeftPos += (child->isHorizontalWritingMode() ? containerBlock->borderRight() : containerBlock->borderBottom());
-    } else
-        logicalLeftPos += (child->isHorizontalWritingMode() ? containerBlock->borderLeft() : containerBlock->borderTop());
+    } else if (child->isHorizontalWritingMode()) {
+        RenderStyle* blockStyle = containerBlock->style();
+        bool containerBlockIsLTR = isLeftToRightDirection(blockStyle->direction());
+        if (containerBlockIsLTR && containerBlock->simplifiedTextAlign(blockStyle->textAlign()) == RIGHT) {

[eae, 2014/01/30 18:38:27]

How about using a local variable to hold the
containerBlock->simplifiedTextAlign(blockStyle->textAlign())? Would make the
code easier to read.

[mario.prada, 2014/02/04 14:30:13]

Ok

+            logicalLeftPos = containerLogicalWidth - (logicalWidthValue + child->style()->logicalRight().value());
+            logicalLeftPos += containerBlock->borderLogicalRight();

[eae, 2014/01/30 18:38:27]

Shouldn't this be borderAndPaddingLogicalRight?

[mario.prada, 2014/02/04 14:30:13]

I've thought the same and actually tried it, but that change was breaking some
other tests so I fell back to only considering the border _for the container_,
which in the end I think it makes sense because we are positioning here the
child object and so we only want to consider the additional space that falls out
of the container block, while still being a part of it: the border.

+        } else if (!containerBlockIsLTR && containerBlock->simplifiedTextAlign(blockStyle->textAlign()) == LEFT) {
+            logicalLeftPos += containerBlock->borderAndPaddingLogicalLeft() + logicalWidthValue;
+        } else {
+            logicalLeftPos += containerBlock->borderLeft();
+        }
+    } else {
+        logicalLeftPos += containerBlock->borderTop();
+    }
 }
 
 void RenderBox::shrinkToFitWidth(const LayoutUnit availableSpace, const LayoutUnit logicalLeftValue, const LayoutUnit bordersPlusPadding, LogicalExtentComputedValues& computedValues) const
 {
     // FIXME: would it be better to have shrink-to-fit in one step?
     LayoutUnit preferredWidth = maxPreferredLogicalWidth() - bordersPlusPadding;
     LayoutUnit preferredMinWidth = minPreferredLogicalWidth() - bordersPlusPadding;
     LayoutUnit availableWidth = availableSpace - logicalLeftValue;
     computedValues.m_extent = min(max(preferredMinWidth, availableWidth), preferredWidth);
 }
@@ skipping 1448 matching lines @@
         return 0;
 
     if (!layoutState && !flowThreadContainingBlock())
         return 0;
 
     RenderBlock* containerBlock = containingBlock();
     return containerBlock->offsetFromLogicalTopOfFirstPage() + logicalTop();
 }
 
 } // namespace WebCore