Removed NOTIFICATION_BROWSER_CLOSING notification

chrome/browser/ui/browser_list_observer.h

Index: chrome/browser/ui/browser_list_observer.h
diff --git a/chrome/browser/ui/browser_list_observer.h b/chrome/browser/ui/browser_list_observer.h
index 42af6d3086e65261be084bbf6d229111a5fb82cf..7de84c8eac641dbee8d393b952b20b03eebfd4f0 100644
--- a/chrome/browser/ui/browser_list_observer.h
+++ b/chrome/browser/ui/browser_list_observer.h
@@ -23,6 +23,12 @@ class BrowserListObserver {
   // 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() {}
 };