Disable some flaky IndexedDB browser tests.

content/browser/in_process_webkit/indexed_db_layout_browsertest.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 "base/file_path.h"
 #include "content/test/layout_browsertest.h"
 
 class IndexedDBLayoutTest : public InProcessBrowserLayoutTest {
  public:
   IndexedDBLayoutTest() : InProcessBrowserLayoutTest(
@@ skipping 21 matching lines @@
   "basics-workers.html",
   "database-basics.html",
   "factory-basics.html",
   "index-basics.html",
   "objectstore-basics.html",
   NULL
 };
 
 static const char* kComplexTests[] = {
   "prefetch-bugfix-108071.html",
-  "pending-version-change-stuck-works-with-terminate.html",
+  // Flaky: http://crbug.com/123685

[jsbell, 2012/04/30 22:48:54]

Formatting-wise, take a look at content/browser/appcache/appcache_browsertest.cc
- possibly more verbose than needed, but very readable.

Otherwise, based on content/browser/worker_host/test/worker_browsertest.cc ...

+//  "pending-version-change-stuck-works-with-terminate.html",

[jsbell, 2012/04/30 22:48:54]

leading // should be indented 2 spaces to align with " on non-commented line,
and followed by 1 space before "

[dgrogan, 2012/05/01 00:46:08]

Done.

   NULL
 };
 
 static const char* kIndexTests[] = {
   "deleteIndex.html",
+  // Flaky: http://crbug.com/123685
+//   "index-basics-workers.html",

[jsbell, 2012/04/30 22:48:54]

Same formatting as above: 2 spaces before //, 1 after

[dgrogan, 2012/05/01 00:46:08]

Done.

   "index-count.html",
   "index-cursor.html",  // Locally takes ~6s compared to <1 for the others.
   "index-get-key-argument-required.html",
   "index-multientry.html",
   "index-population.html",
   "index-unique.html",
   NULL
 };
 
 static const char* kKeyTests[] = {
@@ skipping 35 matching lines @@
 }
 
 IN_PROC_BROWSER_TEST_F(IndexedDBLayoutTest, BasicTests) {
   RunLayoutTests(kBasicTests);
 }
 
 IN_PROC_BROWSER_TEST_F(IndexedDBLayoutTest, ComplexTests) {
   RunLayoutTests(kComplexTests);
 }
 
-// Frequently times out, sometimes due to webkit assertion failure.
-// http://crbug.com/120924

[jsbell, 2012/04/30 22:48:54]

Is 120924 a duplicate of 123685? If not, keep this bug comment somewhere near
index-basics-workers.html reference.

[dgrogan, 2012/05/01 00:46:08]

It was kind of a duplicate.  I closed it in any case so we don't need to refer
to it anymore.

120924 was originally about only index-basics-workers.html flakiness, of which
there were two causes. One is fixed, the other seems like the general flakiness
described by 123685.  So I decided to interpret it as being just about the cause
that is now fixed, and closed it.

-IN_PROC_BROWSER_TEST_F(IndexedDBLayoutTest, FAILS_IndexBasicsWorkersTest) {
-  RunLayoutTest("deleteIndex.html");
-  RunLayoutTest("index-basics-workers.html");
-}
-
 IN_PROC_BROWSER_TEST_F(IndexedDBLayoutTest, IndexTests) {
   RunLayoutTests(kIndexTests);
 }
 
 IN_PROC_BROWSER_TEST_F(IndexedDBLayoutTest, KeyTests) {
   RunLayoutTests(kKeyTests);
 }
 
 IN_PROC_BROWSER_TEST_F(IndexedDBLayoutTest, TransactionTests) {
   RunLayoutTests(kTransactionTests);
 }
 
 IN_PROC_BROWSER_TEST_F(IndexedDBLayoutTest, RegressionTests) {
   RunLayoutTests(kRegressionTests);
 }