[Chromoting] Implement new APIs in DesktopCapturer

remoting/host/ipc_video_frame_capturer.cc

 // 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.
 
 #include "remoting/host/ipc_video_frame_capturer.h"
 
 #include "remoting/host/desktop_session_proxy.h"
 #include "third_party/webrtc/modules/desktop_capture/desktop_frame.h"
 
 namespace remoting {
@@ skipping 23 matching lines @@
 }
 
 void IpcVideoFrameCapturer::OnCaptureResult(
     webrtc::DesktopCapturer::Result result,
     std::unique_ptr<webrtc::DesktopFrame> frame) {
   DCHECK(capture_pending_);
   capture_pending_ = false;
   callback_->OnCaptureResult(result, std::move(frame));
 }
 
+bool IpcVideoFrameCapturer::GetSourceList(SourceList* sources) {
+  return desktop_session_proxy_->GetSourceList(sources);

[Sergey Ulanov, 2016/11/05 00:16:26]

Problem here is that the API is synchronous, but all IPCs need to be
asynchronous. OTOH the host currently doesn't need these functions.
Please add NOTIMPLEMENTED() and return false here. We can figure out what to do
about it later, when we need these functions.

[Hzj_jie, 2016/11/06 04:27:24]

Yes, that's the reason I put the TODO in DesktopSessionProxy instead of this
class. DesktopSessionProxy needs to be able to get source list or select source
without invoking RPC.

Done.

+}
+
+bool IpcVideoFrameCapturer::SelectSource(SourceId id) {
+  return desktop_session_proxy_->SelectSource(id);
+}
+
 }  // namespace remoting