OLD | NEW |
1 // Copyright (c) 2012 The Chromium Authors. All rights reserved. | 1 // Copyright (c) 2012 The Chromium Authors. All rights reserved. |
2 // Use of this source code is governed by a BSD-style license that can be | 2 // Use of this source code is governed by a BSD-style license that can be |
3 // found in the LICENSE file. | 3 // found in the LICENSE file. |
4 | 4 |
5 #include <string> | 5 #include <string> |
6 | 6 |
7 #include "base/bind.h" | 7 #include "base/bind.h" |
8 #include "base/bind_helpers.h" | 8 #include "base/bind_helpers.h" |
9 #include "base/callback.h" | 9 #include "base/callback.h" |
10 #include "base/message_loop/message_loop.h" | 10 #include "base/message_loop/message_loop.h" |
(...skipping 58 matching lines...) Expand 10 before | Expand all | Expand 10 after Loading... |
69 protected: | 69 protected: |
70 void SetUp() override { | 70 void SetUp() override { |
71 reader_.reset(new MessageReader()); | 71 reader_.reset(new MessageReader()); |
72 } | 72 } |
73 | 73 |
74 void TearDown() override { STLDeleteElements(&messages_); } | 74 void TearDown() override { STLDeleteElements(&messages_); } |
75 | 75 |
76 void InitReader() { | 76 void InitReader() { |
77 reader_->SetMessageReceivedCallback( | 77 reader_->SetMessageReceivedCallback( |
78 base::Bind(&MessageReaderTest::OnMessage, base::Unretained(this))); | 78 base::Bind(&MessageReaderTest::OnMessage, base::Unretained(this))); |
79 reader_->StartReading(&socket_); | 79 reader_->StartReading(&socket_, base::Bind(&MessageReaderTest::OnReadError, |
| 80 base::Unretained(this))); |
80 } | 81 } |
81 | 82 |
82 void AddMessage(const std::string& message) { | 83 void AddMessage(const std::string& message) { |
83 std::string data = std::string(4, ' ') + message; | 84 std::string data = std::string(4, ' ') + message; |
84 rtc::SetBE32(const_cast<char*>(data.data()), message.size()); | 85 rtc::SetBE32(const_cast<char*>(data.data()), message.size()); |
85 | 86 |
86 socket_.AppendInputData(data); | 87 socket_.AppendInputData(data); |
87 } | 88 } |
88 | 89 |
89 bool CompareResult(CompoundBuffer* buffer, const std::string& expected) { | 90 bool CompareResult(CompoundBuffer* buffer, const std::string& expected) { |
90 std::string result(buffer->total_bytes(), ' '); | 91 std::string result(buffer->total_bytes(), ' '); |
91 buffer->CopyTo(const_cast<char*>(result.data()), result.size()); | 92 buffer->CopyTo(const_cast<char*>(result.data()), result.size()); |
92 return result == expected; | 93 return result == expected; |
93 } | 94 } |
94 | 95 |
| 96 void OnReadError(int error) { |
| 97 read_error_ = error; |
| 98 reader_.reset(); |
| 99 } |
| 100 |
95 void OnMessage(scoped_ptr<CompoundBuffer> buffer, | 101 void OnMessage(scoped_ptr<CompoundBuffer> buffer, |
96 const base::Closure& done_callback) { | 102 const base::Closure& done_callback) { |
97 messages_.push_back(buffer.release()); | 103 messages_.push_back(buffer.release()); |
98 callback_.OnMessage(done_callback); | 104 callback_.OnMessage(done_callback); |
99 } | 105 } |
100 | 106 |
101 base::MessageLoop message_loop_; | 107 base::MessageLoop message_loop_; |
102 scoped_ptr<MessageReader> reader_; | 108 scoped_ptr<MessageReader> reader_; |
103 FakeStreamSocket socket_; | 109 FakeStreamSocket socket_; |
104 MockMessageReceivedCallback callback_; | 110 MockMessageReceivedCallback callback_; |
| 111 int read_error_ = 0; |
105 std::vector<CompoundBuffer*> messages_; | 112 std::vector<CompoundBuffer*> messages_; |
106 bool in_callback_; | 113 bool in_callback_; |
107 }; | 114 }; |
108 | 115 |
109 // Receive one message and process it with delay | 116 // Receive one message and process it with delay |
110 TEST_F(MessageReaderTest, OneMessage_Delay) { | 117 TEST_F(MessageReaderTest, OneMessage_Delay) { |
111 base::Closure done_task; | 118 base::Closure done_task; |
112 | 119 |
113 AddMessage(kTestMessage1); | 120 AddMessage(kTestMessage1); |
114 | 121 |
(...skipping 159 matching lines...) Expand 10 before | Expand all | Expand 10 after Loading... |
274 | 281 |
275 done_task.Run(); | 282 done_task.Run(); |
276 | 283 |
277 EXPECT_TRUE(socket_.read_pending()); | 284 EXPECT_TRUE(socket_.read_pending()); |
278 } | 285 } |
279 | 286 |
280 // Read() returns error. | 287 // Read() returns error. |
281 TEST_F(MessageReaderTest, ReadError) { | 288 TEST_F(MessageReaderTest, ReadError) { |
282 socket_.AppendReadError(net::ERR_FAILED); | 289 socket_.AppendReadError(net::ERR_FAILED); |
283 | 290 |
284 // Add a message. It should never be read after the error above. | 291 EXPECT_CALL(callback_, OnMessage(_)).Times(0); |
285 AddMessage(kTestMessage1); | |
286 | |
287 EXPECT_CALL(callback_, OnMessage(_)) | |
288 .Times(0); | |
289 | 292 |
290 InitReader(); | 293 InitReader(); |
| 294 |
| 295 EXPECT_EQ(net::ERR_FAILED, read_error_); |
| 296 EXPECT_FALSE(reader_); |
291 } | 297 } |
292 | 298 |
293 // Verify that we the OnMessage callback is not reentered. | 299 // Verify that we the OnMessage callback is not reentered. |
294 TEST_F(MessageReaderTest, ReadFromCallback) { | 300 TEST_F(MessageReaderTest, ReadFromCallback) { |
295 AddMessage(kTestMessage1); | 301 AddMessage(kTestMessage1); |
296 | 302 |
297 EXPECT_CALL(callback_, OnMessage(_)) | 303 EXPECT_CALL(callback_, OnMessage(_)) |
298 .Times(2) | 304 .Times(2) |
299 .WillOnce(Invoke(this, &MessageReaderTest::AddSecondMessage)) | 305 .WillOnce(Invoke(this, &MessageReaderTest::AddSecondMessage)) |
300 .WillOnce(Invoke(this, &MessageReaderTest::OnSecondMessage)); | 306 .WillOnce(Invoke(this, &MessageReaderTest::OnSecondMessage)); |
(...skipping 16 matching lines...) Expand all Loading... |
317 EXPECT_CALL(callback_, OnMessage(_)) | 323 EXPECT_CALL(callback_, OnMessage(_)) |
318 .Times(1) | 324 .Times(1) |
319 .WillOnce(Invoke(this, &MessageReaderTest::DeleteReader)); | 325 .WillOnce(Invoke(this, &MessageReaderTest::DeleteReader)); |
320 | 326 |
321 InitReader(); | 327 InitReader(); |
322 base::RunLoop().RunUntilIdle(); | 328 base::RunLoop().RunUntilIdle(); |
323 } | 329 } |
324 | 330 |
325 } // namespace protocol | 331 } // namespace protocol |
326 } // namespace remoting | 332 } // namespace remoting |
OLD | NEW |