Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(41)

Issue 10083002: Suppress a leak in AutomationProxyTest.AcceleratorHistory (issue 123307) (Closed)

Created:
8 years, 8 months ago by Alexander Potapenko
Modified:
8 years, 8 months ago
Reviewers:
antonm
CC:
chromium-reviews, glider+watch_chromium.org, pam+watch_chromium.org, timurrrr+watch_chromium.org, bruening+watch_chromium.org
Visibility:
Public.

Description

Suppress a leak in AutomationProxyTest.AcceleratorHistory (issue 123307) BUG=123307 TBR=antonm Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=132173

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+18 lines, -0 lines) Patch
M tools/valgrind/memcheck/suppressions.txt View 1 chunk +18 lines, -0 lines 0 comments Download

Messages

Total messages: 2 (0 generated)
Alexander Potapenko
TBR
8 years, 8 months ago (2012-04-13 11:20:23 UTC) #1
antonm
8 years, 8 months ago (2012-04-13 11:22:56 UTC) #2
Thanks a lot!

yours,
anton.

On Fri, Apr 13, 2012 at 3:20 PM,  <glider@chromium.org> wrote:
> Reviewers: antonm,
>
> Message:
> TBR
>
> Description:
> Suppress a leak in AutomationProxyTest.AcceleratorHistory (issue 123307)
>
> BUG=123307
> TBR=antonm
> Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=132173
>
> Please review this at https://chromiumcodereview.appspot.com/10083002/
>
> SVN Base: svn://svn.chromium.org/chrome/trunk/src/
>
> Affected files:
>  M     tools/valgrind/memcheck/suppressions.txt
>
>
> Index: tools/valgrind/memcheck/suppressions.txt
> ===================================================================
> --- tools/valgrind/memcheck/suppressions.txt    (revision 132164)
> +++ tools/valgrind/memcheck/suppressions.txt    (working copy)
> @@ -5765,6 +5765,24 @@
>    fun:_ZN7WebCore13LayerChromium16removeFromParentEv
>    fun:_ZN7WebCore21GraphicsLayerChromium16removeFromParentEv
>  }
> +{
> +   bug_123307
> +   Memcheck:Leak
> +   fun:malloc
> +   fun:_ZN3WTF10fastMallocEm
> +   fun:_ZN3WTF16fastZeroedMallocEm
> +   ...
> +
>
fun:_ZN7WebCore12_GLOBAL__N_111V8ObjectMapIN2v86ObjectEjE3setERKNS2_6HandleIS3_EERKj
> +
>
fun:_ZN7WebCore12_GLOBAL__N_110Serializer10greyObjectERKN2v86HandleINS2_6ObjectEEE
> +
>
fun:_ZN7WebCore12_GLOBAL__N_110Serializer11doSerializeEN2v86HandleINS2_5ValueEEEPNS1_9StateBaseE
> +
> fun:_ZN7WebCore12_GLOBAL__N_110Serializer9serializeEN2v86HandleINS2_5ValueEEE
> +
>
fun:_ZN7WebCore21SerializedScriptValueC1EN2v86HandleINS1_5ValueEEEPN3WTF6VectorINS5_6RefPtrINS_11MessagePortEEELm1EEEPNS6_INS7_INS5_11ArrayBufferEEELm1EEERb
> +
>
fun:_ZN7WebCore21SerializedScriptValue6createEN2v86HandleINS1_5ValueEEEPN3WTF6VectorINS5_6RefPtrINS_11MessagePortEEELm1EEEPNS6_INS7_INS5_11ArrayBufferEEELm1EEERb
> +   fun:_ZN7WebCoreL25handlePostMessageCallbackERKN2v89ArgumentsEb
> +   fun:_ZN7WebCore11V8DOMWindow19postMessageCallbackERKN2v89ArgumentsE
> +
>
fun:_ZN2v88internalL19HandleApiCallHelperILb0EEEPNS0_11MaybeObjectENS0_12_GLOBAL__N_116BuiltinArgumentsILNS0_21BuiltinExtraArgumentsE1EEEPNS0_7IsolateE
> +
>
fun:_ZN2v88internalL21Builtin_HandleApiCallENS0_12_GLOBAL__N_116BuiltinArgumentsILNS0_21BuiltinExtraArgumentsE1EEEPNS0_7IsolateE
> +}
>
>  #-----------------------------------------------------------------------
>  # 4. These only occur on our Google workstations
>
>

Powered by Google App Engine
This is Rietveld 408576698