Route selection bounds updates through WebLayerTreeView

Source/core/rendering/RenderLayer.cpp

Index: Source/core/rendering/RenderLayer.cpp
diff --git a/Source/core/rendering/RenderLayer.cpp b/Source/core/rendering/RenderLayer.cpp
index 0447e81c4db73e425926c53ecf0c6230b6b3d4ff..6a4b6b5dc297d3946317e65be750fec83f96f2d9 100644
--- a/Source/core/rendering/RenderLayer.cpp
+++ b/Source/core/rendering/RenderLayer.cpp
@@ -544,6 +544,25 @@ LayoutPoint RenderLayer::positionFromPaintInvalidationContainer(const RenderObje
     return point;
 }
 
+void RenderLayer::mapPointToPaintBackingCoordinates(const RenderLayerModelObject* paintInvalidationContainer, FloatPoint& point)

[chrishtr, 2014/09/05 22:14:56]

Almost all of this method is already present in
RenderLayer::positionFromPaintInvalidationContainer. If you need the part about
contentOffsetInCompositingLayer(), then make another method to map from paint
invalidation container to paint backing coordinates, similar to what
is present for rectangles.

However, see my comments in RenderedPosition also, I don't think you need this
method at all.

[trchen, 2014/09/06 00:06:26]

Yep, this method maps from invalidation container to paint backing coordinates.
It is almost a copy & paste from the rect version. (Actually, should we try to
make it templated? So when we cleanup the squashing/non-squashing code path we
don't have to do the same thing twice.)

[chrishtr, 2014/09/06 00:12:19]

No, you should use RenderLayer::positionFromPaintInvalidationContainer and
extend it similar to what is already happening with rect.

+{
+    RenderLayer* paintInvalidationLayer = paintInvalidationContainer->layer();
+    if (!paintInvalidationLayer->groupedMapping()) {
+        point.move(paintInvalidationLayer->compositedLayerMapping()->contentOffsetInCompositingLayer());
+        return;
+    }
+
+    RenderLayerModelObject* transformedAncestor = paintInvalidationLayer->enclosingTransformedAncestor()->renderer();
+    if (!transformedAncestor)
+        return;
+
+    // |paintInvalidationContainer| may have a local 2D transform on it, so take that into account when mapping into the space of the
+    // transformed ancestor.
+    point = paintInvalidationContainer->localToContainerPoint(point, transformedAncestor);
+
+    point.moveBy(-paintInvalidationLayer->groupedMapping()->squashingOffsetFromTransformedAncestor());
+}
+
 void RenderLayer::mapRectToPaintBackingCoordinates(const RenderLayerModelObject* paintInvalidationContainer, LayoutRect& rect)
 {
     RenderLayer* paintInvalidationLayer = paintInvalidationContainer->layer();