Delay Input.dispatchKeyEvent response until after key event ack.

content/browser/devtools/render_frame_devtools_agent_host.cc

Index: content/browser/devtools/render_frame_devtools_agent_host.cc
diff --git a/content/browser/devtools/render_frame_devtools_agent_host.cc b/content/browser/devtools/render_frame_devtools_agent_host.cc
index 8975be6473baf109fbe26ae0f6134f63f202d379..34ac01e7f6949c7af671f9ed3ce5201a18fec28e 100644
--- a/content/browser/devtools/render_frame_devtools_agent_host.cc
+++ b/content/browser/devtools/render_frame_devtools_agent_host.cc
@@ -580,6 +580,8 @@ void RenderFrameDevToolsAgentHost::OnClientDetached() {
 #if defined(OS_ANDROID)
   power_save_blocker_.reset();
 #endif
+  if (input_handler_)

[dgozman, 2016/11/23 02:03:53]

I think it's never null (created in constructor).

[tdresser, 2016/11/23 13:38:13]

Looks like you're right. I'm not sure why this is a unique_ptr, instead of being
stack allocated.

[samuong, 2016/11/23 18:40:48]

Done. I removed the "if (input_handler_)", but I noticed that it is checked
unnecessarily in a few other places too. Let me know if you want me to remove
these and I can submit it in a follow-up CL.

[tdresser, 2016/11/23 18:52:46]

I'd be happy to see us stack allocated input_handler_ in a followup CL.

+    input_handler_->Detached();
   if (emulation_handler_)
     emulation_handler_->Detached();
   if (page_handler_)