Update webrtc::DesktopCapturer clients to implement OnCaptureResult().

remoting/host/chromoting_messages.h

 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #ifndef REMOTING_HOST_CHROMOTING_MESSAGES_H_
 #define REMOTING_HOST_CHROMOTING_MESSAGES_H_
 
 #include <stdint.h>
 
 #include "base/memory/shared_memory_handle.h"
 #include "ipc/ipc_platform_file.h"
 #include "net/base/ip_endpoint.h"
 #include "remoting/host/chromoting_param_traits.h"
 #include "remoting/host/screen_resolution.h"
 #include "remoting/protocol/errors.h"
 #include "remoting/protocol/transport.h"
+#include "third_party/webrtc/modules/desktop_capture/desktop_capturer.h"
 #include "third_party/webrtc/modules/desktop_capture/desktop_geometry.h"
 
 #endif  // REMOTING_HOST_CHROMOTING_MESSAGES_H_
 
 // Multiply-included message file, no traditional include guard.
 #include "ipc/ipc_message_macros.h"
 
 #define IPC_MESSAGE_START ChromotingMsgStart
 
 //-----------------------------------------------------------------------------
@@ skipping 138 matching lines @@
   // Time spent in capture. Unit is in milliseconds.
   IPC_STRUCT_MEMBER(int64_t, capture_time_ms)
 
   // Latest event timestamp supplied by the client for performance tracking.
   IPC_STRUCT_MEMBER(int64_t, latest_event_timestamp)
 
   // DPI for this frame.
   IPC_STRUCT_MEMBER(webrtc::DesktopVector, dpi)
 IPC_STRUCT_END()
 
+IPC_ENUM_TRAITS_MAX_VALUE(webrtc::DesktopCapturer::Result,
+                          webrtc::DesktopCapturer::Result::ERROR_PERMANENT)

[Jamie, 2016/06/14 21:22:44]

Relying on ERROR_PERMANENT as being the max value is not very future proof.
Maybe add an explicit MAX_VALUE to the enum so that this doesn't need to change
if it's ever updated?

[Sergey Ulanov, 2016/06/15 20:30:10]

Done.

+
 // Notifies the network process that a shared buffer has been created.
-IPC_MESSAGE_CONTROL1(ChromotingDesktopNetworkMsg_CaptureCompleted,
-                     SerializedDesktopFrame /* frame */ )
+IPC_MESSAGE_CONTROL2(ChromotingDesktopNetworkMsg_CaptureResult,
+                     webrtc::DesktopCapturer::Result /* result */,
+                     SerializedDesktopFrame /* frame */)
 
 // Carries a cursor share update from the desktop session agent to the client.
 IPC_MESSAGE_CONTROL1(ChromotingDesktopNetworkMsg_MouseCursor,
                      webrtc::MouseCursor /* cursor */ )
 
 // Carries a clipboard event from the desktop session agent to the client.
 // |serialized_event| is a serialized protocol::ClipboardEvent.
 IPC_MESSAGE_CONTROL1(ChromotingDesktopNetworkMsg_InjectClipboardEvent,
                      std::string /* serialized_event */ )
 
@@ skipping 75 matching lines @@
 
 // The IPC channel path for this remote_security_key connection.  This message
 // is sent from the well-known IPC server channel.
 IPC_MESSAGE_CONTROL1(ChromotingNetworkToRemoteSecurityKeyMsg_ConnectionDetails,
                      std::string /* IPC Server path */)
 
 // The array of bytes representing a security key response from the remote
 // client.  This message is sent over the per-client IPC channel.
 IPC_MESSAGE_CONTROL1(ChromotingNetworkToRemoteSecurityKeyMsg_Response,
                      std::string /* response bytes */)