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

Issue 516593002: Add test that confirms Pictures don't leak pixel refs. (Closed)

Created:
6 years, 3 months ago by mtklein_C
Modified:
6 years, 3 months ago
Reviewers:
bungeman-skia, mtklein
CC:
reviews_skia.org
Base URL:
https://skia.googlesource.com/skia.git@master
Project:
skia
Visibility:
Public.

Description

Add test that confirms Pictures don't leak pixel refs. Have been investigating memory leak failures that show SkRecords' ImmutableBitmap copy leaking. I think this test proves its not an obvious problem with ImmutableBitmap or SkRecord. http://build.chromium.org/p/chromium.memory/builders/Linux%20Chromium%20OS%20ASan%20LSan%20Tests%20%282%29/builds/2509/steps/interactive_ui_tests/logs/stdio BUG=skia: Committed: https://skia.googlesource.com/skia/+/d72094d1c1062a9daa6fa92682ef29f02ba59335

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+29 lines, -0 lines) Patch
M tests/PictureTest.cpp View 2 chunks +29 lines, -0 lines 0 comments Download

Messages

Total messages: 6 (0 generated)
mtklein
mtklein@google.com changed reviewers: + bungeman@google.com, mtklein@google.com
6 years, 3 months ago (2014-08-27 19:03:08 UTC) #1
mtklein
6 years, 3 months ago (2014-08-27 19:03:08 UTC) #2
bungeman-skia
lgtm
6 years, 3 months ago (2014-08-27 19:03:59 UTC) #3
mtklein
The CQ bit was checked by mtklein@google.com
6 years, 3 months ago (2014-08-27 19:04:10 UTC) #4
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://skia-tree-status.appspot.com/cq/mtklein@chromium.org/516593002/1
6 years, 3 months ago (2014-08-27 19:05:10 UTC) #5
commit-bot: I haz the power
6 years, 3 months ago (2014-08-27 19:12:28 UTC) #6
Message was sent while issue was closed.
Committed patchset #1 (id:1) as d72094d1c1062a9daa6fa92682ef29f02ba59335

Powered by Google App Engine
This is Rietveld 408576698