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

Unified Diff: chrome/browser/media/router/mojo/media_router_struct_traits_unittest.cc

Issue 2679893002: [Media Router] Add ProvideSinks() Mojo API (Closed)
Patch Set: fix android compile error Created 3 years, 8 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
Index: chrome/browser/media/router/mojo/media_router_struct_traits_unittest.cc
diff --git a/chrome/browser/media/router/mojo/media_router_struct_traits_unittest.cc b/chrome/browser/media/router/mojo/media_router_struct_traits_unittest.cc
index 753c43157d63e6b3135153d228170b21b9839b5b..08be65764e8403b060f903f541f8c5b618268e6e 100644
--- a/chrome/browser/media/router/mojo/media_router_struct_traits_unittest.cc
+++ b/chrome/browser/media/router/mojo/media_router_struct_traits_unittest.cc
@@ -57,12 +57,7 @@ TEST_F(MediaRouterStructTraitsTest, DialMediaSink) {
MediaSinkInternal output;
proxy->EchoMediaSink(dial_sink, &output);
- EXPECT_EQ(sink_id, output.sink().id());
- EXPECT_EQ(sink_name, output.sink().name());
- EXPECT_EQ(icon_type, output.sink().icon_type());
- EXPECT_EQ(ip_address, output.dial_data().ip_address.ToString());
- EXPECT_EQ(model_name, output.dial_data().model_name);
- EXPECT_EQ(app_url, output.dial_data().app_url);
+ EXPECT_EQ(dial_sink, output);
}
TEST_F(MediaRouterStructTraitsTest, CastMediaSink) {
@@ -85,13 +80,7 @@ TEST_F(MediaRouterStructTraitsTest, CastMediaSink) {
MediaSinkInternal output;
proxy->EchoMediaSink(cast_sink, &output);
- EXPECT_EQ(sink_id, output.sink().id());
- EXPECT_EQ(sink_name, output.sink().name());
- EXPECT_EQ(icon_type, output.sink().icon_type());
- EXPECT_EQ(ip_address, output.cast_data().ip_address.ToString());
- EXPECT_EQ(model_name, output.cast_data().model_name);
- EXPECT_EQ(2, output.cast_data().capabilities);
- EXPECT_EQ(3, output.cast_data().cast_channel_id);
+ EXPECT_EQ(cast_sink, output);
}
TEST_F(MediaRouterStructTraitsTest, GenericMediaSink) {
@@ -107,11 +96,7 @@ TEST_F(MediaRouterStructTraitsTest, GenericMediaSink) {
MediaSinkInternal output;
proxy->EchoMediaSink(generic_sink, &output);
- EXPECT_EQ(sink_id, output.sink().id());
- EXPECT_EQ(sink_name, output.sink().name());
- EXPECT_EQ(icon_type, output.sink().icon_type());
- EXPECT_FALSE(output.is_cast_sink());
- EXPECT_FALSE(output.is_dial_sink());
+ EXPECT_EQ(generic_sink, output);
}
} // namespace media_router
« no previous file with comments | « chrome/browser/media/router/mojo/media_router_mojo_test.h ('k') | extensions/renderer/resources/media_router_bindings.js » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698