diff options
| author | Jüri Valdmann <juri.valdmann@qt.io> | 2017-10-25 14:46:53 +0200 |
|---|---|---|
| committer | Jüri Valdmann <juri.valdmann@qt.io> | 2017-10-26 11:39:02 +0000 |
| commit | 3938260c7dcceee5cbf184d497b3b72d8b7a4910 (patch) | |
| tree | 3ac95a8978b976244c3f6b874ea4de5471f2aec1 /src/core/delegated_frame_node.h | |
| parent | 4649f2a11fe4b87e9b79f50ff8ac99ba9a2344f4 (diff) | |
Fix overeager freeing of resources in DelegatedFrameNode
After every run of DelegatedFrameNode::commit we take all the resources we
didn't need and queue them for freeing. Problems appear however with the recent
change that introduced clipping of invisible render passes and draw quads as we
may now decide to release resources too early and eventually crash.
Task-number: QTBUG-64032
Change-Id: I95138bc4c1caf8c191bfc801264309c5b03ef0f3
Reviewed-by: Michael Brüning <michael.bruning@qt.io>
Diffstat (limited to 'src/core/delegated_frame_node.h')
| -rw-r--r-- | src/core/delegated_frame_node.h | 2 |
1 files changed, 2 insertions, 0 deletions
diff --git a/src/core/delegated_frame_node.h b/src/core/delegated_frame_node.h index 3b6453a4c..e87b6abe3 100644 --- a/src/core/delegated_frame_node.h +++ b/src/core/delegated_frame_node.h @@ -125,6 +125,8 @@ private: static void fenceAndUnlockQt(DelegatedFrameNode *frameNode); ResourceHolder *findAndHoldResource(unsigned resourceId, QHash<unsigned, QSharedPointer<ResourceHolder> > &candidates); + void holdResources(const cc::DrawQuad *quad, QHash<unsigned, QSharedPointer<ResourceHolder> > &candidates); + void holdResources(const cc::RenderPass *pass, QHash<unsigned, QSharedPointer<ResourceHolder> > &candidates); QSGTexture *initAndHoldTexture(ResourceHolder *resource, bool quadIsAllOpaque, RenderWidgetHostViewQtDelegate *apiDelegate = 0); QExplicitlySharedDataPointer<ChromiumCompositorData> m_chromiumCompositorData; |