Use localToContainerPoint when checking mapToContainer result

Source/core/rendering/RenderObject.cpp

 /*
  * Copyright (C) 1999 Lars Knoll (knoll@kde.org)
  *           (C) 1999 Antti Koivisto (koivisto@kde.org)
  *           (C) 2000 Dirk Mueller (mueller@kde.org)
  *           (C) 2004 Allan Sandfeld Jensen (kde@carewolf.com)
  * Copyright (C) 2004, 2005, 2006, 2007, 2008, 2011 Apple Inc. All rights reserved.
  * Copyright (C) 2009 Google Inc. All rights reserved.
  * Copyright (C) 2009 Torch Mobile Inc. All rights reserved. (http://www.torchmobile.com/)
  *
  * This library is free software; you can redistribute it and/or
@@ skipping 2305 matching lines @@
         updateImage(oldShapeValue ? oldShapeValue->image() : 0, newShapeValue ? newShapeValue->image() : 0);
 }
 
 LayoutRect RenderObject::viewRect() const
 {
     return view()->viewRect();
 }
 
 FloatPoint RenderObject::localToAbsolute(const FloatPoint& localPoint, MapCoordinatesFlags mode) const
 {
+    return localToAbsolute(0, localPoint, mode);
+}
+
+FloatPoint RenderObject::localToAbsolute(const RenderLayerModelObject* repaintContainer, const FloatPoint& localPoint, MapCoordinatesFlags mode) const
+{
     TransformState transformState(TransformState::ApplyTransformDirection, localPoint);
-    mapLocalToContainer(0, transformState, mode | ApplyContainerFlip);
+    mapLocalToContainer(repaintContainer, transformState, mode | ApplyContainerFlip);

[leviw_travelin_and_unemployed, 2014/05/30 17:17:50]

I don't think you actually want this. mapLocalToAbsolute intentionally passes in
zero, which tells mapLocalToContainer to map it to the outermost container (and
hence, absolute). Passing a repaintContainer means you're actually implementing
localToContainer, not localToAbsolute.

     transformState.flatten();
 
     return transformState.lastPlanarPoint();
 }
 
 FloatPoint RenderObject::absoluteToLocal(const FloatPoint& containerPoint, MapCoordinatesFlags mode) const
 {
     TransformState transformState(TransformState::UnapplyInverseTransformDirection, containerPoint);
     mapAbsoluteToLocalPoint(mode, transformState);
     transformState.flatten();
@@ skipping 1137 matching lines @@
 {
     if (object1) {
         const WebCore::RenderObject* root = object1;
         while (root->parent())
             root = root->parent();
         root->showRenderTreeAndMark(object1, "*", object2, "-", 0);
     }
 }
 
 #endif