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

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

Issue 2837363002: [Media Router] Use DialMediaSinkService in MediaRouterMojoImpl (Closed)
Patch Set: 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_mojo_test.cc
diff --git a/chrome/browser/media/router/mojo/media_router_mojo_test.cc b/chrome/browser/media/router/mojo/media_router_mojo_test.cc
index d88c9649ee66492920e2f9a33701084de4617d1c..9ed877fa8379bd3355b9fda2d4717bada7bf1e35 100644
--- a/chrome/browser/media/router/mojo/media_router_mojo_test.cc
+++ b/chrome/browser/media/router/mojo/media_router_mojo_test.cc
@@ -11,14 +11,7 @@
namespace media_router {
namespace {
-
const char kInstanceId[] = "instance123";
-
-void ExpectEqualStrings(const std::string& expected,
- const std::string& actual) {
- EXPECT_EQ(expected, actual);
-}
-
} // namespace
MockMediaRouteProvider::MockMediaRouteProvider() {}
@@ -52,7 +45,9 @@ MockMediaRouteControllerObserver::MockMediaRouteControllerObserver(
MockMediaRouteControllerObserver::~MockMediaRouteControllerObserver() {}
MediaRouterMojoTest::MediaRouterMojoTest()
- : mock_media_router_(new MediaRouterMojoImpl(&mock_event_page_tracker_)) {
+ : profile_(new TestingProfile()),
+ mock_media_router_(
+ new MediaRouterMojoImpl(&mock_event_page_tracker_, profile_.get())) {
mock_media_router_->Initialize();
mock_media_router_->set_instance_id_for_test(kInstanceId);
extension_ = extensions::test_util::CreateEmptyExtension();
@@ -70,9 +65,9 @@ void MediaRouterMojoTest::ConnectProviderManagerService() {
mojom::MediaRouteProviderPtr mojo_media_router;
binding_.reset(new mojo::Binding<mojom::MediaRouteProvider>(
&mock_media_route_provider_, mojo::MakeRequest(&mojo_media_router)));
+ EXPECT_CALL(mock_register_callback_, Run(kInstanceId, false));
media_router_proxy_->RegisterMediaRouteProvider(
- std::move(mojo_media_router),
- base::Bind(&ExpectEqualStrings, kInstanceId));
+ std::move(mojo_media_router), mock_register_callback_.Get());
}
void MediaRouterMojoTest::SetUp() {

Powered by Google App Engine
This is Rietveld 408576698