Fix Ruby visual rects in flipped blocks writing mode.

third_party/WebKit/Source/core/layout/LayoutBox.cpp

Index: third_party/WebKit/Source/core/layout/LayoutBox.cpp
diff --git a/third_party/WebKit/Source/core/layout/LayoutBox.cpp b/third_party/WebKit/Source/core/layout/LayoutBox.cpp
index 78eca2d86f8337920eaf26036524d39d37768599..a2a84252e1674d4b5ef40ae8360dbf4eed7e401d 100644
--- a/third_party/WebKit/Source/core/layout/LayoutBox.cpp
+++ b/third_party/WebKit/Source/core/layout/LayoutBox.cpp
@@ -2102,6 +2102,8 @@ bool LayoutBox::mapToVisualRectInAncestorSpace(const LayoutBoxModelObject* ances
         // coordinate space to the parent space, then back to <tr> / <td>.
         if (tableRowContainer)
             topLeft.moveBy(-tableRowContainer->topLeftLocation(toLayoutBox(container)));
+    } else if (container->isRuby()) {
+        topLeft.moveBy(topLeftLocation());

[wkorman, 2016/07/20 22:12:33]

This was needed to apply offset resulting from any inline blocks that may
precede this one. Note the containing block is not the same as the container in
the relevant ruby case. Ideally we would pass the containing block in to
topLeftLocation() but we actually do not have it currently. Given this is only
done for Ruby, I think it's ok, but open to thoughts.

Here is logging snippet:

[1:1:0720/142316:1042318094031:ERROR:LayoutBox.cpp(2129)] obj="LayoutRubyRun
(anonymous)", container="LayoutRuby (inline) RUBY", container->isBox()=0,
containingBlock="LayoutBlockFlow DIV", origTopLeft="20.000000,2.000000",
topLeftLocation="0.000000,7.000000", inputRect="20.000000,2.000000
12.000000x4.000000", outputRect="20.000000,2.000000 12.000000x4.000000"

for the LayoutRubyRun in layout tree as below:

  LayoutBlockFlow {HTML} at (0,0) size 800x31
    LayoutBlockFlow {BODY} at (8,8) size 784x15
      LayoutBlockFlow {DIV} at (0,0) size 32x15
        LayoutText {#text} at (12,0) size 19x7
          text run at (12,0) width 7: "a"
        LayoutRuby (inline) {RUBY} at (0,0) size 19x8
          LayoutRubyRun (anonymous) at (12,7) size 20x8
            LayoutRubyText {RT} at (-12,0) size 12x8
              LayoutText {#text} at (0,2) size 12x4
                text run at (0,2) width 4: "c"
            LayoutRubyBase (anonymous) at (0,0) size 20x8
              LayoutText {#text} at (0,0) size 19x8
                text run at (0,0) width 8: "b"
        LayoutText {#text} at (0,0) size 0x0

     } else {
         topLeft.moveBy(location());
     }