Fix unload controller.

chrome/browser/devtools/devtools_window.cc

Index: chrome/browser/devtools/devtools_window.cc
diff --git a/chrome/browser/devtools/devtools_window.cc b/chrome/browser/devtools/devtools_window.cc
index a2d355ea3bc2bdb5fe9f10edec80d39e1141a8b1..0def0d275fb1578dc4115323278024b5825bb312 100644
--- a/chrome/browser/devtools/devtools_window.cc
+++ b/chrome/browser/devtools/devtools_window.cc
@@ -770,7 +770,8 @@ bool DevToolsWindow::NeedsToInterceptBeforeUnload(
     WebContents* contents) {
   DevToolsWindow* window =
       DevToolsWindow::GetInstanceForInspectedWebContents(contents);
-  return window && !window->intercepted_page_beforeunload_;
+  return window && !window->intercepted_page_beforeunload_ &&
+         window->life_stage_ != kClosing;

[Michael K. (Yandex Team), 2017/03/01 13:48:52]

On closing NeedsToInterceptBeforeUnload(..) shouldn't return true 'cause
InterceptPageBeforeUnload(..) isn't going to intercept beforeunload (See the
method InterceptPageBeforeUnload(..) above).

[dgozman, 2017/03/01 19:39:48]

Should we do |window->life_stage_ == kLoadCompleted| instead to match the method
above completely?

[Michael K. (Yandex Team), 2017/03/02 04:28:47]

I think you are right.
I'll make a new patchset.

 }
 
 // static