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

Issues Search
    My Issues | Starred     Open | Closed | All

Unified Diff: chrome/browser/media/router/mojo/media_router_mojo_impl_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: Created 4 years, 6 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
« chrome/browser/media/router/media_route.h ('K') | « chrome/browser/media/router/mojo/media_router_mojo_impl.cc ('k') | chrome/browser/media/router/mojo/media_router_type_converters.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
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 50d5bc84b79d84c3667510652fff6f7d96cac74d..2fa889512e2381b6d901fbdc49a6510016f22238 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
@@ -114,7 +114,7 @@ interfaces::MediaRoutePtr CreateMojoRoute() {
route->description = kDescription;
route->is_local = true;
route->for_display = true;
- route->off_the_record = false;
+ route->incognito = false;
return route;
}
@@ -226,7 +226,7 @@ TEST_F(MediaRouterMojoImplTest, CreateRoute) {
.WillOnce(Invoke(
[](const mojo::String& source, const mojo::String& sink,
const mojo::String& presentation_id, const mojo::String& origin,
- int tab_id, int64_t timeout_millis, bool off_the_record,
+ int tab_id, int64_t timeout_millis, bool incognito,
const interfaces::MediaRouteProvider::CreateRouteCallback& cb) {
cb.Run(CreateMojoRoute(), mojo::String(),
interfaces::RouteRequestResultCode::OK);
@@ -250,7 +250,7 @@ TEST_F(MediaRouterMojoImplTest, CreateOffTheRecordRoute) {
MediaSource media_source(kSource);
MediaRoute expected_route(kRouteId, media_source, kSinkId, "", false, "",
false);
- expected_route.set_off_the_record(true);
+ expected_route.set_incognito(true);
// Use a lambda function as an invocation target here to work around
// a limitation with GMock::Invoke that prevents it from using move-only types
@@ -261,11 +261,11 @@ TEST_F(MediaRouterMojoImplTest, CreateOffTheRecordRoute) {
.WillOnce(Invoke(
[](const mojo::String& source, const mojo::String& sink,
const mojo::String& presentation_id, const mojo::String& origin,
- int tab_id, int64_t timeout_millis, bool off_the_record,
+ int tab_id, int64_t timeout_millis, bool incognito,
const interfaces::MediaRouteProvider::CreateRouteCallback& cb) {
interfaces::MediaRoutePtr route = CreateMojoRoute();
route->custom_controller_path = "custom/controller/path";
- route->off_the_record = true;
+ route->incognito = true;
cb.Run(std::move(route), mojo::String(),
interfaces::RouteRequestResultCode::OK);
}));
@@ -292,7 +292,7 @@ TEST_F(MediaRouterMojoImplTest, CreateRouteFails) {
.WillOnce(Invoke(
[](const mojo::String& source, const mojo::String& sink,
const mojo::String& presentation_id, const mojo::String& origin,
- int tab_id, int64_t timeout_millis, bool off_the_record,
+ int tab_id, int64_t timeout_millis, bool incognito,
const interfaces::MediaRouteProvider::CreateRouteCallback& cb) {
cb.Run(interfaces::MediaRoutePtr(), mojo::String(kError),
interfaces::RouteRequestResultCode::TIMED_OUT);
@@ -320,7 +320,7 @@ TEST_F(MediaRouterMojoImplTest, CreateRouteOffTheRecordMismatchFails) {
.WillOnce(Invoke(
[](const mojo::String& source, const mojo::String& sink,
const mojo::String& presentation_id, const mojo::String& origin,
- int tab_id, int64_t timeout_millis, bool off_the_record,
+ int tab_id, int64_t timeout_millis, bool incognito,
const interfaces::MediaRouteProvider::CreateRouteCallback& cb) {
cb.Run(CreateMojoRoute(), mojo::String(),
interfaces::RouteRequestResultCode::OK);
@@ -328,10 +328,9 @@ TEST_F(MediaRouterMojoImplTest, CreateRouteOffTheRecordMismatchFails) {
RouteResponseCallbackHandler handler;
base::RunLoop run_loop;
- std::string error(
- "Mismatch in off the record status: request = 1, response = 0");
+ std::string error("Mismatch in incognito status: request = 1, response = 0");
EXPECT_CALL(handler, DoInvoke(nullptr, "", error,
- RouteRequestResult::OFF_THE_RECORD_MISMATCH))
+ RouteRequestResult::INCOGNITO_MISMATCH))
.WillOnce(InvokeWithoutArgs([&run_loop]() { run_loop.Quit(); }));
std::vector<MediaRouteResponseCallback> route_response_callbacks;
route_response_callbacks.push_back(base::Bind(
@@ -344,7 +343,7 @@ TEST_F(MediaRouterMojoImplTest, CreateRouteOffTheRecordMismatchFails) {
TEST_F(MediaRouterMojoImplTest, OffTheRecordRoutesTerminatedOnProfileShutdown) {
interfaces::MediaRoutePtr route = CreateMojoRoute();
- route->off_the_record = true;
+ route->incognito = true;
EXPECT_CALL(mock_media_route_provider_,
CreateRoute(mojo::String(kSource), mojo::String(kSinkId), _,
@@ -353,10 +352,10 @@ TEST_F(MediaRouterMojoImplTest, OffTheRecordRoutesTerminatedOnProfileShutdown) {
.WillOnce(Invoke(
[](const mojo::String& source, const mojo::String& sink,
const mojo::String& presentation_id, const mojo::String& origin,
- int tab_id, int64_t timeout_millis, bool off_the_record,
+ int tab_id, int64_t timeout_millis, bool incognito,
const interfaces::MediaRouteProvider::CreateRouteCallback& cb) {
interfaces::MediaRoutePtr route = CreateMojoRoute();
- route->off_the_record = true;
+ route->incognito = true;
cb.Run(std::move(route), mojo::String(),
interfaces::RouteRequestResultCode::OK);
}));
@@ -405,7 +404,7 @@ TEST_F(MediaRouterMojoImplTest, JoinRoute) {
.WillOnce(Invoke([&route](
const mojo::String& source, const mojo::String& presentation_id,
const mojo::String& origin, int tab_id, int64_t timeout_millis,
- bool off_the_record,
+ bool incognito,
const interfaces::MediaRouteProvider::JoinRouteCallback& cb) {
cb.Run(std::move(route), mojo::String(),
interfaces::RouteRequestResultCode::OK);
@@ -456,7 +455,7 @@ TEST_F(MediaRouterMojoImplTest, JoinRouteTimedOutFails) {
.WillOnce(Invoke(
[](const mojo::String& source, const mojo::String& presentation_id,
const mojo::String& origin, int tab_id, int64_t timeout_millis,
- bool off_the_record,
+ bool incognito,
const interfaces::MediaRouteProvider::JoinRouteCallback& cb) {
cb.Run(interfaces::MediaRoutePtr(), mojo::String(kError),
interfaces::RouteRequestResultCode::TIMED_OUT);
@@ -497,7 +496,7 @@ TEST_F(MediaRouterMojoImplTest, JoinRouteOffTheRecordMismatchFails) {
.WillOnce(Invoke([&route](
const mojo::String& source, const mojo::String& presentation_id,
const mojo::String& origin, int tab_id, int64_t timeout_millis,
- bool off_the_record,
+ bool incognito,
const interfaces::MediaRouteProvider::JoinRouteCallback& cb) {
cb.Run(std::move(route), mojo::String(),
interfaces::RouteRequestResultCode::OK);
@@ -505,10 +504,9 @@ TEST_F(MediaRouterMojoImplTest, JoinRouteOffTheRecordMismatchFails) {
RouteResponseCallbackHandler handler;
base::RunLoop run_loop;
- std::string error(
- "Mismatch in off the record status: request = 1, response = 0");
+ std::string error("Mismatch in incognito status: request = 1, response = 0");
EXPECT_CALL(handler, DoInvoke(nullptr, "", error,
- RouteRequestResult::OFF_THE_RECORD_MISMATCH))
+ RouteRequestResult::INCOGNITO_MISMATCH))
.WillOnce(InvokeWithoutArgs([&run_loop]() { run_loop.Quit(); }));
std::vector<MediaRouteResponseCallback> route_response_callbacks;
route_response_callbacks.push_back(base::Bind(
@@ -523,7 +521,7 @@ TEST_F(MediaRouterMojoImplTest, ConnectRouteByRouteId) {
MediaSource media_source(kSource);
MediaRoute expected_route(kRouteId, media_source, kSinkId, "", false, "",
false);
- expected_route.set_off_the_record(false);
+ expected_route.set_incognito(false);
interfaces::MediaRoutePtr route = CreateMojoRoute();
// Use a lambda function as an invocation target here to work around
@@ -537,7 +535,7 @@ TEST_F(MediaRouterMojoImplTest, ConnectRouteByRouteId) {
.WillOnce(Invoke([&route](
const mojo::String& source, const mojo::String& route_id,
const mojo::String& presentation_id, const mojo::String& origin,
- int tab_id, int64_t timeout_millis, bool off_the_record,
+ int tab_id, int64_t timeout_millis, bool incognito,
const interfaces::MediaRouteProvider::JoinRouteCallback& cb) {
cb.Run(std::move(route), mojo::String(),
interfaces::RouteRequestResultCode::OK);
@@ -566,7 +564,7 @@ TEST_F(MediaRouterMojoImplTest, ConnectRouteByRouteIdFails) {
.WillOnce(Invoke(
[](const mojo::String& source, const mojo::String& route_id,
const mojo::String& presentation_id, const mojo::String& origin,
- int tab_id, int64_t timeout_millis, bool off_the_record,
+ int tab_id, int64_t timeout_millis, bool incognito,
const interfaces::MediaRouteProvider::JoinRouteCallback& cb) {
cb.Run(interfaces::MediaRoutePtr(), mojo::String(kError),
interfaces::RouteRequestResultCode::TIMED_OUT);
@@ -600,7 +598,7 @@ TEST_F(MediaRouterMojoImplTest, ConnectRouteByIdOffTheRecordMismatchFails) {
.WillOnce(Invoke([&route](
const mojo::String& source, const mojo::String& route_id,
const mojo::String& presentation_id, const mojo::String& origin,
- int tab_id, int64_t timeout_millis, bool off_the_record,
+ int tab_id, int64_t timeout_millis, bool incognito,
const interfaces::MediaRouteProvider::JoinRouteCallback& cb) {
cb.Run(std::move(route), mojo::String(),
interfaces::RouteRequestResultCode::OK);
@@ -608,10 +606,9 @@ TEST_F(MediaRouterMojoImplTest, ConnectRouteByIdOffTheRecordMismatchFails) {
RouteResponseCallbackHandler handler;
base::RunLoop run_loop;
- std::string error(
- "Mismatch in off the record status: request = 1, response = 0");
+ std::string error("Mismatch in incognito status: request = 1, response = 0");
EXPECT_CALL(handler, DoInvoke(nullptr, "", error,
- RouteRequestResult::OFF_THE_RECORD_MISMATCH))
+ RouteRequestResult::INCOGNITO_MISMATCH))
.WillOnce(InvokeWithoutArgs([&run_loop]() { run_loop.Quit(); }));
std::vector<MediaRouteResponseCallback> route_response_callbacks;
route_response_callbacks.push_back(base::Bind(
@@ -882,7 +879,7 @@ TEST_F(MediaRouterMojoImplTest, RegisterAndUnregisterMediaRoutesObserver) {
kDescription, false, "", false));
MediaRoute incognito_expected_route(kRouteId2, media_source, kSinkId,
kDescription, false, "", false);
- incognito_expected_route.set_off_the_record(true);
+ incognito_expected_route.set_incognito(true);
expected_routes.push_back(incognito_expected_route);
std::vector<MediaRoute::Id> expected_joinable_route_ids;
expected_joinable_route_ids.push_back(kJoinableRouteId);
@@ -896,7 +893,7 @@ TEST_F(MediaRouterMojoImplTest, RegisterAndUnregisterMediaRoutesObserver) {
mojo_routes[0] = CreateMojoRoute();
mojo_routes[1] = CreateMojoRoute();
mojo_routes[1]->media_route_id = kRouteId2;
- mojo_routes[1]->off_the_record = true;
+ mojo_routes[1]->incognito = true;
EXPECT_CALL(routes_observer, OnRoutesUpdated(SequenceEquals(expected_routes),
expected_joinable_route_ids));
« chrome/browser/media/router/media_route.h ('K') | « chrome/browser/media/router/mojo/media_router_mojo_impl.cc ('k') | chrome/browser/media/router/mojo/media_router_type_converters.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698