Trace table cell position indicies accurately.

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

Index: third_party/WebKit/Source/core/layout/TracedLayoutObject.cpp
diff --git a/third_party/WebKit/Source/core/layout/TracedLayoutObject.cpp b/third_party/WebKit/Source/core/layout/TracedLayoutObject.cpp
index 84cf0f5d62f5031e3764a1b0d8b2ff155f69481b..4f80f64bb15f4201475fa07cf6427b6b4d79312b 100644
--- a/third_party/WebKit/Source/core/layout/TracedLayoutObject.cpp
+++ b/third_party/WebKit/Source/core/layout/TracedLayoutObject.cpp
@@ -67,7 +67,10 @@ void dumpToTracedValue(const LayoutObject& object,
     tracedValue->setBoolean("childNeeds", object.normalChildNeedsLayout());
   if (object.posChildNeedsLayout())
     tracedValue->setBoolean("posChildNeeds", object.posChildNeedsLayout());
-  if (object.isTableCell()) {
+
+  // Table layout might be dirty if traceGeometry is false.
+  // See https://crbug.com/664271 .
+  if (traceGeometry && object.isTableCell()) {

[kouhei (in TOK), 2016/11/17 22:40:01]

Should we really dump the row/col to 0 if traceGeometry is false?

[kraynov, 2016/11/17 22:52:14]

Thanks! Fixed.

     const LayoutTableCell& c = toLayoutTableCell(object);
     tracedValue->setDouble("row", c.rowIndex());
     tracedValue->setDouble("col", c.absoluteColumnIndex());
@@ -75,7 +78,11 @@ void dumpToTracedValue(const LayoutObject& object,
       tracedValue->setDouble("rowSpan", c.rowSpan());
     if (c.colSpan() != 1)
       tracedValue->setDouble("colSpan", c.colSpan());
+  } else {
+    tracedValue->setDouble("row", 0);
+    tracedValue->setDouble("col", 0);
   }
+
   if (object.isAnonymous())
     tracedValue->setBoolean("anonymous", object.isAnonymous());
   if (object.isRelPositioned())