Add another Worker suppression for valgrind.

Add another Worker suppression for valgrind.
BUG=23018
TEST=none
TBR=dank

Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=27175

[not_the_right_dank, 2009-09-25 14:24:37]

I'm on vacation, can you have stuart or jhawkins review?

On Thu, Sep 24, 2009 at 11:48 PM,  <dimich@chromium.org> wrote:
>
> Reviewers: dank,
>
> Description:
> Add another Worker suppression for valgrind.
> BUG=3D23018
> TEST=3Dnone
> TBR=3Ddank
>
> Please review this at http://codereview.chromium.org/235021
>
> SVN Base: svn://chrome-svn/chrome/trunk/src/
>
> Affected files:
> =A0M =A0 =A0 tools/valgrind/memcheck/suppressions.txt
>
>
> Index: tools/valgrind/memcheck/suppressions.txt
> =3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=
=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=
=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D
> --- tools/valgrind/memcheck/suppressions.txt =A0 =A0(revision 27153)
> +++ tools/valgrind/memcheck/suppressions.txt =A0 =A0(working copy)
> @@ -1319,3 +1319,18 @@
>
> =A0fun:_ZN2v88internal6InvokeEbNS0_6HandleINS0_10JSFunctionEEENS1_INS0_6O=
bjectEEEiPPPS4_Pb
>
> =A0fun:_ZN2v88internal9Execution4CallENS0_6HandleINS0_10JSFunctionEEENS2_=
INS0_6ObjectEEEiPPPS5_Pb
> =A0}
> +{
> + =A0 bug_23018
> + =A0 Memcheck:Addr4
> + =A0 fun:_ZN12_GLOBAL__N_115KillProcessTask3RunEv
> + =A0 fun:_ZN11MessageLoop7RunTaskEP4Task
> + =A0 fun:_ZN11MessageLoop21DeferOrRunPendingTaskERKNS_11PendingTaskE
> + =A0 fun:_ZN11MessageLoop13DoDelayedWorkEPN4base4TimeE
> + =A0 fun:_ZN4base18MessagePumpDefault3RunEPNS_11MessagePump8DelegateE
> + =A0 fun:_ZN11MessageLoop11RunInternalEv
> + =A0 fun:_ZN11MessageLoop10RunHandlerEv
> + =A0 fun:_ZN11MessageLoop3RunEv
> + =A0 fun:_Z10WorkerMainRK18MainFunctionParams
> + =A0 fun:ChromeMain
> + =A0 fun:main
> +}
>
>
>

[stuartmorgan, 2009-09-25 16:27:28]

LGTM

http://codereview.chromium.org/235021/diff/2001/3001
File tools/valgrind/memcheck/suppressions.txt (right):

http://codereview.chromium.org/235021/diff/2001/3001#newcode1066
Line 1066: fun:main
I would omit these last two frames.

[Dmitry Titov, 2009-09-25 20:07:26]

removed those frames as part of http://codereview.chromium.org/235020 (since
this one was submitted TBR)