Fix DesktopSessioAgent crash when handling nullptr OnCaptureCompleted callback.

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 4d1f5402882079ad288c69d8be0b8ce63bfc3fa6..82db2de98bd9ea6bd78dc43c455f7dab4c96757b 100644
--- a/remoting/host/desktop_session_proxy.cc
+++ b/remoting/host/desktop_session_proxy.cc
@@ -471,6 +471,14 @@ void DesktopSessionProxy::OnCaptureCompleted(
     const SerializedDesktopFrame& serialized_frame) {
   DCHECK(caller_task_runner_->BelongsToCurrentThread());
 
+  // If the input serialized_frame does not have a screen size, it means the
+  // capturer returns a nullptr for OnCaptureCompleted call.
+  if (serialized_frame.dimensions.is_empty()) {
+    --pending_capture_frame_requests_;

[Sergey Ulanov, 2016/05/26 12:20:10]

Move this out of the if block and remove duplicate line 499

[Hzj_jie, 2016/05/26 20:59:22]

Done.

+    video_capturer_->OnCaptureCompleted(nullptr);
+    return;
+  }
+
   // Assume that |serialized_frame| is well-formed because it was received from
   // a more privileged process.
   scoped_refptr<IpcSharedBufferCore> shared_buffer_core =