Mark RendererCrashTest.Crash test as flaky on ChromiumOS

chrome/common/logging_chrome_uitest.cc

 // Copyright (c) 2011 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 "build/build_config.h"
 
 #if defined(OS_WIN)
 #include <windows.h>
 #endif
 
@@ skipping 141 matching lines @@
 #if defined(OS_LINUX) && !defined(USE_LINUX_BREAKPAD)
 // On Linux, do not expect a crash dump if Breakpad is disabled.
 #define EXPECTED_CRASH_CRASHES 0
 #else
 #define EXPECTED_CRASH_CRASHES 1
 #endif
 
 #if defined(OS_MACOSX)
 // Crash service doesn't exist for the Mac yet: http://crbug.com/45243
 #define Crash DISABLED_Crash
+#elif defined(OS_CHROME)
+#define Crash FLAKY_Crash
 #endif
 // Launch the app in renderer crash test mode, then close the app.
 TEST_F(RendererCrashTest, Crash) {

[grt (UTC plus 2), 2011/12/12 18:33:27]

would you change this and the #defines above to MAYBE_Crash?  i like the
look/feel of this convention.

[willchan no longer on Chromium, 2011/12/12 18:38:22]

Done.

   scoped_refptr<BrowserProxy> browser(automation()->GetBrowserWindow(0));
   ASSERT_TRUE(browser.get());
   ASSERT_TRUE(browser->WaitForTabCountToBecome(1));
   expected_crashes_ = EXPECTED_CRASH_CRASHES;
 }