[Chromoting] Implement new APIs in DesktopCapturer

remoting/host/ipc_video_frame_capturer.cc

Index: remoting/host/ipc_video_frame_capturer.cc
diff --git a/remoting/host/ipc_video_frame_capturer.cc b/remoting/host/ipc_video_frame_capturer.cc
index 9d061ddac1719abda205640c4f14ccb4f10aef7a..93c1f883b854748e63e0385fbdec455998abc9ce 100644
--- a/remoting/host/ipc_video_frame_capturer.cc
+++ b/remoting/host/ipc_video_frame_capturer.cc
@@ -41,4 +41,12 @@ void IpcVideoFrameCapturer::OnCaptureResult(
   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