Remove valgrind suppressions for non-existent methods

Remove valgrind suppressions for non-existent methods

None of the referenced methods exist at this time.

R=bruening@chromium.org
BUG=363487, 369141
Committed: https://crrev.com/fcc750b7b430443d2a6ca93e63c2f0ff351eaff0
Cr-Commit-Position: refs/heads/master@{#391812}

[Stephen Chennney, 2016-05-02 20:25:36]

The CQ bit was checked by schenney@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-05-02 20:25:55]

Dry run: CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1939273002/1
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1939273002/1

[Stephen Chennney, 2016-05-02 20:26:29]

The methods referenced have been remamed or removed, yet the world has not
collapsed. So I think it's safe to remove these suppressions. Also, in both
cases the bug is marked as Fixed.

[commit-bot: I haz the power, 2016-05-02 21:21:55]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2016-05-02 21:21:55]

Dry run: This issue passed the CQ dry run.

[Stephen Chennney, 2016-05-03 18:25:18]

Not sure what bot to try this on. Ideas? I can't see what I'm doing wrong, if
anything.

[Derek Bruening, 2016-05-05 01:01:50]

I am no longer full time on Chrome so it's best in general to send these to
other reviewers.

This query checks whether these are being used, and they are not:

http://chromium-build-logs.appspot.com/suppression_query?suppression_query=ht...
http://chromium-build-logs.appspot.com/suppression_query?suppression_query=ht...

The tools/valgrind/waterfall.sh script parses these files and is the usual
pre-commit check by memory sheriffs for syntax errors, etc.

LGTM

[Stephen Chennney, 2016-05-05 15:21:30]

On 2016/05/05 01:01:50, Derek Bruening wrote:
> I am no longer full time on Chrome so it's best in general to send these to
> other reviewers.
> 
> This query checks whether these are being used, and they are not:
> 
>
http://chromium-build-logs.appspot.com/suppression_query?suppression_query=ht...
>
http://chromium-build-logs.appspot.com/suppression_query?suppression_query=ht...
> 
> The tools/valgrind/waterfall.sh script parses these files and is the usual
> pre-commit check by memory sheriffs for syntax errors, etc.
> 
> LGTM

Thanks for the info, I appreciate you taking the time off your day job.

[Stephen Chennney, 2016-05-05 15:24:13]

The CQ bit was checked by schenney@chromium.org

[commit-bot: I haz the power, 2016-05-05 15:24:35]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1939273002/20001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1939273002/20001

[commit-bot: I haz the power, 2016-05-05 15:57:53]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-05-05 15:59:21]

Description was changed from

==========
Remove valgrind suppressions for non-existent methods

None of the referenced methods exist at this time.

R=bruening@chromium.org
BUG=363487,369141
==========

to

==========
Remove valgrind suppressions for non-existent methods

None of the referenced methods exist at this time.

R=bruening@chromium.org
BUG=363487,369141

Committed: https://crrev.com/fcc750b7b430443d2a6ca93e63c2f0ff351eaff0
Cr-Commit-Position: refs/heads/master@{#391812}
==========

[commit-bot: I haz the power, 2016-05-05 15:59:22]

Patchset 2 (id:??) landed as
https://crrev.com/fcc750b7b430443d2a6ca93e63c2f0ff351eaff0
Cr-Commit-Position: refs/heads/master@{#391812}