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

Unified Diff: blimp/common/create_blimp_message_unittest.cc

Issue 1492643003: [Blimp Net] Add EngineAuthHandler. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Addresses wez's comments Created 5 years 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/common/create_blimp_message_unittest.cc
diff --git a/blimp/common/create_blimp_message_unittest.cc b/blimp/common/create_blimp_message_unittest.cc
index 1db89566f135fe26cfc4691ba79c09a5be9a1c4a..424b078a563dd325302bab1871fe66d95c6eb1af 100644
--- a/blimp/common/create_blimp_message_unittest.cc
+++ b/blimp/common/create_blimp_message_unittest.cc
@@ -40,5 +40,20 @@ TEST(CreateBlimpMessageTest, RenderWidgetMessage) {
EXPECT_EQ(kTabId, message->target_tab_id());
}
+TEST(CreateBlimpMessageTest, StartConnectionMessage) {
+ std::string client_token("token");
+ int protocol_version = 1;
Wez 2015/12/08 22:59:59 nit: These can both be const, I think; the string
haibinlu 2015/12/09 00:02:33 Done.
+ scoped_ptr<BlimpMessage> message =
+ CreateStartConnectionMessage(client_token, protocol_version);
+ EXPECT_NE(nullptr, message);
+ EXPECT_EQ(BlimpMessage::PROTOCOL_CONTROL, message->type());
+ EXPECT_EQ(ProtocolControlMessage::START_CONNECTION,
+ message->protocol_control().type());
+ EXPECT_EQ(client_token,
+ message->protocol_control().start_connection().client_token());
+ EXPECT_EQ(protocol_version,
+ message->protocol_control().start_connection().protocol_version());
+}
+
} // namespace
} // namespace blimp

Powered by Google App Engine
This is Rietveld 408576698