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

Unified Diff: blimp/net/engine_connection_manager_unittest.cc

Issue 2462183002: GRPC Stream implementation of HeliumStream
Patch Set: Fixed a few minor comments Created 4 years, 1 month 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
Index: blimp/net/engine_connection_manager_unittest.cc
diff --git a/blimp/net/engine_connection_manager_unittest.cc b/blimp/net/engine_connection_manager_unittest.cc
index f9428068066e4d650647a60a149c309e08394b78..3488f68f69602e9cde8eaebe1c72e6ce34f67557 100644
--- a/blimp/net/engine_connection_manager_unittest.cc
+++ b/blimp/net/engine_connection_manager_unittest.cc
@@ -13,6 +13,7 @@
#include "base/memory/ptr_util.h"
#include "base/message_loop/message_loop.h"
#include "base/run_loop.h"
+#include "blimp/common/assignment_options.h"
#include "blimp/net/blimp_transport.h"
#include "blimp/net/common.h"
#include "blimp/net/connection_error_observer.h"
@@ -51,13 +52,13 @@ TEST_F(EngineConnectionManagerTest, ConnectionSucceeds) {
EXPECT_CALL(connection_handler_, HandleConnectionPtr(_)).Times(1);
net::IPAddress ip_addr(net::IPAddress::IPv4Localhost());
- net::IPEndPoint engine_endpoint(ip_addr, 0);
-
- manager_->ConnectTransport(&engine_endpoint,
+ AssignmentOptions assignment_options;
+ assignment_options.engine_endpoint = net::IPEndPoint(ip_addr, 0);
+ manager_->ConnectTransport(&assignment_options,
EngineConnectionManager::EngineTransportType::TCP);
net::TestCompletionCallback connect_callback;
- TCPClientTransport client(engine_endpoint, nullptr);
+ TCPClientTransport client(assignment_options.engine_endpoint, nullptr);
client.Connect(connect_callback.callback());
EXPECT_EQ(net::OK, connect_callback.WaitForResult());
}

Powered by Google App Engine
This is Rietveld 408576698