New issue
Advanced search Search tips
Starred by 2 users

Issue metadata

Status: Fixed
Closed: Jun 2017
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 3
Type: Bug

issue 563816

Sign in to add a comment

[OffscreenCanvas] OffscreenCanvasFrameDispatcherImpl::ReclaimResources is leaking resources

Project Member Reported by, Jun 21 2017

Issue description

In the case where resources are not backed by an SkImage (e.g. the non-gpu-accelerated cases). ReclaimResourceInternal is not being called, causing a resource leak.
Project Member

Comment 1 by, Jun 21 2017

The following revision refers to this bug:

commit 07708943258531dcacbd85b3fc54459989ca5bbd
Author: Justin Novosad <>
Date: Wed Jun 21 18:44:30 2017

Fix leak in OffscreenCanvasFrameDispatcherImpl

This change ensures that ReclaimResources always calls
ReclaimResourceInternal to avoid leaking resources in cases
where the canvas is not backed by an SkImage. It also simplifies
ReclaimResources by always waiting on the sync token.  There was
no measurable advantage in deferring the wait.

Bug:  735470 
Change-Id: I810ec1d016d1f312d9b074aff7ab8c088b188705
Reviewed-by: Fernando Serboncini <>
Commit-Queue: Justin Novosad <>
Cr-Commit-Position: refs/heads/master@{#481258}

Comment 2 by, Jun 21 2017

Status: Fixed (was: Assigned)

Sign in to add a comment