sergeyu | 97568a816 | 2015-02-24 18:00:55 | [diff] [blame] | 1 | // Copyright 2015 The Chromium Authors. All rights reserved. |
| 2 | // Use of this source code is governed by a BSD-style license that can be |
| 3 | // found in the LICENSE file. |
| 4 | |
| 5 | #include "remoting/protocol/client_video_dispatcher.h" |
| 6 | |
avi | 954c9c1d | 2017-01-11 16:52:37 | [diff] [blame] | 7 | #include <memory> |
Erik Jensen | 56d7f2e | 2019-02-20 01:38:56 | [diff] [blame] | 8 | #include <utility> |
avi | 954c9c1d | 2017-01-11 16:52:37 | [diff] [blame] | 9 | #include <vector> |
| 10 | |
sergeyu | 97568a816 | 2015-02-24 18:00:55 | [diff] [blame] | 11 | #include "base/bind.h" |
sergeyu | 97568a816 | 2015-02-24 18:00:55 | [diff] [blame] | 12 | #include "base/run_loop.h" |
Gabriel Charette | c710874 | 2019-08-23 03:31:40 | [diff] [blame] | 13 | #include "base/test/task_environment.h" |
[email protected] | f94cd970 | 2017-12-07 06:09:40 | [diff] [blame] | 14 | #include "net/traffic_annotation/network_traffic_annotation_test_helper.h" |
sergeyu | aa22c08 | 2015-07-20 19:41:13 | [diff] [blame] | 15 | #include "remoting/base/buffered_socket_writer.h" |
sergeyu | 97568a816 | 2015-02-24 18:00:55 | [diff] [blame] | 16 | #include "remoting/base/constants.h" |
| 17 | #include "remoting/proto/video.pb.h" |
sergeyu | 97568a816 | 2015-02-24 18:00:55 | [diff] [blame] | 18 | #include "remoting/protocol/fake_stream_socket.h" |
sergeyu | f1005f6 | 2016-02-03 21:11:30 | [diff] [blame] | 19 | #include "remoting/protocol/message_reader.h" |
sergeyu | 97568a816 | 2015-02-24 18:00:55 | [diff] [blame] | 20 | #include "remoting/protocol/message_serialization.h" |
sergeyu | 00a67b1 | 2016-04-01 00:07:00 | [diff] [blame] | 21 | #include "remoting/protocol/protocol_mock_objects.h" |
sergeyu | 786463e | 2016-02-04 18:46:10 | [diff] [blame] | 22 | #include "remoting/protocol/stream_message_pipe_adapter.h" |
sergeyu | 97568a816 | 2015-02-24 18:00:55 | [diff] [blame] | 23 | #include "remoting/protocol/video_stub.h" |
| 24 | #include "testing/gtest/include/gtest/gtest.h" |
| 25 | |
| 26 | namespace remoting { |
| 27 | namespace protocol { |
| 28 | |
| 29 | class ClientVideoDispatcherTest : public testing::Test, |
| 30 | public VideoStub, |
| 31 | public ChannelDispatcherBase::EventHandler { |
| 32 | public: |
| 33 | ClientVideoDispatcherTest(); |
| 34 | |
| 35 | // VideoStub interface. |
dcheng | 0765c49 | 2016-04-06 22:41:53 | [diff] [blame] | 36 | void ProcessVideoPacket(std::unique_ptr<VideoPacket> video_packet, |
Erik Jensen | 56d7f2e | 2019-02-20 01:38:56 | [diff] [blame] | 37 | base::OnceClosure done) override; |
sergeyu | 97568a816 | 2015-02-24 18:00:55 | [diff] [blame] | 38 | |
| 39 | // ChannelDispatcherBase::EventHandler interface. |
| 40 | void OnChannelInitialized(ChannelDispatcherBase* channel_dispatcher) override; |
sergeyu | d059c46 | 2016-07-20 19:34:10 | [diff] [blame] | 41 | void OnChannelClosed(ChannelDispatcherBase* channel_dispatcher) override; |
sergeyu | 97568a816 | 2015-02-24 18:00:55 | [diff] [blame] | 42 | |
| 43 | protected: |
sergeyu | 786463e | 2016-02-04 18:46:10 | [diff] [blame] | 44 | void OnChannelError(int error); |
| 45 | |
dcheng | 0765c49 | 2016-04-06 22:41:53 | [diff] [blame] | 46 | void OnMessageReceived(std::unique_ptr<CompoundBuffer> buffer); |
sergeyu | a04c046 | 2015-05-14 01:48:40 | [diff] [blame] | 47 | void OnReadError(int error); |
sergeyu | 97568a816 | 2015-02-24 18:00:55 | [diff] [blame] | 48 | |
Gabriel Charette | df0a744 | 2019-09-05 17:32:35 | [diff] [blame] | 49 | base::test::SingleThreadTaskEnvironment task_environment_; |
sergeyu | 97568a816 | 2015-02-24 18:00:55 | [diff] [blame] | 50 | |
| 51 | // Set to true in OnChannelInitialized(). |
sergeyu | 786463e | 2016-02-04 18:46:10 | [diff] [blame] | 52 | bool initialized_ = false; |
sergeyu | 97568a816 | 2015-02-24 18:00:55 | [diff] [blame] | 53 | |
| 54 | // Client side. |
sergeyu | 38e73d4 | 2015-12-18 06:50:04 | [diff] [blame] | 55 | FakeStreamChannelFactory client_channel_factory_; |
sergeyu | 786463e | 2016-02-04 18:46:10 | [diff] [blame] | 56 | StreamMessageChannelFactoryAdapter channel_factory_adapter_; |
sergeyu | 00a67b1 | 2016-04-01 00:07:00 | [diff] [blame] | 57 | MockClientStub client_stub_; |
sergeyu | 97568a816 | 2015-02-24 18:00:55 | [diff] [blame] | 58 | ClientVideoDispatcher dispatcher_; |
sergeyu | 97568a816 | 2015-02-24 18:00:55 | [diff] [blame] | 59 | |
| 60 | // Host side. |
| 61 | FakeStreamSocket host_socket_; |
| 62 | MessageReader reader_; |
sergeyu | 97568a816 | 2015-02-24 18:00:55 | [diff] [blame] | 63 | BufferedSocketWriter writer_; |
| 64 | |
avi | 954c9c1d | 2017-01-11 16:52:37 | [diff] [blame] | 65 | std::vector<std::unique_ptr<VideoPacket>> video_packets_; |
Erik Jensen | 56d7f2e | 2019-02-20 01:38:56 | [diff] [blame] | 66 | std::vector<base::OnceClosure> packet_done_callbacks_; |
sergeyu | 97568a816 | 2015-02-24 18:00:55 | [diff] [blame] | 67 | |
avi | 954c9c1d | 2017-01-11 16:52:37 | [diff] [blame] | 68 | std::vector<std::unique_ptr<VideoAck>> ack_messages_; |
sergeyu | 97568a816 | 2015-02-24 18:00:55 | [diff] [blame] | 69 | }; |
| 70 | |
| 71 | ClientVideoDispatcherTest::ClientVideoDispatcherTest() |
sergeyu | 786463e | 2016-02-04 18:46:10 | [diff] [blame] | 72 | : channel_factory_adapter_( |
| 73 | &client_channel_factory_, |
| 74 | base::Bind(&ClientVideoDispatcherTest::OnChannelError, |
| 75 | base::Unretained(this))), |
sergeyu | 00a67b1 | 2016-04-01 00:07:00 | [diff] [blame] | 76 | dispatcher_(this, &client_stub_) { |
sergeyu | 786463e | 2016-02-04 18:46:10 | [diff] [blame] | 77 | dispatcher_.Init(&channel_factory_adapter_, this); |
sergeyu | 97568a816 | 2015-02-24 18:00:55 | [diff] [blame] | 78 | base::RunLoop().RunUntilIdle(); |
| 79 | DCHECK(initialized_); |
| 80 | host_socket_.PairWith( |
sergeyu | 38e73d4 | 2015-12-18 06:50:04 | [diff] [blame] | 81 | client_channel_factory_.GetFakeChannel(kVideoChannelName)); |
sergeyu | a04c046 | 2015-05-14 01:48:40 | [diff] [blame] | 82 | reader_.StartReading(&host_socket_, |
sergeyu | d8af2ca | 2016-01-30 03:04:36 | [diff] [blame] | 83 | base::Bind(&ClientVideoDispatcherTest::OnMessageReceived, |
| 84 | base::Unretained(this)), |
sergeyu | a04c046 | 2015-05-14 01:48:40 | [diff] [blame] | 85 | base::Bind(&ClientVideoDispatcherTest::OnReadError, |
| 86 | base::Unretained(this))); |
sergeyu | c3497fc | 2016-01-14 06:53:01 | [diff] [blame] | 87 | writer_.Start( |
sergeyu | aa22c08 | 2015-07-20 19:41:13 | [diff] [blame] | 88 | base::Bind(&P2PStreamSocket::Write, base::Unretained(&host_socket_)), |
| 89 | BufferedSocketWriter::WriteFailedCallback()); |
sergeyu | 97568a816 | 2015-02-24 18:00:55 | [diff] [blame] | 90 | } |
| 91 | |
| 92 | void ClientVideoDispatcherTest::ProcessVideoPacket( |
dcheng | 0765c49 | 2016-04-06 22:41:53 | [diff] [blame] | 93 | std::unique_ptr<VideoPacket> video_packet, |
Erik Jensen | 56d7f2e | 2019-02-20 01:38:56 | [diff] [blame] | 94 | base::OnceClosure done) { |
avi | 954c9c1d | 2017-01-11 16:52:37 | [diff] [blame] | 95 | video_packets_.push_back(std::move(video_packet)); |
Erik Jensen | 56d7f2e | 2019-02-20 01:38:56 | [diff] [blame] | 96 | packet_done_callbacks_.push_back(std::move(done)); |
sergeyu | 97568a816 | 2015-02-24 18:00:55 | [diff] [blame] | 97 | } |
| 98 | |
| 99 | void ClientVideoDispatcherTest::OnChannelInitialized( |
| 100 | ChannelDispatcherBase* channel_dispatcher) { |
| 101 | initialized_ = true; |
| 102 | } |
| 103 | |
sergeyu | d059c46 | 2016-07-20 19:34:10 | [diff] [blame] | 104 | void ClientVideoDispatcherTest::OnChannelClosed( |
| 105 | ChannelDispatcherBase* channel_dispatcher) { |
| 106 | // Don't expect channels to be closed. |
| 107 | FAIL(); |
| 108 | } |
| 109 | |
sergeyu | 786463e | 2016-02-04 18:46:10 | [diff] [blame] | 110 | void ClientVideoDispatcherTest::OnChannelError(int error) { |
sergeyu | 97568a816 | 2015-02-24 18:00:55 | [diff] [blame] | 111 | // Don't expect channel creation to fail. |
| 112 | FAIL(); |
| 113 | } |
| 114 | |
sergeyu | d8af2ca | 2016-01-30 03:04:36 | [diff] [blame] | 115 | void ClientVideoDispatcherTest::OnMessageReceived( |
dcheng | 0765c49 | 2016-04-06 22:41:53 | [diff] [blame] | 116 | std::unique_ptr<CompoundBuffer> buffer) { |
| 117 | std::unique_ptr<VideoAck> ack = ParseMessage<VideoAck>(buffer.get()); |
sergeyu | d8af2ca | 2016-01-30 03:04:36 | [diff] [blame] | 118 | EXPECT_TRUE(ack); |
avi | 954c9c1d | 2017-01-11 16:52:37 | [diff] [blame] | 119 | ack_messages_.push_back(std::move(ack)); |
sergeyu | 97568a816 | 2015-02-24 18:00:55 | [diff] [blame] | 120 | } |
| 121 | |
sergeyu | a04c046 | 2015-05-14 01:48:40 | [diff] [blame] | 122 | void ClientVideoDispatcherTest::OnReadError(int error) { |
| 123 | LOG(FATAL) << "Unexpected read error: " << error; |
| 124 | } |
| 125 | |
sergeyu | 97568a816 | 2015-02-24 18:00:55 | [diff] [blame] | 126 | // Verify that the client can receive video packets and acks are not sent for |
| 127 | // VideoPackets that don't have frame_id field set. |
| 128 | TEST_F(ClientVideoDispatcherTest, WithoutAcks) { |
| 129 | VideoPacket packet; |
| 130 | packet.set_data(std::string()); |
| 131 | |
| 132 | // Send a VideoPacket and verify that the client receives it. |
Evan Stade | 86dadf15 | 2020-03-16 20:06:33 | [diff] [blame] | 133 | writer_.Write(SerializeAndFrameMessage(packet), {}, |
[email protected] | f94cd970 | 2017-12-07 06:09:40 | [diff] [blame] | 134 | TRAFFIC_ANNOTATION_FOR_TESTS); |
sergeyu | 97568a816 | 2015-02-24 18:00:55 | [diff] [blame] | 135 | base::RunLoop().RunUntilIdle(); |
| 136 | EXPECT_EQ(1U, video_packets_.size()); |
| 137 | |
Erik Jensen | 56d7f2e | 2019-02-20 01:38:56 | [diff] [blame] | 138 | std::move(packet_done_callbacks_.front()).Run(); |
sergeyu | 97568a816 | 2015-02-24 18:00:55 | [diff] [blame] | 139 | base::RunLoop().RunUntilIdle(); |
| 140 | |
| 141 | // Ack should never be sent for the packet without frame_id. |
| 142 | EXPECT_TRUE(ack_messages_.empty()); |
| 143 | } |
| 144 | |
| 145 | // Verifies that the dispatcher sends Ack message with correct rendering delay. |
| 146 | TEST_F(ClientVideoDispatcherTest, WithAcks) { |
| 147 | int kTestFrameId = 3; |
| 148 | |
| 149 | VideoPacket packet; |
| 150 | packet.set_data(std::string()); |
| 151 | packet.set_frame_id(kTestFrameId); |
| 152 | |
| 153 | // Send a VideoPacket and verify that the client receives it. |
Evan Stade | 86dadf15 | 2020-03-16 20:06:33 | [diff] [blame] | 154 | writer_.Write(SerializeAndFrameMessage(packet), {}, |
[email protected] | f94cd970 | 2017-12-07 06:09:40 | [diff] [blame] | 155 | TRAFFIC_ANNOTATION_FOR_TESTS); |
sergeyu | 97568a816 | 2015-02-24 18:00:55 | [diff] [blame] | 156 | base::RunLoop().RunUntilIdle(); |
| 157 | EXPECT_EQ(1U, video_packets_.size()); |
| 158 | |
| 159 | // Ack should only be sent after the packet is processed. |
| 160 | EXPECT_TRUE(ack_messages_.empty()); |
| 161 | base::RunLoop().RunUntilIdle(); |
| 162 | |
| 163 | // Fake completion of video packet decoding, to trigger the Ack. |
Erik Jensen | 56d7f2e | 2019-02-20 01:38:56 | [diff] [blame] | 164 | std::move(packet_done_callbacks_.front()).Run(); |
sergeyu | 97568a816 | 2015-02-24 18:00:55 | [diff] [blame] | 165 | base::RunLoop().RunUntilIdle(); |
| 166 | |
| 167 | // Verify that the Ack message has been received. |
| 168 | ASSERT_EQ(1U, ack_messages_.size()); |
| 169 | EXPECT_EQ(kTestFrameId, ack_messages_[0]->frame_id()); |
| 170 | } |
| 171 | |
sergeyu | 00a67b1 | 2016-04-01 00:07:00 | [diff] [blame] | 172 | // Verifies that the dispatcher properly synthesizes VideoLayout message when |
| 173 | // screen size changes. |
| 174 | TEST_F(ClientVideoDispatcherTest, VideoLayout) { |
| 175 | const int kScreenSize = 100; |
| 176 | const float kScaleFactor = 2.0; |
| 177 | |
| 178 | VideoPacket packet; |
| 179 | packet.set_data(std::string()); |
| 180 | packet.set_frame_id(42); |
| 181 | packet.mutable_format()->set_screen_width(kScreenSize); |
| 182 | packet.mutable_format()->set_screen_height(kScreenSize); |
| 183 | packet.mutable_format()->set_x_dpi(kDefaultDpi * kScaleFactor); |
| 184 | packet.mutable_format()->set_y_dpi(kDefaultDpi * kScaleFactor); |
| 185 | |
| 186 | VideoLayout layout; |
| 187 | EXPECT_CALL(client_stub_, SetVideoLayout(testing::_)) |
| 188 | .WillOnce(testing::SaveArg<0>(&layout)); |
| 189 | |
| 190 | // Send a VideoPacket and verify that the client receives it. |
Evan Stade | 86dadf15 | 2020-03-16 20:06:33 | [diff] [blame] | 191 | writer_.Write(SerializeAndFrameMessage(packet), {}, |
[email protected] | f94cd970 | 2017-12-07 06:09:40 | [diff] [blame] | 192 | TRAFFIC_ANNOTATION_FOR_TESTS); |
sergeyu | 00a67b1 | 2016-04-01 00:07:00 | [diff] [blame] | 193 | base::RunLoop().RunUntilIdle(); |
| 194 | |
| 195 | ASSERT_EQ(1, layout.video_track_size()); |
| 196 | EXPECT_EQ(0, layout.video_track(0).position_x()); |
| 197 | EXPECT_EQ(0, layout.video_track(0).position_y()); |
| 198 | EXPECT_EQ(kScreenSize / kScaleFactor, layout.video_track(0).width()); |
| 199 | EXPECT_EQ(kScreenSize / kScaleFactor, layout.video_track(0).height()); |
| 200 | EXPECT_EQ(kDefaultDpi * kScaleFactor, layout.video_track(0).x_dpi()); |
| 201 | EXPECT_EQ(kDefaultDpi * kScaleFactor, layout.video_track(0).y_dpi()); |
| 202 | } |
| 203 | |
sergeyu | 97568a816 | 2015-02-24 18:00:55 | [diff] [blame] | 204 | // Verify that Ack messages are sent in correct order. |
| 205 | TEST_F(ClientVideoDispatcherTest, AcksOrder) { |
| 206 | int kTestFrameId = 3; |
| 207 | |
| 208 | VideoPacket packet; |
| 209 | packet.set_data(std::string()); |
| 210 | packet.set_frame_id(kTestFrameId); |
| 211 | |
| 212 | // Send two VideoPackets. |
Evan Stade | 86dadf15 | 2020-03-16 20:06:33 | [diff] [blame] | 213 | writer_.Write(SerializeAndFrameMessage(packet), {}, |
[email protected] | f94cd970 | 2017-12-07 06:09:40 | [diff] [blame] | 214 | TRAFFIC_ANNOTATION_FOR_TESTS); |
sergeyu | 97568a816 | 2015-02-24 18:00:55 | [diff] [blame] | 215 | base::RunLoop().RunUntilIdle(); |
| 216 | |
| 217 | packet.set_frame_id(kTestFrameId + 1); |
Evan Stade | 86dadf15 | 2020-03-16 20:06:33 | [diff] [blame] | 218 | writer_.Write(SerializeAndFrameMessage(packet), {}, |
[email protected] | f94cd970 | 2017-12-07 06:09:40 | [diff] [blame] | 219 | TRAFFIC_ANNOTATION_FOR_TESTS); |
sergeyu | 97568a816 | 2015-02-24 18:00:55 | [diff] [blame] | 220 | base::RunLoop().RunUntilIdle(); |
| 221 | |
| 222 | EXPECT_EQ(2U, video_packets_.size()); |
| 223 | EXPECT_TRUE(ack_messages_.empty()); |
| 224 | |
| 225 | // Call completion callbacks in revers order. |
Erik Jensen | 56d7f2e | 2019-02-20 01:38:56 | [diff] [blame] | 226 | std::move(packet_done_callbacks_[1]).Run(); |
| 227 | std::move(packet_done_callbacks_[0]).Run(); |
sergeyu | 97568a816 | 2015-02-24 18:00:55 | [diff] [blame] | 228 | |
| 229 | base::RunLoop().RunUntilIdle(); |
| 230 | |
| 231 | // Verify order of Ack messages. |
| 232 | ASSERT_EQ(2U, ack_messages_.size()); |
| 233 | EXPECT_EQ(kTestFrameId, ack_messages_[0]->frame_id()); |
| 234 | EXPECT_EQ(kTestFrameId + 1, ack_messages_[1]->frame_id()); |
| 235 | } |
| 236 | |
| 237 | } // namespace protocol |
| 238 | } // namespace remoting |