Rietveld Code Review Tool
Help | Bug tracker | Discussion group | Source code | Sign in
(716)

Issue 6499123: Fixed memory leak in deferred_canvas_record bench (Closed)

Can't Edit
Can't Publish+Mail
Start Review
Created:
12 years, 2 months ago by robertphillips
Modified:
12 years, 2 months ago
Reviewers:
EricB, TomH
CC:
skia-review_googlegroups.com
Base URL:
http://skia.googlecode.com/svn/trunk/
Visibility:
Public.

Description

Before this fix a repeat=20 run of this bench would "leak" 457 SkRefCnt-derived objects. With this fix only 1 is "leaked".

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+6 lines, -2 lines) Patch
M bench/DeferredCanvasBench.cpp View 1 chunk +6 lines, -2 lines 0 comments Download

Messages

Total messages: 3
robertphillips
12 years, 2 months ago (2012-09-17 11:21:19 UTC) #1
EricB
LGTM
12 years, 2 months ago (2012-09-17 11:51:38 UTC) #2
robertphillips
12 years, 2 months ago (2012-09-17 11:53:07 UTC) #3
committed as r5564
Sign in to reply to this message.

Powered by Google App Engine
RSS Feeds Recent Issues | This issue
This is Rietveld f62528b