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

Unified Diff: chrome/browser/ui/webui/media_router/media_router_webui_message_handler_unittest.cc

Issue 2112673002: Removing references to Off the Record from MediaRouter (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: More refactoring on rebased code Created 4 years, 5 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/ui/webui/media_router/media_router_webui_message_handler_unittest.cc
diff --git a/chrome/browser/ui/webui/media_router/media_router_webui_message_handler_unittest.cc b/chrome/browser/ui/webui/media_router/media_router_webui_message_handler_unittest.cc
index df2c79a3ea576ee4150772475d986bb995145df9..b39f90e00d29f4ec89528757cbfb65487101ab43 100644
--- a/chrome/browser/ui/webui/media_router/media_router_webui_message_handler_unittest.cc
+++ b/chrome/browser/ui/webui/media_router/media_router_webui_message_handler_unittest.cc
@@ -385,8 +385,8 @@ TEST_F(MediaRouterWebUIMessageHandlerTest, UpdateRoutes) {
EXPECT_EQ(expected_path, custom_controller_path);
}
-TEST_F(MediaRouterWebUIMessageHandlerTest, UpdateRoutesOffTheRecord) {
- handler_->set_off_the_record_for_test(true);
+TEST_F(MediaRouterWebUIMessageHandlerTest, UpdateRoutesIncognito) {
+ handler_->set_incognito_for_test(true);
MediaRoute::Id route_id("routeId123");
MediaSink::Id sink_id("sinkId123");
@@ -441,10 +441,10 @@ TEST_F(MediaRouterWebUIMessageHandlerTest, OnCreateRouteResponseReceived) {
std::string description("This is a route");
bool is_local = true;
bool is_for_display = true;
- bool off_the_record = false;
+ bool incognito = false;
MediaRoute route(route_id, MediaSource("mediaSource"), sink_id, description,
is_local, kControllerPathForTesting, is_for_display);
- route.set_off_the_record(off_the_record);
+ route.set_incognito(incognito);
EXPECT_CALL(*mock_media_router_ui_, GetRouteProviderExtensionId())
.WillOnce(ReturnRef(provider_extension_id()));
@@ -487,8 +487,8 @@ TEST_F(MediaRouterWebUIMessageHandlerTest, OnCreateRouteResponseReceived) {
}
TEST_F(MediaRouterWebUIMessageHandlerTest,
- OnCreateRouteResponseReceivedOffTheRecord) {
- handler_->set_off_the_record_for_test(true);
+ OnCreateRouteResponseReceivedIncognito) {
+ handler_->set_incognito_for_test(true);
MediaRoute::Id route_id("routeId123");
MediaSink::Id sink_id("sinkId123");
@@ -496,10 +496,10 @@ TEST_F(MediaRouterWebUIMessageHandlerTest,
std::string description("This is a route");
bool is_local = true;
bool is_for_display = true;
- bool off_the_record = true;
+ bool incognito = true;
MediaRoute route(route_id, MediaSource("mediaSource"), sink_id, description,
is_local, kControllerPathForTesting, is_for_display);
- route.set_off_the_record(off_the_record);
+ route.set_incognito(incognito);
EXPECT_CALL(*mock_media_router_ui_, GetRouteProviderExtensionId()).WillOnce(
ReturnRef(provider_extension_id()));

Powered by Google App Engine
This is Rietveld 408576698