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

Unified Diff: third_party/WebKit/Source/platform/graphics/compositing/PaintArtifactCompositor.cpp

Issue 2266223002: cc: Compute draw transforms dynamically. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Fix layout tests Created 4 years, 3 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: third_party/WebKit/Source/platform/graphics/compositing/PaintArtifactCompositor.cpp
diff --git a/third_party/WebKit/Source/platform/graphics/compositing/PaintArtifactCompositor.cpp b/third_party/WebKit/Source/platform/graphics/compositing/PaintArtifactCompositor.cpp
index 8e811bf0f93badfb6929d77332b80e14d94905fa..51f76d8ca7330c0bb4d58f71307be61f14777a1c 100644
--- a/third_party/WebKit/Source/platform/graphics/compositing/PaintArtifactCompositor.cpp
+++ b/third_party/WebKit/Source/platform/graphics/compositing/PaintArtifactCompositor.cpp
@@ -316,6 +316,7 @@ int PropertyTreeManager::compositorIdForClipNode(const ClipPaintPropertyNode* cl
compositorNode.clip = clipNode->clipRect().rect();
compositorNode.transform_id = compositorIdForTransformNode(clipNode->localTransformSpace());
compositorNode.target_transform_id = kRealRootNodeId;
+ compositorNode.target_effect_id = kSecondaryRootNodeId;
compositorNode.applies_local_clip = true;
compositorNode.layers_are_clipped = true;
compositorNode.layers_are_clipped_when_surfaces_disabled = true;
@@ -403,6 +404,7 @@ void PropertyTreeManager::buildEffectNodesRecursively(const EffectPaintPropertyN
dummyClip.owner_id = dummyLayer->id();
dummyClip.transform_id = kRealRootNodeId;
dummyClip.target_transform_id = kRealRootNodeId;
+ dummyClip.target_effect_id = kSecondaryRootNodeId;
cc::EffectNode& effectNode = *effectTree().Node(effectTree().Insert(cc::EffectNode(), compositorIdForCurrentEffectNode()));
effectNode.owner_id = dummyLayer->id();
@@ -474,6 +476,7 @@ void PaintArtifactCompositor::update(const PaintArtifact& paintArtifact)
// Mark the property trees as having been rebuilt.
host->GetLayerTree()->property_trees()->sequence_number = kPropertyTreeSequenceNumber;
host->GetLayerTree()->property_trees()->needs_rebuild = false;
+ host->GetLayerTree()->property_trees()->ResetCachedData();
}
} // namespace blink

Powered by Google App Engine
This is Rietveld 408576698