suppress unit_tests deliberate null deref

tools/valgrind/drmemory/suppressions.txt

Index: tools/valgrind/drmemory/suppressions.txt
diff --git a/tools/valgrind/drmemory/suppressions.txt b/tools/valgrind/drmemory/suppressions.txt
index 4c8987368ed19fb2c99b211de1a3c79e4fc590dd..9c6df2327843239376909e66a9a12eda9743208a 100644
--- a/tools/valgrind/drmemory/suppressions.txt
+++ b/tools/valgrind/drmemory/suppressions.txt
@@ -714,6 +714,16 @@ name=NPAPITesterBase.NoHangIfInitCrashes deliberate crash
 # TODO(bruening): switch to annotation once have support for that
 npapi_test_plugin.dll!NPAPIClient::PluginClient::Initialize
 
+# This test intentionally leaks an object and checks that it's never deleted
+LEAK

[Timur Iskhodzhanov, 2011/11/18 20:05:40]

can you add name= lines to these suppressions?

[Derek Bruening, 2011/12/07 14:53:59]

meant to do this next time I changed this file but hasn't happened yet.  Reid if
you touch this file before me can you add names.

[Timur Iskhodzhanov, 2011/12/07 15:01:29]

I already did that.

On 2011/12/07 14:53:59, Derek Bruening wrote:

+...
+*!BrowserThreadTest_NotReleasedIfTargetThreadNonExistent_Test::TestBody
+
+# This test has a death test at the end that derefences a NULL pointer.
+UNADDRESSABLE ACCESS
+...
+unit_tests.exe!BrowserAboutHandlerTest_WillHandleBrowserAboutURL_Test::TestBody

[Timur Iskhodzhanov, 2011/12/07 13:32:16]

Do you remember what was the stack?
This one matches the reports in http://crbug.com/96010 which is a different
issue

[Derek Bruening, 2011/12/07 14:53:59]

This diff came straight from Reid and I just posted it as he didn't have commit
access at the time.

Reid did you analyze this test and see a "death test"?  b/c 96010 is a real bug
not a "death test" it seems.

[Reid Kleckner, 2011/12/07 15:04:23]

Right:
http://codereview.chromium.org/8585046/
The last line of the test is:
  EXPECT_DEATH(HandleNonNavigationAboutURL(url), "");

Unfortunately I didn't link to any of the failing builds I was looking at at the
time.

+
 ############################
 # Benign issues in Chromium
 
@@ -758,11 +768,6 @@ LEAK
 *!TestURLRequestContextGetter::GetURLRequestContext
 *!URLFetcher::Core::StartURLRequest
 
-# This test intentionally leaks an object and checks that it's never deleted
-LEAK
-...
-*!BrowserThreadTest_NotReleasedIfTargetThreadNonExistent_Test::TestBody
-
 WARNING
 # http://code.google.com/p/chromium/issues/detail?id=72463
 name=Ci#72463: prefetches in generated MemCopy