Address some more valgrind issues

Address some more valgrind issues

R=borenet@google.com

Committed: https://code.google.com/p/skia/source/detail?r=12286

[robertphillips, 2013-11-14 14:21:41]

https://codereview.chromium.org/59713010/diff/1/gm/valgrind.supp
File gm/valgrind.supp (right):

https://codereview.chromium.org/59713010/diff/1/gm/valgrind.supp#newcode69
gm/valgrind.supp:69: }
This suppresses several instances of:

21,008 (20,992 direct, 16 indirect) bytes in 1 blocks are definitely lost in
loss record 331 of 357
at 0x4C29DB4: calloc (in /usr/lib/valgrind/vgpreload_memcheck-amd64-linux.so)
by 0x7DC84EE: xcb_connect_to_fd (in /usr/lib/x86_64-linux-gnu/libxcb.so.1.1.0)
by 0x7DCB7F3: xcb_connect_to_display_with_auth_info (in
/usr/lib/x86_64-linux-gnu/libxcb.so.1.1.0)
by 0x6073DB1: _XConnectXCB (xcb_disp.c:78)
by 0x6064A17: XOpenDisplay (OpenDis.c:141)
by 0x6C69E6: SkNativeGLContext::createGLContext()
(SkNativeGLContext_unix.cpp:70)
by 0x6C57E8: SkGLContextHelper::init(int, int) (SkGLContextHelper.cpp:38)
by 0x47FC88: ReadPixelsTest(skiatest::Reporter*, GrContextFactory*)
(GrContextFactory.h:134)
by 0x4809E5: skiatest::ReadPixelsTestClass::onRun(skiatest::Reporter*)
(ReadPixelsTest.cpp:411)
by 0x49670A: skiatest::Test::run() (Test.cpp:109)
by 0x48FDA0: SkTestRunnable::run() (skia_test.cpp:155)
by 0x48F746: tool_main(int, char**) (skia_test.cpp:246)
by 0x48F922: main (skia_test.cpp:276)

https://codereview.chromium.org/59713010/diff/1/tests/BlurTest.cpp
File tests/BlurTest.cpp (right):

https://codereview.chromium.org/59713010/diff/1/tests/BlurTest.cpp#newcode366
tests/BlurTest.cpp:366: #if SK_SUPPORT_GPU
This addresses (since gpuResult wasn't getting filled in):

Conditional jump or move depends on uninitialised value(s)
at 0x410237: match(int*, int*, int, int) (BlurTest.cpp:315)
by 0x4114F3: test_sigma_range(skiatest::Reporter*, GrContextFactory*)
(BlurTest.cpp:367)
by 0x4117CB: skiatest::BlurTestClass::onRun(skiatest::Reporter*)
(BlurTest.cpp:387)
by 0x49670A: skiatest::Test::run() (Test.cpp:109)
by 0x48FDA0: SkTestRunnable::run() (skia_test.cpp:155)
by 0x48F746: tool_main(int, char**) (skia_test.cpp:246)
by 0x48F922: main (skia_test.cpp:276)
Uninitialised value was created by a stack allocation
at 0x410CAC: test_sigma_range(skiatest::Reporter*, GrContextFactory*)
(BlurTest.cpp:324)

https://codereview.chromium.org/59713010/diff/1/tests/FontHostStreamTest.cpp
File tests/FontHostStreamTest.cpp (right):

https://codereview.chromium.org/59713010/diff/1/tests/FontHostStreamTest.cpp#...
tests/FontHostStreamTest.cpp:102: int ttcIndex;
This addresses:

80 (32 direct, 48 indirect) bytes in 1 blocks are definitely lost in loss record
174 of 357
at 0x4C2B1C7: operator new(unsigned long) (in
/usr/lib/valgrind/vgpreload_memcheck-amd64-linux.so)
by 0x5743EB: SkStream::NewFromFile(char const*) (SkStream.cpp:835)
by 0x65FF86:
SkFontConfigInterfaceDirect::openStream(SkFontConfigInterface::FontIdentity
const&) (SkFontConfigInterface_direct.cpp:555)
by 0x65FD30: FontConfigTypeface::onOpenStream(int*) const
(SkFontHost_fontconfig.cpp:202)
by 0x576EEF: SkTypeface::openStream(int*) const (SkTypeface.cpp:209)
by 0x43AC43: test_fontHostStream(skiatest::Reporter*)
(FontHostStreamTest.cpp:102)
by 0x43AFE1: skiatest::FontHostStreamTestClass::onRun(skiatest::Reporter*)
(FontHostStreamTest.cpp:116)
by 0x49670A: skiatest::Test::run() (Test.cpp:109)
by 0x48FDA0: SkTestRunnable::run() (skia_test.cpp:155)
by 0x64BB65: SkThreadPool::Loop(void*) (SkThreadPool.cpp:97)
by 0x65697A: thread_start(void*) (SkThreadUtils_pthread.cpp:66)
by 0x4E39E99: start_thread (pthread_create.c:308)
by 0x6C71CCC: clone (clone.S:112)

https://codereview.chromium.org/59713010/diff/1/tests/ImageFilterTest.cpp
File tests/ImageFilterTest.cpp (right):

https://codereview.chromium.org/59713010/diff/1/tests/ImageFilterTest.cpp#new...
tests/ImageFilterTest.cpp:132: 
This addresses two problems like:

120 (112 direct, 8 indirect) bytes in 1 blocks are definitely lost in loss
record 194 of 357
at 0x4C2B1C7: operator new(unsigned long) (in
/usr/lib/valgrind/vgpreload_memcheck-amd64-linux.so)
by 0x448B34: ImageFilterTest::Test(skiatest::Reporter*)
(ImageFilterTest.cpp:149)
by 0x448E07: skiatest::ImageFilterTestClass::onRun(skiatest::Reporter*)
(ImageFilterTest.cpp:163)
by 0x49670A: skiatest::Test::run() (Test.cpp:109)
by 0x48FDA0: SkTestRunnable::run() (skia_test.cpp:155)
by 0x64BB65: SkThreadPool::Loop(void*) (SkThreadPool.cpp:97)
by 0x65697A: thread_start(void*) (SkThreadUtils_pthread.cpp:66)
by 0x4E39E99: start_thread (pthread_create.c:308)
by 0x6C71CCC: clone (clone.S:112)

[borenet, 2013-11-14 14:42:49]

LGTM

[robertphillips, 2013-11-14 14:49:57]

Committed patchset #1 manually as r12286 (presubmit successful).