Get rid of remaining uses of AXObject::elementRect

third_party/WebKit/Source/modules/accessibility/AXNodeObject.cpp

 /*
  * Copyright (C) 2012, Google Inc. All rights reserved.
  *
  * Redistribution and use in source and binary forms, with or without
  * modification, are permitted provided that the following conditions
  * are met:
  *
  * 1.  Redistributions of source code must retain the above copyright
  *     notice, this list of conditions and the following disclaimer.
  * 2.  Redistributions in binary form must reproduce the above copyright
@@ skipping 1774 matching lines @@
     if (getNode()->isHTMLElement())
         htmlElement = toHTMLElement(getNode());
     if (htmlElement && isLabelableElement(htmlElement)) {
         if (toLabelableElement(htmlElement)->labels() && toLabelableElement(htmlElement)->labels()->length() > 0)
             return true;
     }
 
     return false;
 }
 
-LayoutRect AXNodeObject::elementRect() const
-{
-    // First check if it has a custom rect, for example if this element is tied to a canvas path.
-    if (!m_explicitElementRect.isEmpty()) {
-        LayoutRect bounds = m_explicitElementRect;
-        AXObject* canvas = axObjectCache().objectFromAXID(m_explicitContainerID);
-        if (canvas)
-            bounds.moveBy(canvas->elementRect().location());
-        return bounds;
-    }
-
-    // FIXME: If there are a lot of elements in the canvas, it will be inefficient.
-    // We can avoid the inefficient calculations by using AXComputedObjectAttributeCache.
-    if (getNode()->parentElement()->isInCanvasSubtree()) {
-        LayoutRect rect;
-
-        for (Node& child : NodeTraversal::childrenOf(*getNode())) {
-            if (child.isHTMLElement()) {
-                if (AXObject* obj = axObjectCache().get(&child)) {
-                    if (rect.isEmpty())
-                        rect = obj->elementRect();
-                    else
-                        rect.unite(obj->elementRect());
-                }
-            }
-        }
-
-        if (!rect.isEmpty())
-            return rect;
-    }
-
-    // If this object doesn't have an explicit element rect or computable from its children,
-    // for now, let's return the position of the ancestor that does have a position,
-    // and make it the width of that parent, and about the height of a line of text, so that it's clear the object is a child of the parent.
-
-    LayoutRect boundingBox;
-
-    for (AXObject* positionProvider = parentObject(); positionProvider; positionProvider = positionProvider->parentObject()) {
-        if (positionProvider->isAXLayoutObject()) {
-            LayoutRect parentRect = positionProvider->elementRect();
-            boundingBox.setSize(LayoutSize(parentRect.width(), LayoutUnit(std::min(10.0f, parentRect.height().toFloat()))));
-            boundingBox.setLocation(parentRect.location());
-            break;
-        }
-    }
-
-    return boundingBox;
-}
-
 void AXNodeObject::getRelativeBounds(AXObject** outContainer, FloatRect& outBoundsInContainer, SkMatrix44& outContainerTransform) const
 {
+    if (layoutObjectForRelativeBounds()) {
+        AXObject::getRelativeBounds(outContainer, outBoundsInContainer, outContainerTransform);
+        return;
+    }
+
     *outContainer = nullptr;
     outBoundsInContainer = FloatRect();
     outContainerTransform.setIdentity();
 
     // First check if it has explicit bounds, for example if this element is tied to a
     // canvas path. When explicit coordinates are provided, the ID of the explicit container
     // element that the coordinates are relative to must be provided too.
     if (!m_explicitElementRect.isEmpty()) {
         *outContainer = axObjectCache().objectFromAXID(m_explicitContainerID);
         if (*outContainer) {
@@ skipping 1073 matching lines @@
     return placeholder;
 }
 
 DEFINE_TRACE(AXNodeObject)
 {
     visitor->trace(m_node);
     AXObject::trace(visitor);
 }
 
 } // namespace blink