Disable TabStripModelTest.FastShutdown on Mac 64.

chrome/browser/ui/tabs/tab_strip_model_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/ui/tabs/tab_strip_model.h"
 
 #include <map>
 #include <string>
 
 #include "base/files/file_path.h"
@@ skipping 1691 matching lines @@
   // Now select the last tab.
   strip.ActivateTabAt(strip.count() - 1, true);
 
   // Now close the last tab. The next adjacent should be selected.
   strip.CloseWebContentsAt(strip.count() - 1, TabStripModel::CLOSE_NONE);
   EXPECT_EQ(page_d_contents, strip.GetWebContentsAt(strip.active_index()));
 
   strip.CloseAllTabs();
 }
 
-// This fails on linux when run with the rest of unit_tests (crbug.com/302156).
-#if defined(OS_LINUX)
+// This fails on Linux when run with the rest of unit_tests (crbug.com/302156)
+// and fails consistently on Mac 64.
+#if defined(OS_LINUX) || (defined(OS_MACOSX) && ARCH_CPU_64_BITS)
 #define MAYBE_FastShutdown \

[Nico, 2013/10/02 18:22:17]

Just disable it for all platforms maybe?

[Avi (use Gerrit), 2013/10/02 18:25:31]

I feel really awkward killing tests.

     DISABLED_FastShutdown
 #else
 #define MAYBE_FastShutdown \
     FastShutdown
 #endif
 // Tests that fast shutdown is attempted appropriately.
 TEST_F(TabStripModelTest, MAYBE_FastShutdown) {
   TabStripDummyDelegate delegate;
   TabStripModel tabstrip(&delegate, profile());
   MockTabStripModelObserver observer(&tabstrip);
@@ skipping 756 matching lines @@
   WebContents* moved_contents = strip_src.DetachWebContentsAt(1);
   EXPECT_EQ(contents2, moved_contents);
 
   // Attach the tab to the destination tab strip.
   strip_dst.AppendWebContents(moved_contents, true);
   EXPECT_TRUE(strip_dst.IsTabBlocked(0));
 
   strip_dst.CloseAllTabs();
   strip_src.CloseAllTabs();
 }