LayoutTableCell::OffsetFromContainer() should use the flipped offset of its parent.

third_party/WebKit/LayoutTests/fast/dom/Element/getBoundingClientRect-vertical-child.html

+<style>
+ td { padding: 10px }

[dgrogan, 2017/04/25 23:23:35]

We rarely indent by 1 space, use 2.

+</style>
+<div style="-webkit-writing-mode: vertical-rl; writing-mode: vertical-rl">

[dgrogan, 2017/04/25 23:23:34]

Does safari require the prefixed property? If not, delete it from here.

+<table style="margin: 50px">

[dgrogan, 2017/04/25 23:23:35]

Why the margin?

+    <tr id="tr1">
+        <td id="td1a"><span id="s1a">1-A</span></td>
+        <td id="td1b"><span id="s1b">1-B</span></td>
+        <td id="td1c"><span id="s1c">1-C</span></td>
+        <td id="td1d"><span id="s1d">1-D</span></td>
+        <td id="td1e"><span id="s1e">1-E</span></td>
+    </tr>
+    <tr id="tr2">
+        <td id="td2a"><span id="s2a">2-A</span></td>
+        <td id="td2b"><span id="s2b">2-B</span></td>
+        <td id="td2c"><span id="s2c">2-C</span></td>
+        <td id="td2d"><span id="s2d">2-D</span></td>
+        <td id="td2e"><span id="s2e">2-E</span></td>
+    </tr>
+    <tr id="tr3">
+        <td id="td3a"><span id="s3a">3-A</span></td>
+        <td id="td3b"><span id="s3b">3-B</span></td>
+        <td id="td3c"><span id="s3c">3-C</span></td>
+        <td id="td3d"><span id="s3d">3-D</span></td>
+        <td id="td3e"><span id="s3e">3-E</span></td>
+    </tr>
+    <tr id="tr4">
+        <td id="td4a"><span id="s4a">4-A</span></td>
+        <td id="td4b"><span id="s4b">4-B</span></td>
+        <td id="td4c"><span id="s4c">4-C</span></td>
+        <td id="td4d"><span id="s4d">4-D</span></td>
+        <td id="td4e"><span id="s4e">4-E</span></td>
+    </tr>
+    <tr id="tr5">
+        <td id="td5a"><span id="s5a">5-A</span></td>
+        <td id="td5b"><span id="s5b">5-B</span></td>
+        <td id="td5c"><span id="s5c">5-C</span></td>
+        <td id="td5d"><span id="s5d">5-D</span></td>
+        <td id="td5e"><span id="s5e">5-E</span></td>
+    </tr>
+</table>
+<div style="padding: 100px"></div>

[dgrogan, 2017/04/25 23:23:35]

div unneeded?

+</div>
+<script src="../../../resources/testharness.js"></script>
+<script src="../../../resources/testharnessreport.js"></script>
+<script>
+function doesRectContainRect(parent, child) {
+    var margin = 0;

[dgrogan, 2017/04/25 23:23:34]

remove margin?

+    if (child.top < (parent.top - margin)) {
+        return false;
+    }
+    if (child.bottom > (parent.bottom + margin)) {
+        return false;
+    }
+    if (child.left < (parent.left - margin)) {
+        return false;
+    }
+    if (child.right > (parent.right + margin)) {
+        return false;
+    }
+    return true;
+}
+
+function assert_contains(parent, child, description) {
+    assert_true(doesRectContainRect(parent, child), description);
+}
+
+function checkRowColumn(row, column) {
+    var columnName = ["a", "b", "c", "d", "e"];
+    var trId = "tr" + (row + 1);
+    var tr = document.getElementById(trId);
+    var trRect = tr.getBoundingClientRect();
+    var name = (row + 1).toString() + "-" + columnName[column].toUpperCase();
+    var spanId = "s" + (row + 1) + columnName[column];
+    var span = document.getElementById(spanId);
+    var spanRect = span.getBoundingClientRect();
+    assert_contains(trRect, spanRect, name);

[dgrogan, 2017/04/25 23:23:35]

Can you add the 2 clientRect values to the fail message?

+}
+
+test(function() {
+    for (var i = 0; i < 5; i++) {

[dgrogan, 2017/04/25 23:23:34]

Rename i->row and j->column?
And, do we need a table with 25 squares? Would reducing to a 2x2 table work just
as well?

+        for (var j = 0; j < 5; j++) {
+            checkRowColumn(i, j);
+        }
+    }
+}, "The child of td should be inside of tr in vertical table");
+</script>