commit 06d37714f722ff9e27fa31e243428afa86bf6bd3
author mtklein <mtklein@chromium.org> Mon Jul 07 07:00:14 2014 -0700
committer Commit bot <commit-bot@chromium.org> Mon Jul 07 07:00:14 2014 -0700
tree e21e415bd0ef6b01f6f91a2ccead43fe81c4f4e0
parent 3ef04532990603888567426dc6275d81f37c7cce

Only you can prevent memory leaks.

SkPaint::setFoo() refs the foo, leaving the foo with a bonus ref unless you
call ->unref().

Valgrind caught this:
http://108.170.220.120:10117/builders/Test-Ubuntu12-ShuttleA-GTX550Ti-x86_64-Release-Valgrind/builds/317/steps/RunTests/logs/stdio

BUG=skia:
R=tomhudson@chromium.org, mtklein@google.com, tomhudson@google.com

Author: mtklein@chromium.org

Review URL: https://codereview.chromium.org/377523002

tests/RecordTest.cpp

diff --git a/tests/RecordTest.cpp b/tests/RecordTest.cpp
index 8ec5bcb..2fcc1e9 100644
--- a/tests/RecordTest.cpp
+++ b/tests/RecordTest.cpp
@@ -102,7 +102,7 @@
     *(bitmap2.getAddr32(0, 0)) = SK_ColorGREEN;
     SkShader* shader = SkShader::CreateBitmapShader(bitmap2, SkShader::kClamp_TileMode,
                                                     SkShader::kClamp_TileMode);
-    paint2.setShader(shader);
+    paint2.setShader(shader)->unref();
     REPORTER_ASSERT(r, shader->asABitmap(NULL, NULL, NULL) == SkShader::kDefault_BitmapType);
 
     APPEND(record, SkRecords::DrawRect, paint2, rect);