Close all windows in test controller shutdown

content/shell/browser/layout_test/blink_test_controller.cc

Index: content/shell/browser/layout_test/blink_test_controller.cc
diff --git a/content/shell/browser/layout_test/blink_test_controller.cc b/content/shell/browser/layout_test/blink_test_controller.cc
index 5b9d1827b01c0545368c4da9f9c521e23baae861..8cc45e38356df75f7a456ef8bfe512e34d9da8ae 100644
--- a/content/shell/browser/layout_test/blink_test_controller.cc
+++ b/content/shell/browser/layout_test/blink_test_controller.cc
@@ -255,6 +255,7 @@ BlinkTestController::~BlinkTestController() {
   CHECK(test_phase_ == BETWEEN_TESTS);
   GpuDataManager::GetInstance()->RemoveObserver(this);
   DiscardMainWindow();
+  Shell::CloseAllWindows();

[Peter Beverloo, 2016/09/22 12:21:40]

Maybe we should consider moving this to LayoutTestBrowserMain() in
//content/shell/browser/layout_test/layout_test_browser_main.cc, since we do it
there for the --check-layout-test-sys-deps flag too.

We'll have to keep an eye out on how the Android bot reacts- that function,
together with RunTests(), shut down the main runner at subtly different times.
(Android doesn't run the message loop before shutting it down all other
platforms do.)

This change means that difference will go away, since CloseAllWindows() does run
the message loop as well. We may be able to delete that difference if it goes
well.

Unfortunately none of this is covered by the CQ, but I'll be around in the next
few days to check the bot too.

   instance_ = NULL;
 }