Kill outlineBox

Source/core/html/HTMLAreaElement.cpp

 /*
  * Copyright (C) 1999 Lars Knoll (knoll@kde.org)
  *           (C) 1999 Antti Koivisto (koivisto@kde.org)
  * Copyright (C) 2004, 2005, 2006, 2009, 2011 Apple Inc. All rights reserved.
  *
  * This library is free software; you can redistribute it and/or
  * modify it under the terms of the GNU Library General Public
  * License as published by the Free Software Foundation; either
  * version 2 of the License, or (at your option) any later version.
  *
@@ skipping 85 matching lines @@
 {
     if (!obj)
         return Path();
 
     // FIXME: This doesn't work correctly with transforms.
     FloatPoint absPos = obj->localToAbsolute();
 
     // Default should default to the size of the containing object.
     LayoutSize size = m_lastSize;
     if (m_shape == Default)
-        size = obj->absoluteOutlineBounds().size();
+        size = obj->absoluteClippedOverflowRect().size();

[eseidel, 2014/03/25 06:32:18]

absoluteClippedOverflowRect does a walk up your parent chain, at least when
outside of layout.  Did absoluteOutlineBounds too?

[Julien - ping for review, 2014/03/25 21:52:34]

It did as it called localToContainerQuad or mapToContainer.

 
     Path p = getRegion(size);
     float zoomFactor = obj->style()->effectiveZoom();
     if (zoomFactor != 1.0f) {
         AffineTransform zoomTransform;
         zoomTransform.scale(zoomFactor);
         p.transform(zoomTransform);
     }
 
     p.translate(toFloatSize(absPos));
@@ skipping 123 matching lines @@
 }
 
 bool HTMLAreaElement::supportsFocus() const
 {
     // If the AREA element was a link, it should support focus.
     // FIXME: This means that an AREA that is not a link cannot be made focusable through contenteditable or tabindex. Is it correct?
     return isLink();
 }
 
 }