Store physical location in LayoutTableRow, just like in all other objects.

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

 // Copyright 2014 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "core/layout/LayoutBox.h"
 
 #include "core/html/HTMLElement.h"
 #include "core/layout/ImageQualityController.h"
 #include "core/layout/LayoutTestHelper.h"
 #include "testing/gtest/include/gtest/gtest.h"
@@ skipping 155 matching lines @@
 
   const LayoutBox* row1 = toLayoutBox(getLayoutObjectByElementId("row1"));
   EXPECT_EQ(LayoutPoint(0, 0), row1->location());
   EXPECT_EQ(LayoutPoint(300, 0), row1->topLeftLocation());
 
   const LayoutBox* cell1 = toLayoutBox(getLayoutObjectByElementId("cell1"));
   EXPECT_EQ(LayoutPoint(0, 0), cell1->location());
   EXPECT_EQ(LayoutPoint(300, 0), cell1->topLeftLocation());
 
   const LayoutBox* row2 = toLayoutBox(getLayoutObjectByElementId("row2"));
-  // TODO(crbug.com/652496): LayoutTableRow's location is in logical coordinates
-  // of the containing section, and topLeftLocation() is incorrect.
-  // This should be (100, 0).
-  EXPECT_EQ(LayoutPoint(0, 100), row2->location());
-  // This should be (0, 0).
-  EXPECT_EQ(LayoutPoint(100, 100), row2->topLeftLocation());
+  EXPECT_EQ(LayoutPoint(100, 0), row2->location());
+  EXPECT_EQ(LayoutPoint(0, 0), row2->topLeftLocation());
 
   const LayoutBox* cell2 = toLayoutBox(getLayoutObjectByElementId("cell2"));
   EXPECT_EQ(LayoutPoint(100, 0), cell2->location());
   EXPECT_EQ(LayoutPoint(0, 0), cell2->topLeftLocation());
 }
 
 }  // namespace blink