Prevent duplicate navigation to debug URLs from Telemetry.

content/browser/browser_url_handler_impl.cc

Index: content/browser/browser_url_handler_impl.cc
diff --git a/content/browser/browser_url_handler_impl.cc b/content/browser/browser_url_handler_impl.cc
index 1b7050f8ddf1d8d687670713515da9ba3a9bf3a7..43b4e3f9554828fbe9f682e616c4a76b98926ee4 100644
--- a/content/browser/browser_url_handler_impl.cc
+++ b/content/browser/browser_url_handler_impl.cc
@@ -73,22 +73,6 @@ static bool ReverseViewSource(GURL* url, BrowserContext* browser_context) {
   return true;
 }
 
-static bool DebugURLHandler(GURL* url, BrowserContext* browser_context) {
-  // If running inside the Telemetry test harness, allow automated
-  // navigations to access browser-side debug URLs. They must use the
-  // 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 IsRendererDebugURL(*url);

[Charlie Reis, 2014/07/23 23:04:17]

Why is this ok to delete?

[Ken Russell (switch to Gerrit), 2014/07/23 23:08:43]

Yes, that's a good point. This should instead be reverted back to its state from
before crrev.com/27603004.

[vmiura, 2014/07/23 23:39:59]

Done.  I'm not sure what that's there for, but I don't see a need to remove it.

-}
-
 // static
 BrowserURLHandler* BrowserURLHandler::GetInstance() {
   return BrowserURLHandlerImpl::GetInstance();
@@ -106,8 +90,6 @@ BrowserURLHandlerImpl* BrowserURLHandlerImpl::GetInstance() {
 }
 
 BrowserURLHandlerImpl::BrowserURLHandlerImpl() {
-  AddHandlerPair(&DebugURLHandler, BrowserURLHandlerImpl::null_handler());
-
   GetContentClient()->browser()->BrowserURLHandlerCreated(this);
 
   // view-source: