Update tests valgrind suppressions

Update tests valgrind suppressions

Committed: http://code.google.com/p/skia/source/detail?r=9871

[borenet, 2013-07-02 17:11:52]

LGTM

https://codereview.chromium.org/18510005/diff/1/tests/valgrind.supp
File tests/valgrind.supp (left):

https://codereview.chromium.org/18510005/diff/1/tests/valgrind.supp#oldcode13
tests/valgrind.supp:13: obj:/usr/lib/nvidia-current/libGL.so.280.13
Do we want to remove the nvidia suppression?  The bot is currently running on
AMD, but that may not always be the case.

https://codereview.chromium.org/18510005/diff/1/tests/valgrind.supp
File tests/valgrind.supp (right):

https://codereview.chromium.org/18510005/diff/1/tests/valgrind.supp#newcode22
tests/valgrind.supp:22: }
How much is left after these suppressions?

[robertphillips, 2013-07-02 17:31:08]

https://codereview.chromium.org/18510005/diff/1/tests/valgrind.supp
File tests/valgrind.supp (left):

https://codereview.chromium.org/18510005/diff/1/tests/valgrind.supp#oldcode13
tests/valgrind.supp:13: obj:/usr/lib/nvidia-current/libGL.so.280.13
I think we do want to remove them for now. I'm not even sure if they are correct
for nVidia.

https://codereview.chromium.org/18510005/diff/1/tests/valgrind.supp
File tests/valgrind.supp (right):

https://codereview.chromium.org/18510005/diff/1/tests/valgrind.supp#newcode22
tests/valgrind.supp:22: }
There are still a bunch. I defer to Ben for the Font issues. There are also a
couple that may be real.

[commit-bot: I haz the power, 2013-07-02 17:31:28]

CQ is trying da patch. Follow status at
https://skia-tree-status.appspot.com/cq/robertphillips@google.com/18510005/1

[borenet, 2013-07-02 17:31:48]

On 2013/07/02 17:31:08, robertphillips wrote:
> https://codereview.chromium.org/18510005/diff/1/tests/valgrind.supp
> File tests/valgrind.supp (left):
> 
> https://codereview.chromium.org/18510005/diff/1/tests/valgrind.supp#oldcode13
> tests/valgrind.supp:13: obj:/usr/lib/nvidia-current/libGL.so.280.13
> I think we do want to remove them for now. I'm not even sure if they are
correct
> for nVidia.
> 
> https://codereview.chromium.org/18510005/diff/1/tests/valgrind.supp
> File tests/valgrind.supp (right):
> 
> https://codereview.chromium.org/18510005/diff/1/tests/valgrind.supp#newcode22
> tests/valgrind.supp:22: }
> There are still a bunch. I defer to Ben for the Font issues. There are also a
> couple that may be real.

SGTM

[commit-bot: I haz the power, 2013-07-02 17:40:23]

Change committed as 9871