Remove DumpWithoutCrashing from ShouldAllowOpenURL.

chrome/browser/extensions/window_open_apitest.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 <stddef.h>
 
 #include "base/path_service.h"
 #include "base/strings/stringprintf.h"
 #include "base/test/histogram_tester.h"
 #include "build/build_config.h"
@@ skipping 307 matching lines @@
   ASSERT_TRUE(newtab);
 
   EXPECT_NE(extension_url, newtab->GetMainFrame()->GetLastCommittedURL());
   EXPECT_FALSE(newtab->GetMainFrame()->GetSiteInstance()->GetSiteURL().SchemeIs(
       extensions::kExtensionScheme));
 
   // Verify that the blocking was recorded correctly in UMA.
   uma.ExpectUniqueSample("Extensions.ShouldAllowOpenURL.Failure",
                          2, /* FAILURE_SCHEME_NOT_HTTP_OR_HTTPS_OR_EXTENSION */
                          1);
+  uma.ExpectUniqueSample("Extensions.ShouldAllowOpenURL.Failure.Scheme",
+                         6 /* SCHEME_DATA */, 1);
 }
 
 // Test that navigating to an extension URL is allowed on chrome:// and
 // chrome-search:// pages, even for URLs that are not web-accessible.
 // See https://crbug.com/662602.
 IN_PROC_BROWSER_TEST_F(ExtensionBrowserTest,
                        NavigateToInaccessibleResourceFromChromeURL) {
   // Mint an extension URL which is not web-accessible.
   const extensions::Extension* extension = LoadExtension(
       test_data_dir_.AppendASCII("uitest").AppendASCII("window_open"));
@@ skipping 18 matching lines @@
     ASSERT_TRUE(content::ExecuteScript(
         tab, "location.href = '" + extension_url.spec() + "';"));
     observer.Wait();
     EXPECT_EQ(extension_url, tab->GetMainFrame()->GetLastCommittedURL());
     std::string result;
     ASSERT_TRUE(content::ExecuteScriptAndExtractString(
         tab, "domAutomationController.send(document.body.innerText)", &result));
     EXPECT_EQ("HOWDIE!!!", result);
   }
 }