Allow MainThreadEventQueue to call the RenderWidget directly.

content/renderer/idle_user_detector.cc

Index: content/renderer/idle_user_detector.cc
diff --git a/content/renderer/idle_user_detector.cc b/content/renderer/idle_user_detector.cc
index 43979cfb722ebbbb322b822c0d665c78a998475d..268015789c222fdb00a5beeb039fe3a09ef09dfb 100644
--- a/content/renderer/idle_user_detector.cc
+++ b/content/renderer/idle_user_detector.cc
@@ -25,11 +25,7 @@ bool IdleUserDetector::OnMessageReceived(const IPC::Message& message) {
   return false;
 }
 
-void IdleUserDetector::OnHandleInputEvent(
-    const blink::WebInputEvent* event,
-    const std::vector<const blink::WebInputEvent*>& coalesced_events,
-    const ui::LatencyInfo& latency_info,
-    InputEventDispatchType dispatch_type) {
+void IdleUserDetector::HandleInputMessageReceived() {

[mustaq, 2017/04/12 19:27:49]

This has nothing to do with handling input. Let's call it |ActivityDetected|
instead.

[dtapuska, 2017/04/13 16:56:24]

Done.

   if (GetContentClient()->renderer()->RunIdleHandlerWhenWidgetsHidden()) {
     RenderThreadImpl* render_thread = RenderThreadImpl::current();
     if (render_thread != NULL) {
@@ -38,8 +34,15 @@ void IdleUserDetector::OnHandleInputEvent(
   }
 }
 
+void IdleUserDetector::OnHandleInputEvent(
+    const blink::WebInputEvent* event,
+    const std::vector<const blink::WebInputEvent*>& coalesced_events,
+    const ui::LatencyInfo& latency_info,
+    InputEventDispatchType dispatch_type) {
+  HandleInputMessageReceived();
+}
+
 void IdleUserDetector::OnDestruct() {
-  delete this;

[tdresser, 2017/04/10 15:54:17]

Out of curiosity, why this change? It seems to be a common pattern here.

[dtapuska, 2017/04/10 16:08:05]

Because this is a IPC message filter that self deletes but it listens for IPC
message events which don't normally go through that flow anymore. So I need to
call it directly. So I need a handle in the RenderView to it.. since I have a
handle, I'll control deletion via a unique_ptr...

[tdresser, 2017/04/10 16:25:00]

Acknowledged.

 }
 
 }  // namespace content