Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(365)

Unified Diff: remoting/test/fake_socket_factory.cc

Issue 2464613002: Update remoting_perftest to evaluate bandwidth estimation. (Closed)
Patch Set: Created 4 years, 2 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « no previous file | remoting/test/protocol_perftest.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: remoting/test/fake_socket_factory.cc
diff --git a/remoting/test/fake_socket_factory.cc b/remoting/test/fake_socket_factory.cc
index 6313fbf19fb664c802f862a25e4f78bd16f267b4..f639d8e29c5a860da27c2cc1298bf9cb545cd4f6 100644
--- a/remoting/test/fake_socket_factory.cc
+++ b/remoting/test/fake_socket_factory.cc
@@ -151,8 +151,8 @@ int FakeUdpSocket::GetOption(rtc::Socket::Option option, int* value) {
}
int FakeUdpSocket::SetOption(rtc::Socket::Option option, int value) {
- NOTIMPLEMENTED();
- return -1;
+ // All options are currently ignored.
+ return 0;
}
int FakeUdpSocket::GetError() const {
« no previous file with comments | « no previous file | remoting/test/protocol_perftest.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698