Route selection bounds updates through WebLayerTreeView

Source/core/rendering/RenderObject.cpp

Index: Source/core/rendering/RenderObject.cpp
diff --git a/Source/core/rendering/RenderObject.cpp b/Source/core/rendering/RenderObject.cpp
index 437c6654ccc6e1cdf87004ffc4463a6d2b934183..2a505e3786c11adb9bbb907abb8fdf592c803376 100644
--- a/Source/core/rendering/RenderObject.cpp
+++ b/Source/core/rendering/RenderObject.cpp
@@ -2449,6 +2449,25 @@ FloatPoint RenderObject::localToContainerPoint(const FloatPoint& localPoint, con
     return transformState.lastPlanarPoint();
 }
 
+FloatPoint RenderObject::localToInvalidationBackingPoint(const LayoutPoint& localPoint, RenderLayer** backingLayer)

[chrishtr, 2014/09/17 16:41:33]

Why can't this be unified with positionFromPaintInvalidationContainer?

[trchen, 2014/09/17 19:34:16]

They do different things.
positionFromPaintInvalidationContainer maps (0, 0) in local space into container
space.
localToInvalidationBackingPoint maps arbitrary point in local space into backing
layer space.

IMO positionFromPaintInvalidationContainer itself doesn't add much value since
all it can do is a subset of localToContainerPoint.

[chrishtr, 2014/09/18 04:06:00]

Then delete the RenderObject::positionFromPaintInvalidationContainer, and have
all the logic in RenderLayer.

+{
+    const RenderLayerModelObject* paintInvalidationContainer = containerForPaintInvalidation();
+    ASSERT(paintInvalidationContainer);
+    RenderLayer* layer = paintInvalidationContainer->layer();
+    ASSERT(layer);
+
+    if (backingLayer)
+        *backingLayer = layer;
+    FloatPoint containerPoint = localToContainerPoint(localPoint, paintInvalidationContainer, TraverseDocumentBoundaries);
+
+    if (layer->compositingState() == NotComposited) // This can happen for RenderFlowThread.
+        return containerPoint;
+
+    RenderLayer::mapPointToPaintBackingCoordinates(paintInvalidationContainer, containerPoint);
+    return containerPoint;
+}
+
+
 LayoutSize RenderObject::offsetFromContainer(const RenderObject* o, const LayoutPoint& point, bool* offsetDependsOnPoint) const
 {
     ASSERT(o == container());