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

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

Issue 2675033002: [Media Router] Add MediaSink subtypes (Closed)
Patch Set: resolve code review comments from Derek and Mark 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 45ed33577bc67a3e8bdaa7d4f1b550ea64c1e3ff..1cb5c080ee98ee285406f0328f5f33810225f8e2 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
@@ -696,13 +696,17 @@ TEST_F(MediaRouterMojoImplTest, RegisterAndUnregisterMediaSinksObserver) {
expected_sinks.push_back(
MediaSink(kSinkId2, kSinkName, MediaSink::IconType::CAST));
+ std::vector<MediaSinkInternal> sinks;
+ sinks.push_back(MediaSinkInternal(expected_sinks[0]));
+ sinks.push_back(MediaSinkInternal(expected_sinks[1]));
+
base::RunLoop run_loop;
EXPECT_CALL(*sinks_observer, OnSinksReceived(SequenceEquals(expected_sinks)));
EXPECT_CALL(*extra_sinks_observer,
OnSinksReceived(SequenceEquals(expected_sinks)))
.WillOnce(InvokeWithoutArgs([&run_loop]() { run_loop.Quit(); }));
media_router_proxy_->OnSinksReceived(
- media_source.id(), expected_sinks,
+ media_source.id(), sinks,
std::vector<url::Origin>(1, url::Origin(GURL(kOrigin))));
run_loop.Run();

Powered by Google App Engine
This is Rietveld 408576698