summaryrefslogtreecommitdiffstats
path: root/tools/valgrind
Commit message (Collapse)AuthorAgeFilesLines
* Use local copies of files from bug tracker (URL still in comments); avoids ↵dkegel@google.com2009-07-213-7/+131
| | | | | | | | wget dependency Review URL: http://codereview.chromium.org/159175 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@21238 0039d316-1c4b-4281-b951-d872f2087c98
* gtk: Two fixes for the tab leak:jhawkins@chromium.org2009-07-211-12/+0
| | | | | | | | | | | * The tab strip handles the lifetime of the tab widget, so don't try to increase the ref count in order to destroy the widget later. * Use DeleteSoon when deleting the dragged tab in order to give gtk a chance to clean up its ref counts added during the drag operation. BUG=12863 TEST=Extensive tab dragging. No crashes and the Linux UI valgrind bot should stay green. Review URL: http://codereview.chromium.org/159166 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@21230 0039d316-1c4b-4281-b951-d872f2087c98
* Revert r21117 as it caused reliability failures.estade@chromium.org2009-07-211-4/+2
| | | | | | | | | | Also included in this revert: r21165, r21180, and a couple valgrind suppression edits TBR=huanr Review URL: http://codereview.chromium.org/155876 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@21216 0039d316-1c4b-4281-b951-d872f2087c98
* Add suppression for RevocableStore leakdkegel@google.com2009-07-211-0/+17
| | | | | | | | | | Fix a missing ... in a suppression for an infrequent warning BUG=16625,17385 TEST=watch mac valgrind unit test bot go green Review URL: http://codereview.chromium.org/159150 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@21211 0039d316-1c4b-4281-b951-d872f2087c98
* Widen jam's suppression from this morning to match different inlining choicesdkegel@google.com2009-07-211-0/+1
| | | | | | | | | BUG=16128 TEST=watch valgrind ui bots go green Review URL: http://codereview.chromium.org/155867 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@21204 0039d316-1c4b-4281-b951-d872f2087c98
* Update the callstack of a supressed leak since my recent code refactoring ↵jam@chromium.org2009-07-211-2/+3
| | | | | | | | | changed the callstack. TBR=dkegel Review URL: http://codereview.chromium.org/155836 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@21181 0039d316-1c4b-4281-b951-d872f2087c98
* Script to build valgrind for chromiumdank@chromium.org2009-07-211-0/+34
| | | | | | Review URL: http://codereview.chromium.org/155848 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@21170 0039d316-1c4b-4281-b951-d872f2087c98
* Don't complain about leaks during fatal error loggingdkegel@google.com2009-07-201-1/+9
| | | | | | Review URL: http://codereview.chromium.org/155807 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@21126 0039d316-1c4b-4281-b951-d872f2087c98
* Widen ExtensionUnpacker_Test suppression yet againstuartmorgan@google.com2009-07-201-1/+1
| | | | | | | | | BUG=17129 TEST=Greener Mac valgind unit_test bot. Review URL: http://codereview.chromium.org/155808 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@21122 0039d316-1c4b-4281-b951-d872f2087c98
* Add a Mac suppression for a sporadic leak.stuartmorgan@google.com2009-07-201-0/+10
| | | | | | | | | BUG=17297 TEST=Greener Mac valgrind unit_test bot Review URL: http://codereview.chromium.org/155806 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@21115 0039d316-1c4b-4281-b951-d872f2087c98
* Linux: remove suppression for 9757.agl@chromium.org2009-07-201-8/+0
| | | | | | | | | | It appears that this issue has been fixed amoungst the multiple rewrites that this code has received in the mean time. BUG=9757 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@21106 0039d316-1c4b-4281-b951-d872f2087c98
* 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