Do not generate crash dumps in CrashRecoveryBrowserTest.

chrome/browser/crash_recovery_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/command_line.h"
 #include "base/files/file_path.h"
 #include "base/strings/stringprintf.h"
 #include "chrome/browser/ui/browser.h"
 #include "chrome/browser/ui/browser_commands.h"
 #include "chrome/browser/ui/tabs/tab_strip_model.h"
+#include "chrome/common/chrome_switches.h"
 #include "chrome/common/url_constants.h"
 #include "chrome/test/base/in_process_browser_test.h"
 #include "chrome/test/base/ui_test_utils.h"
 #include "content/public/browser/notification_service.h"
 #include "content/public/browser/notification_types.h"
 #include "content/public/browser/web_contents.h"
 #include "content/public/common/page_transition_types.h"
 #include "content/public/test/browser_test_utils.h"
 #include "net/test/embedded_test_server/embedded_test_server.h"
 #include "net/test/embedded_test_server/http_request.h"
 #include "net/test/embedded_test_server/http_response.h"
 #include "testing/gtest/include/gtest/gtest.h"
 
-using content::NavigationController;
 using content::OpenURLParams;
 using content::Referrer;
 using content::WebContents;
 
 // TODO(jam): http://crbug.com/350550
 #if !(defined(OS_CHROMEOS) && defined(ADDRESS_SANITIZER))
 
 namespace {
 
 void SimulateRendererCrash(Browser* browser) {
@@ skipping 23 matching lines @@
         new net::test_server::BasicHttpResponse);
     response->set_content(base::StringPrintf("<title>%d</title>",
                                              request_count_));
     response->set_content_type("text/html");
     response->AddCustomHeader("Cache-Control", "max-age=99999");
     return response.PassAs<net::test_server::HttpResponse>();
   }
  private:
   std::string path_;
   int request_count_;
+
+  DISALLOW_COPY_AND_ASSIGN(CacheMaxAgeHandler);
 };
 
-}  // namespace
-
 class CrashRecoveryBrowserTest : public InProcessBrowserTest {
  protected:
   WebContents* GetActiveWebContents() {
     return browser()->tab_strip_model()->GetActiveWebContents();
   }
+
+ private:
+  virtual void SetUpCommandLine(base::CommandLine* command_line) OVERRIDE {
+    command_line->AppendSwitch(switches::kDisableBreakpad);

[Paweł Hajdan Jr., 2014/05/22 07:00:27]

Don't we want to disable this for all browser tests? If applicable it'd be good
to at this flag at content layer level.

[Lei Zhang, 2014/05/22 07:21:07]

Several years ago, Chromium builds didn't have Breakpad, so this wasn't a
problem.

Now all builds have Breakpad and it's enabled for bots because of
CHROME_HEADLESS. CHROME_HEADLESS also implies no upload, so the files go in the
"Crash Reports" directory. The problem with passing this switch everywhere is
that there's no way to remove the switch. So if a test does need it, we can't
enable Breakpad.

This means when tests (besides this one) crash for any reason, we'd get a few
crash dumps. Thus we have to clean the directory after the test finishes. I
think this is ok, if we just enabled tools/build/scripts/slave/process_dumps.py
for Linux to process and delete the dumps like we do on Windows.

If any case, we certainly don't want the dumps from this test, they are boring
and we don't really want to process them anyway.

+  }
 };
 
 // Test that reload works after a crash.
 // Disabled, http://crbug.com/29331 , http://crbug.com/69637 .
 IN_PROC_BROWSER_TEST_F(CrashRecoveryBrowserTest, Reload) {
   // The title of the active tab should change each time this URL is loaded.
   GURL url(
       "data:text/html,<script>document.title=new Date().valueOf()</script>");
   ui_test_utils::NavigateToURL(browser(), url);
 
@@ skipping 34 matching lines @@
   ASSERT_TRUE(ui_test_utils::GetCurrentTabTitle(browser(),
                                                 &title_after_crash));
   EXPECT_NE(title_before_crash, title_after_crash);
 }
 
 // Tests that loading a crashed page in a new tab correctly updates the title.
 // There was an earlier bug (1270510) in process-per-site in which the max page
 // ID of the RenderProcessHost was stale, so the NavigationEntry in the new tab
 // was not committed.  This prevents regression of that bug.
 IN_PROC_BROWSER_TEST_F(CrashRecoveryBrowserTest, LoadInNewTab) {
-  const base::FilePath::CharType* kTitle2File =
+  const base::FilePath::CharType kTitle2File[] =
       FILE_PATH_LITERAL("title2.html");
 
   ui_test_utils::NavigateToURL(
       browser(), ui_test_utils::GetTestUrl(
                      base::FilePath(base::FilePath::kCurrentDirectory),
                      base::FilePath(kTitle2File)));
 
   base::string16 title_before_crash;
   base::string16 title_after_crash;
 
@@ skipping 18 matching lines @@
 
   chrome::Reload(browser(), CURRENT_TAB);
   content::WaitForLoadStop(GetActiveWebContents());
   ASSERT_EQ(url, GetActiveWebContents()->GetVisibleURL());
 
   chrome::Reload(browser(), CURRENT_TAB);
   content::WaitForLoadStop(GetActiveWebContents());
   ASSERT_EQ(url, GetActiveWebContents()->GetVisibleURL());
 }
 
+}  // namespace
+
 #endif