[Chromoting] A crash in DesktopSessionProxy::OnCaptureResult

remoting/host/desktop_session_proxy.cc

Index: remoting/host/desktop_session_proxy.cc
diff --git a/remoting/host/desktop_session_proxy.cc b/remoting/host/desktop_session_proxy.cc
index ed7ba98793e6048cad24942bfb4bb5dbd6f1220b..35384e8decf14a827da2941da9239c3872a5e00d 100644
--- a/remoting/host/desktop_session_proxy.cc
+++ b/remoting/host/desktop_session_proxy.cc
@@ -484,6 +484,12 @@ void DesktopSessionProxy::OnCaptureResult(
 
   --pending_capture_frame_requests_;
 
+  if (!video_capturer_) {
+    // video_capturer_ has been released before DesktopSessionAgent returns this

[Sergey Ulanov, 2016/08/12 04:15:18]

nit: 
 1. the capturer was deleted, not released.
 2. reader may not be aware what is DesktopSessionAgent and it's not that
important here. If it has to be mentioned here, then it's better to call it "the
desktop process".
suggest rephrasing: "Capturer was deleted while capture request was pending." Or
maybe just remove this comment, as it should be clear what's happening without
this comment

+    // frame.
+    return;
+  }
+
   if (result != webrtc::DesktopCapturer::Result::SUCCESS) {
     video_capturer_->OnCaptureResult(result, nullptr);
     return;