Sky: Move iframe geometry updates to FrameView::updateLayout instead of

sky/engine/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 940 matching lines @@
 {
     m_nodeToDraw = node;
 }
 
 void FrameView::updateLayoutAndStyleForPainting()
 {
     // Updating layout can run script, which can tear down the FrameView.
     RefPtr<FrameView> protector(this);
 
     updateLayoutAndStyleIfNeededRecursive();
+    updateIFramesAfterLayout();
 
     if (RenderView* view = renderView()) {
         TRACE_EVENT_INSTANT1(TRACE_DISABLED_BY_DEFAULT("devtools.timeline"), "UpdateLayerTree", TRACE_EVENT_SCOPE_PROCESS, "frame", m_frame.get());
         view->compositor()->updateIfNeededRecursive();
 
         updateCompositedSelectionBoundsIfNeeded();
 
         invalidateTreeIfNeededRecursive();
     }
 
@@ skipping 140 matching lines @@
 
 void FrameView::setLayoutSizeInternal(const IntSize& size)
 {
     if (m_layoutSize == size)
         return;
 
     m_layoutSize = size;
     contentsResized();
 }
 
+void FrameView::addIFrame(RenderIFrame* iframe)
+{
+    m_iframes.add(iframe);
+}
+
+void FrameView::removeIFrame(RenderIFrame* iframe)
+{
+    m_iframes.remove(iframe);

[esprehn, 2014/11/17 20:07:49]

I don't think you need this change, we should track Render objects in
RenderView.

[Matt Perry, 2014/11/17 20:18:17]

Done.

+}
+
+void FrameView::updateIFramesAfterLayout()
+{
+    for (auto& iframe: m_iframes)
+        iframe->updateWidgetBounds();
+}
+
 } // namespace blink