Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(591)

Unified Diff: sky/engine/core/rendering/RenderView.cpp

Issue 737523002: Sky: Move iframe geometry updates to FrameView::updateLayout instead of (Closed) Base URL: git@github.com:domokit/mojo.git@master
Patch Set: . Created 6 years, 1 month ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: sky/engine/core/rendering/RenderView.cpp
diff --git a/sky/engine/core/rendering/RenderView.cpp b/sky/engine/core/rendering/RenderView.cpp
index 44a01efe65b1f1a8916082d4a1254132ede607a1..beb07ac23d3d953a508e91643f8a6b864f345430 100644
--- a/sky/engine/core/rendering/RenderView.cpp
+++ b/sky/engine/core/rendering/RenderView.cpp
@@ -299,11 +299,6 @@ void RenderView::invalidateTreeIfNeeded(const PaintInvalidationState& paintInval
const RenderLayerModelObject* paintInvalidationContainer = &paintInvalidationState.paintInvalidationContainer();
mapRectToPaintInvalidationBacking(paintInvalidationContainer, dirtyRect, &paintInvalidationState);
invalidatePaintUsingContainer(paintInvalidationContainer, dirtyRect, InvalidationFull);
-
- // Also need to handle iframes, since they have a separate view outside
- // the hierarchy.
- for (auto& iframe: m_iframes)
- iframe->invalidateWidgetBounds();
}
RenderBlock::invalidateTreeIfNeeded(paintInvalidationState);
}
@@ -759,14 +754,4 @@ double RenderView::layoutViewportHeight() const
return viewHeight(IncludeScrollbars);
}
-void RenderView::addIFrame(RenderIFrame* iframe)
-{
- m_iframes.add(iframe);
-}
-
-void RenderView::removeIFrame(RenderIFrame* iframe)
-{
- m_iframes.remove(iframe);
-}
-
} // namespace blink
« sky/engine/core/frame/FrameView.cpp ('K') | « sky/engine/core/rendering/RenderView.h ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698