Add IndexedDB browser test that causes a crash.

content/browser/in_process_webkit/indexed_db_browsertest.cc

 // Copyright (c) 2010 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/command_line.h"
 #include "base/file_path.h"
 #include "base/file_util.h"
 #include "base/ref_counted.h"
 #include "base/scoped_temp_dir.h"
 #include "base/utf_string_conversions.h"
 #include "chrome/browser/ui/browser.h"
+#include "chrome/common/chrome_switches.h"
 #include "chrome/test/in_process_browser_test.h"
 #include "chrome/test/testing_profile.h"
 #include "chrome/test/thread_test_helper.h"
 #include "chrome/test/ui_test_utils.h"
 #include "content/browser/in_process_webkit/indexed_db_context.h"
 #include "content/browser/in_process_webkit/webkit_context.h"
 #include "content/browser/tab_contents/tab_contents.h"
 
 // This browser test is aimed towards exercising the IndexedDB bindings and
 // the actual implementation that lives in the browser side (in_process_webkit).
@@ skipping 17 matching lines @@
     LOG(INFO) << "Navigation done.";
     std::string result = browser()->GetSelectedTabContents()->GetURL().ref();
     if (result != "pass") {
       std::string js_result;
       ASSERT_TRUE(ui_test_utils::ExecuteJavaScriptAndExtractString(
           browser()->GetSelectedTabContents()->render_view_host(), L"",
           L"window.domAutomationController.send(getLog())", &js_result));
       FAIL() << "Failed: " << js_result;
     }
   }
+  virtual void SetUpCommandLine(CommandLine* command_line) {
+    command_line->AppendSwitchASCII(switches::kJavaScriptFlags, "--expose-gc");

[jorlow, 2011/03/15 02:30:04]

Let's only do this on tests we have to.  I've seen some bugs not show up when
it's on.

[dgrogan, 2011/03/15 03:32:27]

Sounds good.  Is it easy to add this per test in this class or do I need to make
a new class that inherits from InProcessBrowserTest?

[jorlow, 2011/03/15 03:35:37]

I believe you just need another text fixture that inherits from this one.

[Paweł Hajdan Jr., 2011/03/15 19:49:11]

Can we create a constant for --expose-gc?

[jorlow, 2011/03/15 20:25:46]

The only reason to bother making a constant is if we replace all instances of
the string with it.  Does v8 expose its constant for this in any way?  If not,
then making it a constant really doesn't buy us anything.

[dgrogan, 2011/03/16 22:00:12]

Done.

+  }
 };
 
 IN_PROC_BROWSER_TEST_F(IndexedDBBrowserTest, CursorTest) {
   SimpleTest(testUrl(FilePath(FILE_PATH_LITERAL("cursor_test.html"))));
 }
 
 // Flaky: http://crbug.com/70773
 IN_PROC_BROWSER_TEST_F(IndexedDBBrowserTest, DISABLED_IndexTest) {
   SimpleTest(testUrl(FilePath(FILE_PATH_LITERAL("index_test.html"))));
 }
@@ skipping 14 matching lines @@
 
 IN_PROC_BROWSER_TEST_F(IndexedDBBrowserTest, DatabaseTest) {
   SimpleTest(testUrl(FilePath(FILE_PATH_LITERAL("database_test.html"))));
 }
 
 // Flaky: http://crbug.com/70773
 IN_PROC_BROWSER_TEST_F(IndexedDBBrowserTest, DISABLED_TransactionTest) {
   SimpleTest(testUrl(FilePath(FILE_PATH_LITERAL("transaction_test.html"))));
 }
 
+IN_PROC_BROWSER_TEST_F(IndexedDBBrowserTest, DatabaseCallbacksTest) {
+  SimpleTest(
+      testUrl(FilePath(FILE_PATH_LITERAL("database_callbacks_first.html"))));
+}
+
 // Flaky: http://crbug.com/70773
 IN_PROC_BROWSER_TEST_F(IndexedDBBrowserTest, DISABLED_DoesntHangTest) {
   SimpleTest(testUrl(FilePath(
       FILE_PATH_LITERAL("transaction_run_forever.html"))));
   ui_test_utils::CrashTab(browser()->GetSelectedTabContents());
   SimpleTest(testUrl(FilePath(FILE_PATH_LITERAL("transaction_test.html"))));
 }
 
 // In proc browser test is needed here because ClearLocalState indirectly calls
 // WebKit's isMainThread through WebSecurityOrigin->SecurityOrigin.
@@ skipping 26 matching lines @@
   // Make sure we wait until the destructor has run.
   scoped_refptr<ThreadTestHelper> helper(
       new ThreadTestHelper(BrowserThread::WEBKIT));
   ASSERT_TRUE(helper->Run());
 
   // Because we specified https for scheme to be skipped the second file
   // should survive and the first go into vanity.
   ASSERT_FALSE(file_util::PathExists(temp_file_path_1));
   ASSERT_TRUE(file_util::PathExists(temp_file_path_2));
 }