Fixed flakiness of context_lost tests on GPU bots.

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/strings/utf_string_conversions.h"
 #include "content/browser/gpu/gpu_process_host_ui_shim.h"
@@ skipping 62 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) {
+  return IsRendererDebugURL(url) ||
+      (url.is_valid() &&
+       (url.host() == kChromeUIBrowserCrashHost ||

[Charlie Reis, 2014/05/09 23:21:51]

I worry about this getting stale when people add things to HandleDebugURL but
not here.  Maybe there's a good place to add a comment to make sure both get
updated together?

[Ken Russell (switch to Gerrit), 2014/05/09 23:43:05]

Added comments to both HandleDebugURL and IsDebugURL.

+        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(kJavaScriptScheme))
     return true;
 
   return url == GURL(kChromeUICrashURL) ||
          url == GURL(kChromeUIKillURL) ||
          url == GURL(kChromeUIHangURL) ||
          url == GURL(kChromeUIShorthangURL);
 }
 
 }  // namespace content