Remove duplicated code in favor of IsRendererDebugURL.

content/browser/browser_url_handler_impl.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 "content/browser/browser_url_handler_impl.h"
 
 #include "base/command_line.h"
 #include "base/strings/string_util.h"
 #include "cc/base/switches.h"
 #include "content/browser/frame_host/debug_urls.h"
@@ skipping 69 matching lines @@
   // chrome:// scheme, since the about: scheme won't be rewritten in
   // this code path.
   if (CommandLine::ForCurrentProcess()->HasSwitch(
           cc::switches::kEnableGpuBenchmarking)) {
     if (HandleDebugURL(*url, PAGE_TRANSITION_FROM_ADDRESS_BAR)) {
       return true;
     }
   }
 
   // Circumvent processing URLs that the renderer process will handle.
-  return *url == GURL(kChromeUICrashURL) ||

[Charlie Reis, 2014/02/12 18:50:28]

Technically this did not return true for javascript: URLs originally, but those
fall in the same spirit of the comment and I tested that they still work.

-         *url == GURL(kChromeUIHangURL) ||
-         *url == GURL(kChromeUIKillURL) ||
-         *url == GURL(kChromeUIShorthangURL);
+  return IsRendererDebugURL(*url);
 }
 
 // static
 BrowserURLHandler* BrowserURLHandler::GetInstance() {
   return BrowserURLHandlerImpl::GetInstance();
 }
 
 // static
 BrowserURLHandler::URLHandler BrowserURLHandler::null_handler() {
   // Required for VS2010: http://connect.microsoft.com/VisualStudio/feedback/details/520043/error-converting-from-null-to-a-pointer-type-in-std-pair
@@ skipping 47 matching lines @@
           return true;
       } else if (handler(&test_url, browser_context)) {
         return reverse_rewriter(url, browser_context);
       }
     }
   }
   return false;
 }
 
 }  // namespace content