Remove suppression for bidiresolver/bug_333572

Remove suppression for bidiresolver/bug_333572

A fix for the bidi leak was committed as blink r164996 thus the
suppressions are no longer needed.

R=thakis@chromium.org
BUG=333572
Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=245170

[Nico, 2014-01-14 23:02:17]

Lgtm
On Jan 14, 2014 1:52 PM, <eae@chromium.org> wrote:

> Reviewers: Nico,
>
> Description:
> Remove suppression for bidiresolver/bug_333572
>
> A fix for the bidi leak was committed as blink r164996 thus the
> suppressions are no longer needed.
>
> R=thakis@chromium.org
> BUG=333572
>
> Please review this at https://codereview.chromium.org/138453006/
>
> SVN Base: svn://svn.chromium.org/chrome/trunk/src
>
> Affected files (+0, -8 lines):
>   M tools/valgrind/memcheck/suppressions.txt
>
>
> Index: tools/valgrind/memcheck/suppressions.txt
> diff --git a/tools/valgrind/memcheck/suppressions.txt
> b/tools/valgrind/memcheck/suppressions.txt
> index 461d40cea8249fe5d395e81c0eca08bfa17a6de3..
> c1883eb1fc4b01f0548c8938055dad8dac7350c0 100644
> --- a/tools/valgrind/memcheck/suppressions.txt
> +++ b/tools/valgrind/memcheck/suppressions.txt
> @@ -6388,14 +6388,6 @@
>     fun:_ZNK7WebCore11RenderLayer25shouldBeSelfPaintingLayerEv
>  }
>  {
> -   bug_333572
> -   Memcheck:Leak
> -   fun:_Znw*
> -   fun:_ZN7WebCore12BidiResolverINS_15TextRunIteratorENS_
> 16BidiCharacterRunEE9appendRunEv
> -   fun:_ZN7WebCore12BidiResolverINS_15TextRunIteratorENS_
> 16BidiCharacterRunEE21createBidiRunsForLineERKS1_NS_
> 23VisualDirectionOverrideEbb
> -   fun:_ZN7WebCore10RenderText29computePreferredLogicalWidthsEfRN3WT
> F7HashSetIPKNS_14SimpleFontDataENS1_7PtrHashIS5_EENS1_
> 10HashTraitsIS5_EENS1_16DefaultAllocatorEEERNS_13GlyphOverflowE
> -}
> -{
>     bug_333992
>     Memcheck:Leak
>     fun:_Znw*
>
>
>

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 05:04:01]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/eae@chromium.org/138453006/1

[commit-bot: I haz the power, 2014-01-16 06:32:06]

Change committed as 245170