DialogBrowserTest implementation to invoke Content settings bubble dialogs.

chrome/browser/ui/test/browser_dialog_browsertest.cc

 // Copyright 2016 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 "base/command_line.h"
 #include "base/process/launch.h"
 #include "base/test/launcher/test_launcher.h"
 #include "base/test/test_switches.h"
 #include "base/test/test_timeouts.h"
 #include "chrome/browser/ui/test/test_browser_dialog.h"
+#include "content/public/common/content_switches.h"
 #include "testing/gtest/include/gtest/gtest.h"
 #include "ui/compositor/compositor_switches.h"
 
 namespace {
 
 // Switch for BrowserDialogTest.Invoke to spawn a subprocess testing the
 // provided argument under a consistent setup.
 constexpr const char kDialogSwitch[] = "dialog";
 
 // Pattern to search in test names that indicate support for dialog testing.
@@ skipping 38 matching lines @@
 
   base::LaunchOptions options;
 
   // Disable timeouts and generate screen output if --interactive was specified.
   if (command.HasSwitch(internal::kInteractiveSwitch)) {
     command.AppendSwitchASCII(switches::kUiTestActionMaxTimeout,
                               TestTimeouts::kNoTimeoutSwitchValue);
     command.AppendSwitchASCII(switches::kTestLauncherTimeout,
                               TestTimeouts::kNoTimeoutSwitchValue);
     command.AppendSwitch(switches::kEnablePixelOutputInTests);
+#if defined(OS_WIN)
+    // Under Windows, pass in --disable-gpu for now due to
+    // http://crbug.com/687387.

[tapted, 2017/02/03 00:35:39]

There was a chromium-dev thread lately about not just citing bugs, but ensuring
there is enough context as well. E.g., the comment should mention something like

// Under Windows, dialogs (but not the browser window) created in the
// spawned browser_test process are invisible for an unknown reason. Pass
// --disable-gpu to resolve this for now. See http://crbug.com/687387.

[kylix_rd, 2017/02/03 18:55:03]

LOL! I just got nailed by my own pet-peeve... ;) Citing a bug reference only
without any summary context. Silly me.

[kylix_rd, 2017/02/03 18:55:03]

Done.

+    command.AppendSwitch(switches::kDisableGpu);
+#endif
   } else {
     options.wait = true;
   }
 
   base::LaunchProcess(command, options);
 }