Don't let Blink handle events if we're shuting down.

third_party/WebKit/Source/web/WebFrameWidgetImpl.cpp

Index: third_party/WebKit/Source/web/WebFrameWidgetImpl.cpp
diff --git a/third_party/WebKit/Source/web/WebFrameWidgetImpl.cpp b/third_party/WebKit/Source/web/WebFrameWidgetImpl.cpp
index f49ee7aa5ab235a35085d8540b88632c8d82184d..cc5d4b7645afc52547288c9a68e05182c15ecad1 100644
--- a/third_party/WebKit/Source/web/WebFrameWidgetImpl.cpp
+++ b/third_party/WebKit/Source/web/WebFrameWidgetImpl.cpp
@@ -311,9 +311,12 @@ const WebInputEvent* WebFrameWidgetImpl::m_currentInputEvent = nullptr;
 
 WebInputEventResult WebFrameWidgetImpl::handleInputEvent(const WebInputEvent& inputEvent)
 {
-
     TRACE_EVENT1("input", "WebFrameWidgetImpl::handleInputEvent", "type", inputTypeToName(inputEvent.type));
 
+    // Don't handle events once we've started shutting down.
+    if (!page())

[dtapuska, 2016/04/29 14:31:27]

Is it preferred to collapse this with the return below; like

if (!page() || m_ignoreInputEvents)

[bokan, 2016/04/29 14:49:00]

IMO, in this case, they're unrelated and it's more natural to comment the cases
if they're separate.

[dcheng, 2016/04/29 18:36:29]

Dumb question: how come we don't hit this in the WebViewImpl case?

+        return WebInputEventResult::NotHandled;
+
     // Report the event to be NOT processed by WebKit, so that the browser can handle it appropriately.
     if (m_ignoreInputEvents)
         return WebInputEventResult::NotHandled;