Remove valgrind memcheck suppression after r165092

Remove valgrind memcheck suppression after r165092

BUG=334049
Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=245113

[rhogan, 2014-01-15 18:52:18]

This was fixed by r165092.

[(unused - use chromium), 2014-01-15 19:00:54]

lgtm, thanks!


On Wed, Jan 15, 2014 at 10:52 AM, <robhogan@gmail.com> wrote:

> Reviewers: thakis,
>
> Message:
> This was fixed by r165092.
>
> Description:
> Remove valgrind memcheck suppression after r165092
>
> BUG=334049
>
> Please review this at https://codereview.chromium.org/138913002/
>
> SVN Base: https://chromium.googlesource.com/chromium/src.git@master
>
> Affected files (+1, -5 lines):
>   M AUTHORS
>   M tools/valgrind/memcheck/suppressions.txt
>
>
> Index: AUTHORS
> diff --git a/AUTHORS b/AUTHORS
> index ed4ecf31ed3cb94989d120a4854145ae56ccdff9..
> 5a7c379ec54923c74d311d741ba6b23d54bf564c 100644
> --- a/AUTHORS
> +++ b/AUTHORS
> @@ -250,6 +250,7 @@ Rob Buis <rob.buis@samsung.com>
>  Robert Bear Travis <bear.travis@gmail.com>
>  Robert Bear Travis <betravis@adobe.com>
>  Robert Goldberg <goldberg@adobe.com>
> +Robert Hogan <robhogan@gmail.com>
>  Robert Nagy <robert.nagy@gmail.com>
>  Robert Sesek <rsesek@bluestatic.org>
>  Rosen Dash <nqk836@motorola.com>
> Index: tools/valgrind/memcheck/suppressions.txt
> diff --git a/tools/valgrind/memcheck/suppressions.txt
> b/tools/valgrind/memcheck/suppressions.txt
> index 461d40cea8249fe5d395e81c0eca08bfa17a6de3..
> bae140f782acff16a88fb9650dda770b66a6059e 100644
> --- a/tools/valgrind/memcheck/suppressions.txt
> +++ b/tools/valgrind/memcheck/suppressions.txt
> @@ -6402,8 +6402,3 @@
>     fun:_ZN7content12_GLOBAL__N_123CreateWebFrameTestProxyEPNS
> _14RenderViewImplEi
>     fun:_ZN7content15RenderFrameImpl6CreateEPNS_14RenderViewImplEi
>  }
> -{
> -   bug_334049
> -   Memcheck:Uninitialized
> -   fun:_ZNK7WebCore11RenderBlock21isSelfCollapsingBlockEv
> -}
>
>
>

To unsubscribe from this group and stop receiving emails from it, send an email
to chromium-reviews+unsubscribe@chromium.org.

[commit-bot: I haz the power, 2014-01-15 19:17:09]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/robhogan@gmail.com/138913002/1

[commit-bot: I haz the power, 2014-01-16 04:43:34]

Change committed as 245113