Refactor sudden termination

content/renderer/render_frame_impl.cc

Index: content/renderer/render_frame_impl.cc
diff --git a/content/renderer/render_frame_impl.cc b/content/renderer/render_frame_impl.cc
index 8fa21dc458b1cac2fe80adfc54126e31a35d352e..583ec7da7856ab026a9f85e0bb3597b8b448e15f 100644
--- a/content/renderer/render_frame_impl.cc
+++ b/content/renderer/render_frame_impl.cc
@@ -691,6 +691,8 @@ RenderFrameImpl::RenderFrameImpl(RenderViewImpl* render_view, int routing_id)
       manifest_manager_(NULL),
       accessibility_mode_(AccessibilityModeOff),
       renderer_accessibility_(NULL),
+      beforeunload_handlers_present_(false),
+      unload_handlers_present_(false),
       weak_factory_(this) {
   std::pair<RoutingIDFrameMap::iterator, bool> result =
       g_routing_id_frame_map.Get().insert(std::make_pair(routing_id_, this));
@@ -3566,6 +3568,25 @@ bool RenderFrameImpl::exitFullscreen() {
   return true;
 }
 
+void RenderFrameImpl::suddenTerminationDisablerChanged(
+    bool present,
+    blink::WebFrameClient::SuddenTerminationDisablerType type) {
+  switch (type) {
+    case blink::WebFrameClient::BeforeUnloadHandler:
+      beforeunload_handlers_present_ = present;

[Charlie Reis, 2015/01/29 23:20:42]

We don't need these members anymore, do we?  We're just sending |present| to the
browser any time we hear it from Blink now.

[clamy, 2015/02/02 16:16:27]

Done.

+      Send(new FrameHostMsg_BeforeUnloadHandlersPresent(
+          routing_id_, beforeunload_handlers_present_));
+      break;
+    case blink::WebFrameClient::UnloadHandler:
+      unload_handlers_present_ = present;
+      Send(new FrameHostMsg_UnloadHandlersPresent(
+          routing_id_, unload_handlers_present_));
+      break;
+    default:
+      NOTREACHED();
+  }
+}
+
 void RenderFrameImpl::DidPlay(blink::WebMediaPlayer* player) {
   Send(new FrameHostMsg_MediaPlayingNotification(
       routing_id_, reinterpret_cast<int64>(player), player->hasVideo(),