mac: Disable parts of ProcessInfoSnapshotMacTest on OSX 10.9+.

chrome/browser/process_info_snapshot_mac_unittest.cc

 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "chrome/browser/process_info_snapshot.h"
 
 #include <sys/types.h>  // For |uid_t| (and |pid_t|).
 #include <unistd.h>  // For |getpid()|, |getuid()|, etc.
 
 #include <vector>
 
 #include "base/command_line.h"
 #include "base/files/file_path.h"
 #include "base/logging.h"
+#include "base/mac/mac_util.h"
 #include "base/posix/eintr_wrapper.h"
 #include "base/process/kill.h"
 #include "base/process/launch.h"
 #include "base/process/process_handle.h"
 #include "base/process/process_metrics.h"
 #include "testing/gtest/include/gtest/gtest.h"
 
 typedef testing::Test ProcessInfoSnapshotMacTest;
 
 TEST_F(ProcessInfoSnapshotMacTest, FindPidOneTest) {
@@ skipping 45 matching lines @@
   ProcessInfoSnapshot snapshot;
   ASSERT_TRUE(snapshot.Sample(pid_list));
 
   // Find our process.
   ProcessInfoSnapshot::ProcInfoEntry proc_info;
   ASSERT_TRUE(snapshot.GetProcInfo(pid, &proc_info));
   EXPECT_EQ(pid, proc_info.pid);
   EXPECT_EQ(ppid, proc_info.ppid);
   EXPECT_EQ(uid, proc_info.uid);
   EXPECT_EQ(euid, proc_info.euid);
-  EXPECT_GE(proc_info.rss, 100u);     // Sanity check: we're running, so we
-                                      // should occupy at least 100 kilobytes.
-  EXPECT_GE(proc_info.vsize, 1024u);  // Sanity check: our |vsize| is presumably
-                                      // at least a megabyte.
-  EXPECT_GE(proc_info.rshrd, 1024u);  // Shared memory should also > 1 MB.
-  EXPECT_GE(proc_info.rprvt, 1024u);  // Same with private memory.
+  // Sanity check: we're running, so we should occupy at least 100 kilobytes.
+  EXPECT_GE(proc_info.rss, 100u);
+  // Sanity check: our |vsize| is presumably at least a megabyte.
+  EXPECT_GE(proc_info.vsize, 1024u);
+
+  // Collection of some memory statistics is broken in OSX 10.9+.
+  // http://crbug.com/383553
+  if (!base::mac::IsOSMavericksOrLater()) {
+    // Shared memory should also > 1 MB.
+    EXPECT_GE(proc_info.rshrd, 1024u);
+    // Same with private memory.
+    EXPECT_GE(proc_info.rprvt, 1024u);
+  }
 
   // Find our parent.
   ASSERT_TRUE(snapshot.GetProcInfo(ppid, &proc_info));
   EXPECT_EQ(ppid, proc_info.pid);
   EXPECT_NE(static_cast<int64>(proc_info.ppid), 0);
   EXPECT_EQ(uid, proc_info.uid);    // This (and the following) should be true
   EXPECT_EQ(euid, proc_info.euid);  // under reasonable circumstances.
   // Can't say anything definite about its |rss|.
   EXPECT_GT(proc_info.vsize, 0u);   // Its |vsize| should be nonzero though.
 }
@@ skipping 35 matching lines @@
   ProcessInfoSnapshot::ProcInfoEntry proc_info;
   ASSERT_TRUE(snapshot.GetProcInfo(process_handle, &proc_info));
   // Effective user ID should be 0 (root).
   EXPECT_EQ(proc_info.euid, 0u);
   // Real user ID should match the calling process's user id.
   EXPECT_EQ(proc_info.uid, geteuid());
 
   ASSERT_TRUE(base::KillProcess(process_handle, 0, true));
   PCHECK(IGNORE_EINTR(close(fds[0])) == 0);
 }