Implement closed shadow adjustment for Element.offsetParent

third_party/WebKit/Source/core/html/HTMLElement.cpp

 /*
  * Copyright (C) 1999 Lars Knoll (knoll@kde.org)
  *           (C) 1999 Antti Koivisto (koivisto@kde.org)
  * Copyright (C) 2004-2008, 2013, 2014 Apple Inc. All rights reserved.
  * Copyright (C) 2009 Torch Mobile Inc. All rights reserved. (http://www.torchmobile.com/)
  * Copyright (C) 2011 Motorola Mobility. 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
@@ skipping 36 matching lines @@
 #include "core/events/KeyboardEvent.h"
 #include "core/frame/Settings.h"
 #include "core/frame/UseCounter.h"
 #include "core/html/HTMLBRElement.h"
 #include "core/html/HTMLFormElement.h"
 #include "core/html/HTMLInputElement.h"
 #include "core/html/HTMLMenuElement.h"
 #include "core/html/HTMLTemplateElement.h"
 #include "core/html/HTMLTextFormControlElement.h"
 #include "core/html/parser/HTMLParserIdioms.h"
+#include "core/layout/LayoutBoxModelObject.h"
 #include "core/layout/LayoutObject.h"
 #include "core/page/SpatialNavigation.h"
 #include "platform/Language.h"
 #include "platform/text/BidiResolver.h"
 #include "platform/text/BidiTextRun.h"
 #include "platform/text/TextRunIterator.h"
 #include "wtf/StdLibExtras.h"
 #include "wtf/text/CString.h"
 
 namespace blink {
@@ skipping 977 matching lines @@
         event->setDefaultHandled();
     }
 }
 
 const AtomicString& HTMLElement::eventParameterName()
 {
     DEFINE_STATIC_LOCAL(const AtomicString, eventString, ("event"));
     return eventString;
 }
 
+int HTMLElement::offsetLeftForBinding()
+{
+    document().updateStyleAndLayoutIgnorePendingStylesheetsForNode(this);
+    if (LayoutBoxModelObject* layoutObject = layoutBoxModelObject())
+        return adjustLayoutUnitForAbsoluteZoom(LayoutUnit(layoutObject->pixelSnappedOffsetLeft(unclosedOffsetParent())), layoutObject->styleRef()).round();
+    return 0;
+}
+
+int HTMLElement::offsetTopForBinding()
+{
+    document().updateStyleAndLayoutIgnorePendingStylesheetsForNode(this);
+    if (LayoutBoxModelObject* layoutObject = layoutBoxModelObject())
+        return adjustLayoutUnitForAbsoluteZoom(LayoutUnit(layoutObject->pixelSnappedOffsetTop(unclosedOffsetParent())), layoutObject->styleRef()).round();
+    return 0;
+}
+
+int HTMLElement::offsetWidthForBinding()
+{
+    document().updateStyleAndLayoutIgnorePendingStylesheetsForNode(this);
+    if (LayoutBoxModelObject* layoutObject = layoutBoxModelObject())
+        return adjustLayoutUnitForAbsoluteZoom(LayoutUnit(layoutObject->pixelSnappedOffsetWidth(unclosedOffsetParent())), layoutObject->styleRef()).round();
+    return 0;
+}
+
+int HTMLElement::offsetHeightForBinding()
+{
+    document().updateStyleAndLayoutIgnorePendingStylesheetsForNode(this);
+    if (LayoutBoxModelObject* layoutObject = layoutBoxModelObject())
+        return adjustLayoutUnitForAbsoluteZoom(LayoutUnit(layoutObject->pixelSnappedOffsetHeight(unclosedOffsetParent())), layoutObject->styleRef()).round();
+    return 0;
+}
+
+Element* HTMLElement::unclosedOffsetParent()
+{
+    document().updateStyleAndLayoutIgnorePendingStylesheetsForNode(this);
+
+    LayoutObject* layoutObject = this->layoutObject();
+    if (!layoutObject)
+        return nullptr;
+
+    Element* element = layoutObject->offsetParent();
+    while (element && (!element->isUnclosedNodeOf(*this) || (element->isInShadowTree() && element->containingShadowRoot()->type() == ShadowRootType::UserAgent)))

[hayato, 2016/06/27 06:58:32]

I do not think it is a good idea to loop here by calling offsetParent
repeatedly.
This loop should be merged into a LayoutObject::offsetParent itself.

Unless that, I am afraid that this would not be a spec compliant.

Also, is there any way to avoid calling isUnclosdeNodeOf repeatedly in each
loop?
If there is, it would be nice.

[kochi, 2016/06/28 07:59:59]

That will break LayoutObject::offsetParent, as you already commented previously.

To be spec compliant, all the dependency is isolated here to keep existing
Element::offsetParent() and LayoutObject::offsetParent() users working.
All the HTMLElement.offsetParent()-relative offsets are done in this file
(e.g. HTMLElement::offsetTopForBinding() etc.)

We could have LayoutObject::unclosedOffsetParent(), but it doesn't improve any
performance because it would use the same loop here.
That's a valid concern, but for most cases isUnclosedNodeOf() is called only
once,
so the additional cost for offsetParent should not be significant on average.

In cases where repeated calls are necessary (the element is distributed deep
under
shadow trees), this could be optimized so that only the first unclosed node
(probably
the shadow host of the element) and above are searched, but I may miss some
conditions, and need to give more thought on possible cases.

I added a TODO comment here to note that it is possible to a performance hit.

+        element = element->layoutObject()->offsetParent();
+    return element;
+}
+
 } // namespace blink
 
 #ifndef NDEBUG
 
 // For use in the debugger
 void dumpInnerHTML(blink::HTMLElement*);
 
 void dumpInnerHTML(blink::HTMLElement* element)
 {
     printf("%s\n", element->innerHTML().ascii().data());
 }
 #endif