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

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

Issue 2679893002: [Media Router] Add ProvideSinks() Mojo API (Closed)
Patch Set: Created 3 years, 10 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_mojo_impl_unittest.cc
diff --git a/chrome/browser/media/router/mojo/media_router_mojo_impl_unittest.cc b/chrome/browser/media/router/mojo/media_router_mojo_impl_unittest.cc
index cd8f2ed483ffed7e96e9aa75753ff55569dedc01..e5edffc9ba6521e1140d1699a4ff79f7ba01ba36 100644
--- a/chrome/browser/media/router/mojo/media_router_mojo_impl_unittest.cc
+++ b/chrome/browser/media/router/mojo/media_router_mojo_impl_unittest.cc
@@ -1181,6 +1181,31 @@ TEST_F(MediaRouterMojoImplTest, SearchSinks) {
run_loop.RunUntilIdle();
}
+TEST_F(MediaRouterMojoImplTest, OnSinksDiscovered) {
+ MediaRouter::MediaSinkList* sinks = new MediaRouter::MediaSinkList();
+ MediaSink* expected_sink =
+ new MediaSink(kSinkId, kSinkName, MediaSink::IconType::CAST);
+ sinks->push_back(base::WrapUnique(expected_sink));
+ EXPECT_EQ(expected_sink, sinks->at(0).get());
+
+ EXPECT_CALL(mock_media_route_provider_, OnSinksDiscovered_(_))
+ .WillOnce(Invoke([](const MediaRouter::MediaSinkList& actual_sinks) {
+ EXPECT_EQ(size_t(1), actual_sinks.size());
+ MediaSink* actual_sink = actual_sinks[0].get();
+ EXPECT_EQ(kSinkId, actual_sink->id());
+ EXPECT_EQ(kSinkName, actual_sink->name());
+ EXPECT_EQ(MediaSink::IconType::CAST, actual_sink->icon_type());
+ EXPECT_FALSE(actual_sink->description());
+ EXPECT_FALSE(actual_sink->domain());
+ EXPECT_FALSE(actual_sink->model_name());
+ }));
+
+ router()->OnSinksDiscovered(base::WrapUnique(sinks));
+
+ base::RunLoop run_loop;
+ run_loop.RunUntilIdle();
+}
+
class MediaRouterMojoExtensionTest : public ::testing::Test {
public:
MediaRouterMojoExtensionTest() : process_manager_(nullptr) {}

Powered by Google App Engine
This is Rietveld 408576698