Broaden valgrind supression for known issue in zlib. This seems ok be…

…cause 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
Denger-Network · Jun 1, 2009 · 94bdde1 · 94bdde1
1 parent fa922b1
commit 94bdde1
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 6 deletions.
diff --git a/chrome/common/zip_unittest.cc b/chrome/common/zip_unittest.cc
@@ -75,9 +75,7 @@ class ZipTest : public PlatformTest {
   // hard-coded contents of a known zip file
   std::set<FilePath> zip_contents_;
 };
-// TODO(aa): re-enable this test for Mac (was disabled temporarily to clear
-// valgrind error on mac.
-#if !defined(OS_MACOSX)
+
 TEST_F(ZipTest, Unzip) {
   TestUnzipFile(FILE_PATH_LITERAL("test.zip"));
 }
@@ -103,5 +101,5 @@ TEST_F(ZipTest, Zip) {
 
   EXPECT_TRUE(file_util::Delete(zip_file, false));
 }
-#endif // !defined(OS_MACOSX)
+
 }  // namespace
diff --git a/tools/valgrind/suppressions.txt b/tools/valgrind/suppressions.txt
@@ -14,8 +14,7 @@
 {
    Uninitialized value in deflate
    Memcheck:Cond
-   fun:longest_match
-   fun:deflate_slow
+   ...
    fun:MOZ_Z_deflate
 }
 {