Move audio decoding to protocol layer

remoting/protocol/webrtc_connection_to_host.cc

 // Copyright 2015 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/protocol/webrtc_connection_to_host.h"
 
 #include <utility>
 
 #include "base/strings/string_util.h"
 #include "jingle/glue/thread_wrapper.h"
@@ skipping 54 matching lines @@
 }
 
 void WebrtcConnectionToHost::set_clipboard_stub(ClipboardStub* clipboard_stub) {
   clipboard_stub_ = clipboard_stub;
 }
 
 void WebrtcConnectionToHost::set_video_renderer(VideoRenderer* video_renderer) {
   video_renderer_ = video_renderer;
 }
 
-void WebrtcConnectionToHost::set_audio_stub(AudioStub* audio_stub) {
+void WebrtcConnectionToHost::InitializeAudio(
+    scoped_refptr<base::SingleThreadTaskRunner> audio_decode_task_runner,
+    base::WeakPtr<AudioStub> audio_stub) {
   NOTIMPLEMENTED();

[Jamie, 2016/10/04 00:25:38]

Is NOTIMPLEMENTED appropriate here? NOTIMPLEMENTED suggests to me that it should
be implemented (and it logs a warning to remind you to do so), but based on your
CL comment, I think this is intentionally a no-op for webrtc, so an empty method
body seems more appropriate.

[Sergey Ulanov, 2016/10/04 00:42:22]

NOIMPLEMENTED is appropriate here. It will be implemented in a separate CL. This
class won't be using AudioDecodeScheduler because webrtc takes care of audio
decoding, but it will need to store audio_stub and audio_decode_task_runner
here.

 }
 
 void WebrtcConnectionToHost::OnSessionStateChange(Session::State state) {
   DCHECK(event_callback_);
 
   switch (state) {
     case Session::INITIALIZING:
     case Session::CONNECTING:
     case Session::ACCEPTING:
     case Session::ACCEPTED:
@@ skipping 118 matching lines @@
 
   if (state != state_) {
     state_ = state;
     error_ = error;
     event_callback_->OnConnectionState(state_, error_);
   }
 }
 
 }  // namespace protocol
 }  // namespace remoting