Clean compositing inputs for location APIs for sticky-affected elements.

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
@@ skipping 1123 matching lines @@
     event->SetDefaultHandled();
   }
 }
 
 const AtomicString& HTMLElement::EventParameterName() {
   DEFINE_STATIC_LOCAL(const AtomicString, event_string, ("event"));
   return event_string;
 }
 
 int HTMLElement::offsetLeftForBinding() {
+  EnsureLifecycleValidForLocationAPIs();

[fs, 2017/04/29 09:59:06]

I think you need this in offsetWidth/Height too (because of the pixelsnapping.)

[smcgruer, 2017/05/01 20:15:15]

Again done, again I'm not sure how to best test this :)

   Element* offset_parent = unclosedOffsetParent();
   if (LayoutBoxModelObject* layout_object = GetLayoutBoxModelObject())
     return AdjustLayoutUnitForAbsoluteZoom(
                LayoutUnit(layout_object->PixelSnappedOffsetLeft(offset_parent)),
                layout_object->StyleRef())
         .Round();
   return 0;
 }
 
 int HTMLElement::offsetTopForBinding() {
+  EnsureLifecycleValidForLocationAPIs();
   Element* offset_parent = unclosedOffsetParent();
   if (LayoutBoxModelObject* layout_object = GetLayoutBoxModelObject())
     return AdjustLayoutUnitForAbsoluteZoom(
                LayoutUnit(layout_object->PixelSnappedOffsetTop(offset_parent)),
                layout_object->StyleRef())
         .Round();
   return 0;
 }
 
 int HTMLElement::offsetWidthForBinding() {
@@ skipping 33 matching lines @@
 
 #ifndef NDEBUG
 
 // For use in the debugger
 void dumpInnerHTML(blink::HTMLElement*);
 
 void dumpInnerHTML(blink::HTMLElement* element) {
   printf("%s\n", element->innerHTML().Ascii().data());
 }
 #endif