Move page closing hang timer duties to its own timer.

content/browser/frame_host/render_frame_host_manager.h

 // Copyright 2013 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 CONTENT_BROWSER_FRAME_HOST_RENDER_FRAME_HOST_MANAGER_H_
 #define CONTENT_BROWSER_FRAME_HOST_RENDER_FRAME_HOST_MANAGER_H_
 
 #include <stdint.h>
 
 #include <list>
@@ skipping 262 matching lines @@
 
   // Instructs the various live views to stop. Called when the user directed the
   // page to stop loading.
   void Stop();
 
   // Notifies the regular and pending RenderViewHosts that a load is or is not
   // happening. Even though the message is only for one of them, we don't know
   // which one so we tell both.
   void SetIsLoading(bool is_loading);
 
-  // Whether to close the tab or not when there is a hang during an unload
-  // handler. If we are mid-crosssite navigation, then we should proceed
-  // with the navigation instead of closing the tab.
-  bool ShouldCloseTabOnUnresponsiveRenderer();

[Charlie Reis, 2017/03/03 01:05:59]

Hooray!  This is exactly what I hoped to get out of this change.

[Avi (use Gerrit), 2017/03/03 01:25:25]

Acknowledged.

-
   // Confirms whether we should close the page or navigate away.  This is called
   // before a cross-site request or before a tab/window is closed (as indicated
   // by the first parameter) to allow the appropriate renderer to approve or
   // deny the request.  |proceed| indicates whether the user chose to proceed.
   // |proceed_time| is the time when the request was allowed to proceed.
   void OnBeforeUnloadACK(bool for_cross_site_transition,
                          bool proceed,
                          const base::TimeTicks& proceed_time);
 
   // The |pending_render_frame_host| is ready to commit a page.  We should
@@ skipping 511 matching lines @@
   std::unique_ptr<RenderFrameHostImpl> speculative_render_frame_host_;
 
   base::WeakPtrFactory<RenderFrameHostManager> weak_factory_;
 
   DISALLOW_COPY_AND_ASSIGN(RenderFrameHostManager);
 };
 
 }  // namespace content
 
 #endif  // CONTENT_BROWSER_FRAME_HOST_RENDER_FRAME_HOST_MANAGER_H_