Keep event page alive when there's some Pepper plugin on it.

content/renderer/render_view_impl.cc

Index: content/renderer/render_view_impl.cc
diff --git a/content/renderer/render_view_impl.cc b/content/renderer/render_view_impl.cc
index eb9971320db4aaff0a97e71a2efcc757be981ee0..199e8c0537223476ffe91d778a57f714a4dfe37c 100644
--- a/content/renderer/render_view_impl.cc
+++ b/content/renderer/render_view_impl.cc
@@ -1179,6 +1179,11 @@ blink::WebView* RenderViewImpl::webview() const {
 void RenderViewImpl::PepperInstanceCreated(
     PepperPluginInstanceImpl* instance) {
   active_pepper_instances_.insert(instance);
+
+  RenderFrameImpl* const render_frame = instance->render_frame();
+  if (render_frame)

[nasko, 2015/05/19 17:49:34]

RenderFrame should never be null, so there is no reason to check the pointer for
validity. A crash on the call to Send would indicate a bug and we will want to
know about it.

[emaxx, 2015/05/19 18:14:59]

You are right.
I've now placed a DCHECK here - to reassure and to show the intention.

[nasko, 2015/05/19 18:51:51]

In general, we don't add DCHECK when the line after is going to crash anyway.
The nullptr crash is equivalent to the DCHECK, so let's remove those.

+    render_frame->Send(
+        new FrameHostMsg_PepperInstanceCreated(render_frame->GetRoutingID()));
 }
 
 void RenderViewImpl::PepperInstanceDeleted(
@@ -1189,6 +1194,11 @@ void RenderViewImpl::PepperInstanceDeleted(
     pepper_last_mouse_event_target_ = NULL;
   if (focused_pepper_plugin_ == instance)
     PepperFocusChanged(instance, false);
+
+  RenderFrameImpl* const render_frame = instance->render_frame();
+  if (render_frame)
+    render_frame->Send(
+        new FrameHostMsg_PepperInstanceDeleted(render_frame->GetRoutingID()));
 }
 
 void RenderViewImpl::PepperFocusChanged(PepperPluginInstanceImpl* instance,