Update ICE protocol to handle closed channel

remoting/protocol/connection_unittest.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.
 
 #define _USE_MATH_DEFINES // For VC++ to get M_PI. This has to be first.
 
 #include <utility>
 
 #include "base/bind.h"
 #include "base/macros.h"
 #include "base/memory/ptr_util.h"
 #include "base/message_loop/message_loop.h"
 #include "base/run_loop.h"
 #include "base/test/scoped_task_scheduler.h"
 #include "base/threading/thread.h"
 #include "base/threading/thread_checker.h"
+#include "base/threading/thread_task_runner_handle.h"

[joedow, 2017/03/17 15:13:28]

I don't see any usage of base::ThreadTaskRunnerHandle, is this header needed?

[Sergey Ulanov, 2017/03/17 18:18:22]

Done.

 #include "remoting/base/constants.h"
 #include "remoting/proto/audio.pb.h"
 #include "remoting/protocol/audio_source.h"
 #include "remoting/protocol/audio_stream.h"
 #include "remoting/protocol/audio_stub.h"
 #include "remoting/protocol/fake_session.h"
 #include "remoting/protocol/fake_video_renderer.h"
 #include "remoting/protocol/ice_connection_to_client.h"
 #include "remoting/protocol/ice_connection_to_host.h"
 #include "remoting/protocol/protocol_mock_objects.h"
@@ skipping 650 matching lines @@
   auto capturer = base::MakeUnique<TestScreenCapturer>();
   capturer->FailNthFrame(1);
   auto video_stream = host_connection_->StartVideoStream(std::move(capturer));
 
   WaitNextVideoFrame();
   WaitNextVideoFrame();
 }
 
 }  // namespace protocol
 }  // namespace remoting