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

Unified Diff: sky/engine/core/rendering/compositing/CompositedLayerMapping.cpp

Issue 686903003: Remove RenderPart. (Closed) Base URL: git@github.com:domokit/mojo.git@master
Patch Set: Created 6 years, 2 months 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/compositing/CompositedLayerMapping.cpp
diff --git a/sky/engine/core/rendering/compositing/CompositedLayerMapping.cpp b/sky/engine/core/rendering/compositing/CompositedLayerMapping.cpp
index 6f369d6367e5d5f457bcd9c713a128774f41ebc6..1454904c2104b9b7b085b98334b76ce894f3806a 100644
--- a/sky/engine/core/rendering/compositing/CompositedLayerMapping.cpp
+++ b/sky/engine/core/rendering/compositing/CompositedLayerMapping.cpp
@@ -41,7 +41,6 @@
#include "core/rendering/FilterEffectRenderer.h"
#include "core/rendering/RenderImage.h"
#include "core/rendering/RenderLayerStackingNodeIterator.h"
-#include "core/rendering/RenderPart.h"
#include "core/rendering/RenderVideo.h"
#include "core/rendering/RenderView.h"
#include "core/rendering/compositing/RenderLayerCompositor.h"
@@ -283,18 +282,6 @@ void CompositedLayerMapping::updateCompositedBounds()
m_contentOffsetInCompositingLayerDirty = true;
}
-void CompositedLayerMapping::updateAfterWidgetResize()
-{
- if (renderer()->isRenderPart()) {
- if (RenderLayerCompositor* innerCompositor = RenderLayerCompositor::frameContentsCompositor(toRenderPart(renderer()))) {
- innerCompositor->frameViewDidChangeSize();
- // We can floor this point because our frameviews are always aligned to pixel boundaries.
- ASSERT(m_compositedBounds.location() == flooredIntPoint(m_compositedBounds.location()));
- innerCompositor->frameViewDidChangeLocation(flooredIntPoint(contentsBox().location()));
- }
- }
-}
-
void CompositedLayerMapping::updateCompositingReasons()
{
// All other layers owned by this mapping will have the same compositing reason
@@ -455,8 +442,6 @@ bool CompositedLayerMapping::updateGraphicsLayerConfiguration()
m_graphicsLayer->setContentsToPlatformLayer(context->platformLayer());
layerConfigChanged = true;
}
- if (renderer->isRenderPart())
- layerConfigChanged = RenderLayerCompositor::parentFrameContentLayers(toRenderPart(renderer));
// Changes to either the internal hierarchy or the mask layer have an impact
// on painting phases, so we need to update when either are updated.
@@ -647,7 +632,6 @@ void CompositedLayerMapping::updateGraphicsLayerGeometry(const RenderLayer* comp
updateBackgroundColor();
updateDrawsContent();
updateContentsOpaque();
- updateAfterWidgetResize();
updateRenderingContext();
updateShouldFlattenTransform();
updateChildrenTransform();

Powered by Google App Engine
This is Rietveld 408576698