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

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

Issue 2495973002: [SPv2] Add filter support in PaintArtifactCompositor (Closed)
Patch Set: Created 4 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: 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 20bdc54ae91a8f1adb14099335f9ae675adbbb35..c28ae4db64fefbd85b05b4c8a3ba8b07ebfaa874 100644
--- a/third_party/WebKit/Source/platform/graphics/compositing/PaintArtifactCompositor.cpp
+++ b/third_party/WebKit/Source/platform/graphics/compositing/PaintArtifactCompositor.cpp
@@ -676,25 +676,20 @@ void PropertyTreeManager::buildEffectNodesRecursively(
scoped_refptr<cc::Layer> dummyLayer = cc::Layer::Create();
m_rootLayer->AddChild(dummyLayer);
- // Also cc assumes a clip node is always created by a layer that creates
- // render surface.
- cc::ClipNode& dummyClip =
- *clipTree().Node(clipTree().Insert(cc::ClipNode(), kSecondaryRootNodeId));
- dummyClip.owner_id = dummyLayer->id();
- dummyClip.transform_id = kRealRootNodeId;
- dummyClip.target_transform_id = kRealRootNodeId;
- dummyClip.target_effect_id = kSecondaryRootNodeId;
+ int outputClipId = compositorIdForClipNode(nextEffect->outputClip());
cc::EffectNode& effectNode = *effectTree().Node(effectTree().Insert(
cc::EffectNode(), compositorIdForCurrentEffectNode()));
effectNode.owner_id = dummyLayer->id();
- effectNode.clip_id = dummyClip.id;
+ effectNode.clip_id = outputClipId;
+ effectNode.has_render_surface = true;
effectNode.opacity = nextEffect->opacity();
+ effectNode.filters = nextEffect->filter().asCcFilterOperations();
m_effectStack.append(BlinkEffectAndCcIdPair{nextEffect, effectNode.id});
dummyLayer->set_property_tree_sequence_number(kPropertyTreeSequenceNumber);
dummyLayer->SetTransformTreeIndex(kSecondaryRootNodeId);
- dummyLayer->SetClipTreeIndex(dummyClip.id);
+ dummyLayer->SetClipTreeIndex(outputClipId);
dummyLayer->SetEffectTreeIndex(effectNode.id);
dummyLayer->SetScrollTreeIndex(kRealRootNodeId);
}

Powered by Google App Engine
This is Rietveld 408576698