Removed NOTIFICATION_BROWSER_CLOSING notification

chrome/browser/ui/browser_commands.cc

Index: chrome/browser/ui/browser_commands.cc
diff --git a/chrome/browser/ui/browser_commands.cc b/chrome/browser/ui/browser_commands.cc
index efcc39cc4d372d674a7f6f303bf3f5334abefc80..2ae9d8f28c2b5c06e4755251ee9a965c950560a0 100644
--- a/chrome/browser/ui/browser_commands.cc
+++ b/chrome/browser/ui/browser_commands.cc
@@ -11,6 +11,7 @@
 #include "chrome/app/chrome_command_ids.h"
 #include "chrome/browser/bookmarks/bookmark_model_factory.h"
 #include "chrome/browser/browser_process.h"
+#include "chrome/browser/browser_shutdown.h"
 #include "chrome/browser/browsing_data/browsing_data_helper.h"
 #include "chrome/browser/browsing_data/browsing_data_remover.h"
 #include "chrome/browser/browsing_data/browsing_data_remover_factory.h"
@@ -756,6 +757,10 @@ void ConvertPopupToTabbedBrowser(Browser* browser) {
 
 void Exit() {
   content::RecordAction(UserMetricsAction("Exit"));
+// On Mac, the platform-specific part handles setting this.
+#if !defined(OS_MACOSX)
+  browser_shutdown::SetTryingToQuit(true);

[Alexey Seren, 2017/04/09 17:51:37]

By comments for SetTryingToQuit function we should call SetTryingToQuit(true)
when the User explicitly chooses to shutdown the app (via the "Exit" or "Quit"
menu items). See:
https://cs.chromium.org/chromium/src/chrome/browser/browser_shutdown.h?q=brow...

[hashimoto, 2017/04/14 08:54:37]

IIUC the comment discourages to call SetTryingToQuit(true) directly.

In https://codereview.chromium.org/25603004, they intentionally stopped calling
SetTryingToQuit(true) for AttemptUserExit().
I'm not still sure if it's still relevant.

+#endif
   chrome::AttemptUserExit();
 }