Removed NOTIFICATION_BROWSER_CLOSING notification

chrome/browser/ui/browser_list_observer.h

 // 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.
 
 #ifndef CHROME_BROWSER_UI_BROWSER_LIST_OBSERVER_H_
 #define CHROME_BROWSER_UI_BROWSER_LIST_OBSERVER_H_
 
 class Browser;
 
 namespace chrome {
 
 class BrowserListObserver {
   public:
   // Called immediately after a browser is added to the list
   virtual void OnBrowserAdded(Browser* browser) {}
 
   // Called immediately after a browser is removed from the list
   virtual void OnBrowserRemoved(Browser* browser) {}
 
   // Called immediately after a browser is set active (SetLastActive)
   virtual void OnBrowserSetLastActive(Browser* browser) {}
 
   // Called immediately after a browser becomes not active.
   virtual void OnBrowserNoLongerActive(Browser* browser) {}
 
+  // Called when browser window is closing.
+  virtual void OnBrowserWindowClosing(Browser* browser) {}

[sky, 2017/03/31 14:36:54]

Please move OnBrowserRemoved after this (or move the two new functions you are
adding above it) and document the relation among these. Also, rename
OnBrowserWindowClosing to OnBrowserCloseStarting. I'm thinking something like:

// When a browser closes OnBrowserCloseStarting() is called. This is
// followed by one of OnBrowserCloseCanceled() or OnBrowserRemoved().
OnBrowserStartingClose()

[Alexey Seren, 2017/04/02 12:02:35]

Acknowledged.

+
+  // Called when browser window close is cancelled.
+  virtual void OnBrowserCloseCancelled(Browser* browser) {}
+
  protected:
   virtual ~BrowserListObserver() {}
 };
 
 }  // namespace chrome
 
 #endif  // CHROME_BROWSER_UI_BROWSER_LIST_OBSERVER_H_