Initial ResizeObserver implementation

third_party/WebKit/Source/core/frame/FrameView.cpp

 /*
  * Copyright (C) 1998, 1999 Torben Weis <weis@kde.org>
  *                     1999 Lars Knoll <knoll@kde.org>
  *                     1999 Antti Koivisto <koivisto@kde.org>
  *                     2000 Dirk Mueller <mueller@kde.org>
  * Copyright (C) 2004, 2005, 2006, 2007, 2008 Apple Inc. All rights reserved.
  *           (C) 2006 Graham Dennis (graham.dennis@gmail.com)
  *           (C) 2006 Alexey Proskuryakov (ap@nypop.com)
  * Copyright (C) 2009 Google Inc. All rights reserved.
  *
@@ skipping 19 matching lines @@
 #include "core/MediaTypeNames.h"
 #include "core/css/FontFaceSet.h"
 #include "core/css/resolver/StyleResolver.h"
 #include "core/dom/AXObjectCache.h"
 #include "core/dom/Fullscreen.h"
 #include "core/dom/IntersectionObserverController.h"
 #include "core/editing/EditingUtilities.h"
 #include "core/editing/FrameSelection.h"
 #include "core/editing/RenderedPosition.h"
 #include "core/editing/markers/DocumentMarkerController.h"
+#include "core/events/ErrorEvent.h"
 #include "core/fetch/ResourceFetcher.h"
 #include "core/frame/EventHandlerRegistry.h"
 #include "core/frame/FrameHost.h"
 #include "core/frame/LocalFrame.h"
 #include "core/frame/Location.h"
 #include "core/frame/PageScaleConstraintsSet.h"
 #include "core/frame/Settings.h"
 #include "core/frame/TopControls.h"
 #include "core/frame/VisualViewport.h"
 #include "core/html/HTMLFrameElement.h"
@@ skipping 15 matching lines @@
 #include "core/layout/TracedLayoutObject.h"
 #include "core/layout/api/LayoutBoxModel.h"
 #include "core/layout/api/LayoutItem.h"
 #include "core/layout/api/LayoutViewItem.h"
 #include "core/layout/compositing/CompositedLayerMapping.h"
 #include "core/layout/compositing/CompositedSelection.h"
 #include "core/layout/compositing/PaintLayerCompositor.h"
 #include "core/layout/svg/LayoutSVGRoot.h"
 #include "core/loader/FrameLoader.h"
 #include "core/loader/FrameLoaderClient.h"
+#include "core/observer/ResizeObserverController.h"
 #include "core/page/AutoscrollController.h"
 #include "core/page/ChromeClient.h"
 #include "core/page/FocusController.h"
 #include "core/page/FrameTree.h"
 #include "core/page/Page.h"
 #include "core/page/scrolling/ScrollingCoordinator.h"
 #include "core/paint/FramePainter.h"
 #include "core/paint/PaintLayer.h"
 #include "core/paint/PrePaintTreeWalk.h"
 #include "core/plugins/PluginView.h"
@@ skipping 2353 matching lines @@
 void FrameView::scheduleVisualUpdateForPaintInvalidationIfNeeded()
 {
     LocalFrame* localFrameRoot = frame().localFrameRoot();
     if (!localFrameRoot->view()->m_isUpdatingAllLifecyclePhases || lifecycle().state() >= DocumentLifecycle::PrePaintClean) {
         // Schedule visual update to process the paint invalidation in the next cycle.
         localFrameRoot->scheduleVisualUpdateUnlessThrottled();
     }
     // Otherwise the paint invalidation will be handled in paint invalidation phase of this cycle.
 }
 
+void FrameView::notifyResizeObserver()
+{
+    if (!m_frame->document()->resizeObserverController())
+        return;
+
+    ResizeObserverController& resizeController = m_frame->document()->ensureResizeObserverController();
+
+    DCHECK(lifecycle().state() >= DocumentLifecycle::LayoutClean);
+
+    size_t resizeNotifyCount = 0;
+
+    while (resizeController.gatherObservations()) {

[szager1, 2016/07/11 18:33:47]

This is a much clearer organization of the loop, thanks!

[atotic1, 2016/07/11 23:18:57]

Done.

+        if (++resizeNotifyCount > ResizeObserverController::kRenderLoopLimit) {
+            resizeController.clearObservations();
+            // Report the error.
+            ErrorEvent * error = ErrorEvent::create("ResizeObserver loop limit exceeded", SourceLocation::capture(m_frame->document()), nullptr);
+            m_frame->document()->reportException(error, NotSharableCrossOrigin);
+            // Ensure notifications will get delivered in next cycle.

[szager1, 2016/07/11 18:33:47]

This comment is obsolete, I think.  Maybe just something about scheduling a new
frame because at this point, the layout may be left in an inconsistent state.

[atotic1, 2016/07/11 23:18:58]

The comment is meaningful. Without scheduleAnimation(), notifications will not
get delivered until next redraw. I can remove the comment if you'd like.
I put the comment in because this was a bug fix I put in after a test failed.

[szager1, 2016/07/11 23:33:34]

But just a few lines up, you call resizeController.clearObservations().  So
presumably, at this point, there won't be any notifications left for next frame
(which is, I think, the right thing to do).

+            if (FrameView* frameView = m_frame->view())
+                frameView->scheduleAnimation();
+            break;
+        }
+
+        resizeController.deliverObservations();
+
+        if (lifecycle().state() < DocumentLifecycle::LayoutClean || layoutView()->needsLayout())
+            updateStyleAndLayoutIfNeededRecursive();
+    }
+
+    DCHECK(!layoutView()->needsLayout());
+}
+
 // TODO(leviw): We don't assert lifecycle information from documents in child PluginViews.
 void FrameView::updateLifecyclePhasesInternal(LifeCycleUpdateOption phases)
 {
     Optional<TemporaryChange<bool>> isUpdatingAllLifecyclePhasesScope;
     if (phases == AllPhases)
         isUpdatingAllLifecyclePhasesScope.emplace(m_isUpdatingAllLifecyclePhases, true);
 
     // This must be called from the root frame, since it recurses down, not up.
     // Otherwise the lifecycles of the frames might be out of sync.
     ASSERT(m_frame->isLocalRoot());
 
     if (shouldThrottleRendering()) {
         updateViewportIntersectionsForSubtree(std::min(phases, OnlyUpToCompositingCleanPlusScrolling));
         return;
     }
 
     updateStyleAndLayoutIfNeededRecursive();
-    ASSERT(lifecycle().state() >= DocumentLifecycle::LayoutClean);
 
     if (phases == OnlyUpToLayoutClean) {
         updateViewportIntersectionsForSubtree(phases);
         return;
     }
 
+    notifyResizeObserver();

[szager1, 2016/07/11 18:33:47]

Are you sure we don't want to run this when (phases == OnlyUpToLayoutClean)? 
This begs the question: in a world with ResizeObserver, what do we mean by
"layout clean"?

[atotic1, 2016/07/11 23:18:57]

I believe that ResizeObserver should not run if phases == OnlyUpToLayoutClean
According to spec, ResizeObserver only notifies before paint. 
OnlyUpToLayoutClean does not paint.

I've looked at callers that trigger OnlyUpToLayoutClean condition, 
and only one is FrameSelection::unclippedBounds()

[szager1, 2016/07/11 23:33:34]

Good enough for me, I just wanted to make sure you were thinking about it.

+
+    DCHECK(lifecycle().state() >= DocumentLifecycle::LayoutClean);
+
     if (LayoutViewItem view = layoutViewItem()) {
         {
             TRACE_EVENT1("devtools.timeline", "UpdateLayerTree", "data", InspectorUpdateLayerTreeEvent::data(m_frame.get()));
 
             // This was required for slimming paint v1 but is only temporarily
             // needed for slimming paint v2.
             view.compositor()->updateIfNeededRecursive();
             scrollContentsIfNeededRecursive();
 
             ASSERT(lifecycle().state() >= DocumentLifecycle::CompositingClean);
@@ skipping 1695 matching lines @@
     return m_subtreeThrottled || (m_hiddenForThrottling && m_crossOriginForThrottling);
 }
 
 LayoutBox& FrameView::boxForScrollControlPaintInvalidation() const
 {
     ASSERT(!layoutViewItem().isNull());
     return *layoutView();
 }
 
 } // namespace blink