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

Issue 7066007: Valgrind Mac: Combine a few suppressions inside objc_msgSend. (Closed)

Created:
9 years, 7 months ago by Lei Zhang
Modified:
9 years, 7 months ago
Reviewers:
stuartmorgan
CC:
chromium-reviews, Timur Iskhodzhanov, Alexander Potapenko, pam+watch_chromium.org, stuartmorgan+watch_chromium.org
Visibility:
Public.

Description

Valgrind Mac: Combine a few suppressions inside objc_msgSend. BUG=73036, 73275, 80050 TEST=none TBR=stuartmorgan Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=86546

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+1 line, -87 lines) Patch
M tools/valgrind/memcheck/suppressions_mac.txt View 3 chunks +1 line, -87 lines 0 comments Download

Messages

Total messages: 4 (0 generated)
Lei Zhang
9 years, 7 months ago (2011-05-23 20:46:10 UTC) #1
Lei Zhang
If there's no objections, I'm going to add this suppression to try to get the ...
9 years, 7 months ago (2011-05-24 19:03:48 UTC) #2
Lei Zhang
On 2011/05/24 19:03:48, Lei Zhang wrote: > If there's no objections, I'm going to add ...
9 years, 7 months ago (2011-05-25 01:53:38 UTC) #3
Mark Mentovai
9 years, 7 months ago (2011-05-25 01:55:45 UTC) #4
I was only on the CC line, but LGTM.

Powered by Google App Engine
This is Rietveld 408576698