summaryrefslogtreecommitdiffstats
path: root/tools/valgrind
Commit message (Collapse)AuthorAgeFilesLines
* Widen suppression to match error on bot.dkegel@google.com2009-07-201-1/+0
| | | | | | | | | BUG=16579 TEST=watch valgrind ui bots go green Review URL: http://codereview.chromium.org/159089 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@21090 0039d316-1c4b-4281-b951-d872f2087c98
* Wait for valgrind to finish (the python script already does this, just forgot)dank@chromium.org2009-07-201-1/+14
| | | | | | Review URL: http://codereview.chromium.org/159065 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@21068 0039d316-1c4b-4281-b951-d872f2087c98
* Suppress two or so leaks and one invalid read in layout_testsdank@chromium.org2009-07-181-0/+38
| | | | | | | | | BUG=17184,17185,17187,17188 TEST=suppresses all warnings in sh tools/valgrind/valgrind_webkit_tests.sh LayoutTests/editing/pasteboard Review URL: http://codereview.chromium.org/159058 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@21061 0039d316-1c4b-4281-b951-d872f2087c98
* Script to valgrind layout testsdank@chromium.org2009-07-181-0/+23
| | | | | | Review URL: http://codereview.chromium.org/159047 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@21043 0039d316-1c4b-4281-b951-d872f2087c98
* Suppress yet another UtilityProcessHostTest.ExtensionUnpacker leak.dkegel@google.com2009-07-171-0/+7
| | | | | | | | | BUG=17129 TEST=none Review URL: http://codereview.chromium.org/155726 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@21025 0039d316-1c4b-4281-b951-d872f2087c98
* Suppress both leaks mentioned in bug16625 on both mac and linux.dkegel@google.com2009-07-171-0/+11
| | | | | | | | | BUG=16625 TEST=run UtilityProcessHostTest.ExtensionUnpacker under valgrind 40 times, verify no leaks reported Review URL: http://codereview.chromium.org/159014 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@20979 0039d316-1c4b-4281-b951-d872f2087c98
* Another occasional leak in WebCore::HTMLNames::init()dkegel@google.com2009-07-171-1/+13
| | | | | | | | | BUG=16579 TEST=run ui tests all day, verify no reported leaks :-) Review URL: http://codereview.chromium.org/159008 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@20970 0039d316-1c4b-4281-b951-d872f2087c98
* Fix wayward Q left over from debugging.dkegel@google.com2009-07-171-1/+1
| | | | | | | | TBR=stuart Review URL: http://codereview.chromium.org/159007 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@20957 0039d316-1c4b-4281-b951-d872f2087c98
* Fixes a leak in [LinkInfoBarController addAdditionalControls].rohitrao@chromium.org2009-07-171-30/+0
| | | | | | | | | | | | Removes the valgrind suppression for this leak. Valgrind: 1, rohitrao: 0 BUG=17002 TEST=No more leaky. Review URL: http://codereview.chromium.org/159005 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@20955 0039d316-1c4b-4281-b951-d872f2087c98
* Reland http://codereview.chromium.org/155465dank@chromium.org2009-07-172-17/+7
| | | | | | | | | | | Bug 15762 happens on Linux, too, so move suppression to generic file. Also trim unneeded end of stack. BUG=15762 TEST=watch valgrind test_shell_tests go greener on linux (and not redder on mac) Review URL: http://codereview.chromium.org/155658 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@20954 0039d316-1c4b-4281-b951-d872f2087c98
* Unbreak demangling on macdank@chromium.org2009-07-171-1/+2
| | | | | | Review URL: http://codereview.chromium.org/155659 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@20953 0039d316-1c4b-4281-b951-d872f2087c98
* Temporary suppression for infobar leaknirnimesh@chromium.org2009-07-171-0/+29
| | | | | | | | | BUG=17002 TBR=rohitrao Review URL: http://codereview.chromium.org/159004 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@20947 0039d316-1c4b-4281-b951-d872f2087c98
* Valgrind doesn't have an option (yet) to put suppressions into its xmldkegel@google.com2009-07-162-5/+24
| | | | | | | | | | | | | output file, so generate them by hand, and always print them. Lets you copy and paste from the bot into the suppressions file rather than having to reproduce yourself, which is handy for hard-to-reproduce warnings. BUG=none TEST=watch the bot. See the suppressions. Review URL: http://codereview.chromium.org/149762 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@20887 0039d316-1c4b-4281-b951-d872f2087c98
* Add two ui_test suppressions, widen another, to make my home jaunty system happydkegel@google.com2009-07-161-2/+27
| | | | | | | | | BUG=16093,16908 TEST=run ui tests on Jaunty with ISP with highish packet loss, verify no getaddrinfo or cssyyparse valgrind warnings Review URL: http://codereview.chromium.org/155638 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@20875 0039d316-1c4b-4281-b951-d872f2087c98
* Patch from timurrrr: Re-factor valgrind scripts to add tsan supportdkegel@google.com2009-07-169-249/+521
| | | | | | | | | | First reviewed at http://codereview.chromium.org/125272 BUG=none TEST=none Review URL: http://codereview.chromium.org/155528 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@20870 0039d316-1c4b-4281-b951-d872f2087c98
* Reverting 20592. Causing a bunch of valgrind MAC redness.mhm@chromium.org2009-07-142-7/+17
| | | | | | | TBR: dkegel Review URL: http://codereview.chromium.org/155483 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@20597 0039d316-1c4b-4281-b951-d872f2087c98
* Bug 15762 happens on Linux, too, so move suppression to generic file.dkegel@google.com2009-07-142-17/+7
| | | | | | | | | | Also trim unneeded end of stack. BUG=15762 TEST=watch valgrind test_shell_tests go greener on linux Review URL: http://codereview.chromium.org/155465 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@20592 0039d316-1c4b-4281-b951-d872f2087c98
* Widen two suppressions that weren't matching properly on my ghardy box.dkegel@google.com2009-07-131-10/+1
| | | | | | | | | BUG=16161,16583 TEST=run ui tests under valgrind over and over on ghardy, watch them fail less Review URL: http://codereview.chromium.org/149562 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@20534 0039d316-1c4b-4281-b951-d872f2087c98
* Add yet more suppressions to make the ui_tests not spew warnings under valgrinddkegel@google.com2009-07-131-0/+93
| | | | | | | | | | BUG=16089,16091,16576,16577,16578,16579,16583,16584,16586 TEST=watch valgrind ui bots go greener Review URL: http://codereview.chromium.org/155410 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@20481 0039d316-1c4b-4281-b951-d872f2087c98
* Point to correct app test binarynirnimesh@chromium.org2009-07-101-1/+1
| | | | | | | TBR=thomasvl Review URL: http://codereview.chromium.org/149451 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@20372 0039d316-1c4b-4281-b951-d872f2087c98
* Make sure we end up with an executable binary when juggling stripped ↵stuartmorgan@chromium.org2009-07-091-0/+1
| | | | | | | | binaries and fake dSYMs. Review URL: http://codereview.chromium.org/155305 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@20305 0039d316-1c4b-4281-b951-d872f2087c98
* Support running app_unittests under Valgrindnirnimesh@chromium.org2009-07-091-0/+4
| | | | | | Review URL: http://codereview.chromium.org/155216 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@20290 0039d316-1c4b-4281-b951-d872f2087c98
* Add more suppressions, adapt more to work on hardy (and not just jaunty), ↵dkegel@google.com2009-07-091-30/+101
| | | | | | | | | | | | fix the whitespace problem (again?!) BUG=10737,10737,16129,16153,16156,16161,16324,16326 TEST=watch linux ui valgrind test bot go greener Review URL: http://codereview.chromium.org/155196 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@20262 0039d316-1c4b-4281-b951-d872f2087c98
* Add temporary suppressions for a couple of newly-file Mac leaks.stuartmorgan@chromium.org2009-07-081-0/+25
| | | | | | | | BUG=16209,16210 TEST=somewhat happier Mac UI valgrind bots Review URL: http://codereview.chromium.org/155232 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@20189 0039d316-1c4b-4281-b951-d872f2087c98
* Fix a leak of the new location GURL by NotificationTask when doing a redirect.eroman@chromium.org2009-07-081-25/+0
| | | | | | | | | | | | | The problem is scoped_ptr<ResourceRequestDetails> is used to delete an instance of ResourceRedirectDetails, however the base class's destructor is non-virtual, so ResourceRedirectDetails extra field (GURL new_url_) does not get torn down properly. Note that the blame callstacks in the bug report appear unrelated, because std::string is doing some fancy refcounting under the hood. So as GURLs get passed around, they take over the GURL::spec's memory that got allocated earlier by the network stack. BUG=http://crbug.com/10873 TEST=The "RedirectTest.*" ui tests when run on linux valgrind bots should show no leaks Review URL: http://codereview.chromium.org/155204 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@20179 0039d316-1c4b-4281-b951-d872f2087c98
* Shuffle the Mac valgind suppressions around a bit to match the other ↵stuartmorgan@chromium.org2009-07-081-104/+118
| | | | | | | | | | suppression file format. No actual changes to the suppressions except to get rid of a name duplication. BUG=none TEST=Mac valgrind bots should be unchanged. Review URL: http://codereview.chromium.org/155225 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@20176 0039d316-1c4b-4281-b951-d872f2087c98
* Add a suppression for a leak the Mac UI tests get when plugins are first loaded.stuartmorgan@chromium.org2009-07-081-0/+14
| | | | | | | | BUG=none TEST=Fewer leaks on the Mac valgrind UI test bots Review URL: http://codereview.chromium.org/155190 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@20156 0039d316-1c4b-4281-b951-d872f2087c98
* Fix subtle whitespace problem. Lesson: never copy and paste a suppressiondank@chromium.org2009-07-071-10/+10
| | | | | | | | | from a web page, and always run it through valgrind once no matter how perfect it is visually. Review URL: http://codereview.chromium.org/149271 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@20065 0039d316-1c4b-4281-b951-d872f2087c98
* Rejigger a few suppressions to fire properly on the buildbots.dank@chromium.org2009-07-071-2/+28
| | | | | | | | | BUG=16096, 10737, 10738 TEST=watch the linux ui buildbots for valgrind warnings; this should make several go away Review URL: http://codereview.chromium.org/155152 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@20050 0039d316-1c4b-4281-b951-d872f2087c98
* Widen a couple suppressions, add a few more.dank@chromium.org2009-07-071-2/+29
| | | | | | | | | TEST=none BUG=12863,16093,16102,16103,16104 Review URL: http://codereview.chromium.org/155144 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@20036 0039d316-1c4b-4281-b951-d872f2087c98
* Suppress a wad of valgrind warnings that happen in nearly every ui test.dank@chromium.org2009-07-071-0/+87
| | | | | | | | | BUG=12863,16089,16091,16092,16093,16095,16096 TEST=run tools/valgrind/chrome_tests.sh -t ui and see many fewer warnings Review URL: http://codereview.chromium.org/149250 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@20025 0039d316-1c4b-4281-b951-d872f2087c98
* Fix tools/valgrind/chrome_tests.sh to always use --log-file optiondank@chromium.org2009-07-071-8/+15
| | | | | | | | | | | | | | | so fork() doesn't corrupt log files; lets --generate_suppressions work even with the hacky valgrind fix that makes anything but --log-file crash on child of fork(). Also avoid killing user desktop session if zygote dies! BUG=none, but related to fix for http://crbug.com/15771 TEST=patch valgrind with fork workaround; sh tools/valgrind/chrome_tests.sh --generate_suppressions -t ui logs you out on linux without this. Review URL: http://codereview.chromium.org/155130 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@20024 0039d316-1c4b-4281-b951-d872f2087c98
* Disable IPCSyncChannelTest.* under Valgrindnirnimesh@chromium.org2009-07-021-9/+14
| | | | | | | | | Times out too often under Valgrind on Mac Tracking bug: crbug.com/15817 Review URL: http://codereview.chromium.org/150218 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@19846 0039d316-1c4b-4281-b951-d872f2087c98
* Suppress valgrind errors in Apple librariesnirnimesh@chromium.org2009-07-021-0/+18
| | | | | | | | | TEST=check Mac valgrind bot BUG=15819 Review URL: http://codereview.chromium.org/149129 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@19831 0039d316-1c4b-4281-b951-d872f2087c98
* Fix the suppression using the output from valgrind.ojan@google.com2009-07-011-5/+15
| | | | | | | | | TBR=jorlow TEST=none BUG=15762 Review URL: http://codereview.chromium.org/151175 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@19774 0039d316-1c4b-4281-b951-d872f2087c98
* Add bug number now that the issue tracker is back up.ojan@google.com2009-07-011-2/+1
| | | | | | | | | TBR=jorlow TEST=none BUG=15762 Review URL: http://codereview.chromium.org/150173 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@19750 0039d316-1c4b-4281-b951-d872f2087c98
* Suppress valgrind error that started with r19701, whichojan@google.com2009-07-011-0/+8
| | | | | | | | | | | | | enabled the ImageOperations test. This should be fixed eventually, but suppressing it now to get the build green since it's not a regression. TEST=none BUG=Issue tracker is down Review URL: http://codereview.chromium.org/150172 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@19748 0039d316-1c4b-4281-b951-d872f2087c98
* Fix forjrg@chromium.org2009-06-301-0/+12
| | | | | | | | http://build.chromium.org/buildbot/waterfall/builders/Modules%20Mac%20(valgrind)/builds/1303/steps/valgrind%20test:%20unit/logs/stdio Review URL: http://codereview.chromium.org/151107 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@19666 0039d316-1c4b-4281-b951-d872f2087c98
* Widen /lib/ld.so suppression to match on 64 bit systems.dkegel@google.com2009-06-291-8/+6
| | | | | | | | | BUG=none TEST="make -r base_unittests; sh tools/valgrind/chrome-tests.sh -t base_unittests" on 64 bit ubuntu and verify no valgrind warnings from ld-*.so Review URL: http://codereview.chromium.org/147254 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@19497 0039d316-1c4b-4281-b951-d872f2087c98
* Teach the valgrind and layout tests about the output directory used by the ↵dkegel@google.com2009-06-271-26/+40
| | | | | | | | | | | Make build on Linux BUG=http://crbug.com/15454 TEST=export GYP_GENERATORS=make; rm -rf sconsbuild; gclient runhooks --force; make -r base_unittests layout_tests; sh tools/valgrind/chrome_tests.sh -t base_unittests; verify that it finds and runs base_unittests; same thing with -t layout_tests Review URL: http://codereview.chromium.org/147209 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@19453 0039d316-1c4b-4281-b951-d872f2087c98
* Add a framework leak suppression on the Mac.stuartmorgan@chromium.org2009-06-241-1/+11
| | | | | | | | | | Already filed with Apple. BUG=none TEST=fewer leaks reported from the UI tests under valgrind on Mac Review URL: http://codereview.chromium.org/146115 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@19195 0039d316-1c4b-4281-b951-d872f2087c98
* Add a switch to allow longer shutdown timeouts, since quitting under ↵stuartmorgan@chromium.org2009-06-191-1/+2
| | | | | | | | | | valgrind is very slow on the Mac. BUG=none TEST=Most/all of the UI tests should pass on the Mac when run under valgrind. Review URL: http://codereview.chromium.org/140008 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@18849 0039d316-1c4b-4281-b951-d872f2087c98
* Add a suppression for couple of valgrind warnings in OpenGL.framework ↵stuartmorgan@chromium.org2009-06-191-0/+7
| | | | | | | | | | initialization. BUG=none TEST=Run ui_tests under valgrind; see fewer warning that we can't fix. Review URL: http://codereview.chromium.org/131081 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@18824 0039d316-1c4b-4281-b951-d872f2087c98
* Remove valgrind suppression for BrowserWindowGtk::ConnectAccelerators().willchan@chromium.org2009-06-151-12/+0
| | | | | | | | BUG=http://crbug.com/10737 Review URL: http://codereview.chromium.org/118491 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@18422 0039d316-1c4b-4281-b951-d872f2087c98
* Increase valgrind's --num-callers to 50nirnimesh@chromium.org2009-06-111-1/+1
| | | | | | Review URL: http://codereview.chromium.org/119445 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@18162 0039d316-1c4b-4281-b951-d872f2087c98
* Fix suppression with PNGEncoder::EncodeBGRASkBitmap()nirnimesh@chromium.org2009-06-091-3/+1
| | | | | | | | BUG=12640 Review URL: http://codereview.chromium.org/119354 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@17950 0039d316-1c4b-4281-b951-d872f2087c98
* Re-Retry. This is the change from hell.glen@chromium.org2009-06-061-0/+10
| | | | | | | | | | | | | | | | | | | | | | | Revert DCHECKs to what they used to be - turns out a bunch of stuff in the code depended on input.empty() == true. --- I believe the code (with your width() * bbp fix) is correct. The UMR errors occur when the source image contain alpha. I believe the issue comes from webkit glue image decoder, and the reason this only started triggering valgrind errors is that the old ones were masked by the change detailed in bug 12640 Also adds valgrind suppression for the new code. TBR=Nick BUG=12891,12640 TEST=none Review URL: http://codereview.chromium.org/119271 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@17811 0039d316-1c4b-4281-b951-d872f2087c98
* Revert pngencoder changes.glen@chromium.org2009-06-051-10/+0
| | | | | | | BUG=none TEST=none git-svn-id: svn://svn.chromium.org/chrome/trunk/src@17803 0039d316-1c4b-4281-b951-d872f2087c98
* Retry. I believe the code (with your width() * bbp fix) is correct.glen@chromium.org2009-06-051-0/+10
| | | | | | | | | | | | | The UMR errors occur when the source image contain alpha. I believe the issue comes from webkit glue image decoder, and the reason this only started triggering valgrind errors is that the old ones were masked by the change detailed in bug 12640 Also adds valgrind suppression for the new code. BUG=12891,12640 TEST=none Review URL: http://codereview.chromium.org/118297 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@17792 0039d316-1c4b-4281-b951-d872f2087c98
* Broaden valgrind supression for known issue in zlib. This seems ok because ↵aa@chromium.org2009-06-011-2/+1
| | | | | | | | | | this code changes infrequently. Re-enable zip unit tests on mac. Review URL: http://codereview.chromium.org/118087 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@17369 0039d316-1c4b-4281-b951-d872f2087c98