Prevent duplicate navigation to debug URLs from Telemetry.

content/browser/frame_host/debug_urls.cc

 // Copyright 2013 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/frame_host/debug_urls.h"
 
 #include <vector>
 
+#include "base/command_line.h"
 #include "base/debug/asan_invalid_access.h"
 #include "base/debug/profiler.h"
 #include "base/strings/utf_string_conversions.h"
+#include "cc/base/switches.h"
 #include "content/browser/gpu/gpu_process_host_ui_shim.h"
 #include "content/browser/ppapi_plugin_process_host.h"
 #include "content/public/browser/browser_thread.h"
 #include "content/public/common/content_constants.h"
 #include "content/public/common/url_constants.h"
 #include "ppapi/proxy/ppapi_messages.h"
 #include "url/gurl.h"
 
 namespace content {
 
@@ skipping 77 matching lines @@
   }
 #endif
 
   return true;
 }
 
 
 }  // namespace
 
 bool HandleDebugURL(const GURL& url, PageTransition transition) {
-  // Ensure that the user explicitly navigated to this URL.
-  if (!(transition & PAGE_TRANSITION_FROM_ADDRESS_BAR))
+  // Ensure that the user explicitly navigated to this URL, unless
+  // kEnableGpuBenchmarking is enabled by Telemetry.
+  bool is_telemetry_navigation = CommandLine::ForCurrentProcess()->HasSwitch(
+                                     cc::switches::kEnableGpuBenchmarking) &&
+                                 (transition & PAGE_TRANSITION_TYPED);
+
+  if (!(transition & PAGE_TRANSITION_FROM_ADDRESS_BAR) &&
+      !is_telemetry_navigation)
     return false;
 
-  // NOTE: when you add handling of any URLs to this function, also
-  // update IsDebugURL, below.
-
   if (IsAsanDebugURL(url))
     return HandleAsanDebugURL(url);
 
   if (url.host() == kChromeUIBrowserCrashHost) {
     // Induce an intentional crash in the browser process.
     CHECK(false);
     return true;
   }
 
   if (url == GURL(kChromeUIGpuCleanURL)) {
@@ skipping 20 matching lines @@
   if (url == GURL(kChromeUIPpapiFlashCrashURL) ||
       url == GURL(kChromeUIPpapiFlashHangURL)) {
     BrowserThread::PostTask(BrowserThread::IO, FROM_HERE,
                             base::Bind(&HandlePpapiFlashDebugURL, url));
     return true;
   }
 
   return false;
 }
 
-bool IsDebugURL(const GURL& url) {
-  // NOTE: when you add any URLs to this list, also update
-  // HandleDebugURL, above.
-  return IsRendererDebugURL(url) || IsAsanDebugURL(url) ||
-      (url.is_valid() &&
-       (url.host() == kChromeUIBrowserCrashHost ||
-        url == GURL(kChromeUIGpuCleanURL) ||
-        url == GURL(kChromeUIGpuCrashURL) ||
-        url == GURL(kChromeUIGpuHangURL) ||
-        url == GURL(kChromeUIPpapiFlashCrashURL) ||
-        url == GURL(kChromeUIPpapiFlashHangURL)));
-}
-
 bool IsRendererDebugURL(const GURL& url) {
   if (!url.is_valid())
     return false;
 
   if (url.SchemeIs(url::kJavaScriptScheme))
     return true;
 
   return url == GURL(kChromeUICrashURL) ||
          url == GURL(kChromeUIDumpURL) ||
          url == GURL(kChromeUIKillURL) ||
          url == GURL(kChromeUIHangURL) ||
          url == GURL(kChromeUIShorthangURL);
 }
 
 }  // namespace content