Ensures NOTIFICATION_APP_TERMINATING at the end of InProcessBrowserTest.

chrome/test/base/in_process_browser_test.cc

 // Copyright (c) 2012 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 "chrome/test/base/in_process_browser_test.h"
 
 #include "base/auto_reset.h"
 #include "base/basictypes.h"
 #include "base/bind.h"
 #include "base/command_line.h"
@@ skipping 440 matching lines @@
   QuitBrowsers();
   // All BrowserLists should be empty at this point.
   for (chrome::HostDesktopType t = chrome::HOST_DESKTOP_TYPE_FIRST;
        t < chrome::HOST_DESKTOP_TYPE_COUNT;
        t = static_cast<chrome::HostDesktopType>(t + 1)) {
     CHECK(BrowserList::GetInstance(t)->empty()) << t;
   }
 }
 
 void InProcessBrowserTest::QuitBrowsers() {
-  if (chrome::GetTotalBrowserCount() == 0)
+  if (chrome::GetTotalBrowserCount() == 0) {
+    chrome::NotifyAppTerminating();

[sky, 2013/09/05 15:33:45]

Might this result in sending out notification twice?

[Jun Mukai, 2013/09/05 18:46:56]

No, as far as I see.

Usually app-terminating will be emitted through chrome::AttemptExit() below, and
I cannot find no other code paths to emit it. I also injected a log into
NotifyAppTerminating(), and at least some test doesn't print the log.

It also seems that NotifyAppTerminating() uses a static boolean flag to avoid
notifying twice, so I think it's safe.

     return;
+  }
 
   // Invoke AttemptExit on a running message loop.
   // AttemptExit exits the message loop after everything has been
   // shut down properly.
   base::MessageLoopForUI::current()->PostTask(FROM_HERE,
                                               base::Bind(&chrome::AttemptExit));
   content::RunMessageLoop();
 
 #if defined(OS_MACOSX)
   // chrome::AttemptExit() will attempt to close all browsers by deleting
   // their tab contents. The last tab contents being removed triggers closing of
   // the browser window.
   //
   // On the Mac, this eventually reaches
   // -[BrowserWindowController windowWillClose:], which will post a deferred
   // -autorelease on itself to ultimately destroy the Browser object. The line
   // below is necessary to pump these pending messages to ensure all Browsers
   // get deleted.
   content::RunAllPendingInMessageLoop();
   delete autorelease_pool_;
   autorelease_pool_ = NULL;
 #endif
 }