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

Unified Diff: chrome/browser/ui/webui/signin/signin_create_profile_handler_unittest.cc

Issue 2468723003: Move session service and supervised users to buildflags. (Closed)
Patch Set: Created 4 years, 1 month 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
« no previous file with comments | « chrome/browser/ui/webui/signin/signin_create_profile_handler.cc ('k') | chrome/common/BUILD.gn » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/browser/ui/webui/signin/signin_create_profile_handler_unittest.cc
diff --git a/chrome/browser/ui/webui/signin/signin_create_profile_handler_unittest.cc b/chrome/browser/ui/webui/signin/signin_create_profile_handler_unittest.cc
index c2f09ec003f465ecd15d1f108f726f1c1f1970cf..5c746b5eb9076c7bdb4388ce7a636d0294eb32a9 100644
--- a/chrome/browser/ui/webui/signin/signin_create_profile_handler_unittest.cc
+++ b/chrome/browser/ui/webui/signin/signin_create_profile_handler_unittest.cc
@@ -14,6 +14,7 @@
#include "chrome/browser/signin/signin_error_controller_factory.h"
#include "chrome/browser/signin/signin_manager_factory.h"
#include "chrome/browser/ui/webui/signin/signin_utils.h"
+#include "chrome/common/features.h"
#include "chrome/common/pref_names.h"
#include "chrome/grit/generated_resources.h"
#include "chrome/test/base/browser_with_test_window_test.h"
@@ -31,7 +32,7 @@
#include "testing/gtest/include/gtest/gtest.h"
#include "ui/base/l10n/l10n_util.h"
-#if defined(ENABLE_SUPERVISED_USERS)
+#if BUILDFLAG(ENABLE_SUPERVISED_USERS)
#include "chrome/browser/supervised_user/legacy/supervised_user_sync_service.h"
#include "chrome/browser/supervised_user/legacy/supervised_user_sync_service_factory.h"
#endif
@@ -52,7 +53,7 @@ const char kTestEmail2[] = "foo2@bar.com";
const char kTestWebUIResponse[] = "cr.webUIListenerCallback";
-#if defined(ENABLE_SUPERVISED_USERS)
+#if BUILDFLAG(ENABLE_SUPERVISED_USERS)
const char kSupervisedUserId1[] = "test-supervised-id-1";
const char kSupervisedUserId2[] = "test-supervised-id-2";
@@ -142,7 +143,7 @@ class TestSigninCreateProfileHandler : public SigninCreateProfileHandler {
Profile* custodian_profile,
Profile* new_profile));
-#if defined(ENABLE_SUPERVISED_USERS)
+#if BUILDFLAG(ENABLE_SUPERVISED_USERS)
// Calls the callback method to resume profile creation flow.
void RealRegisterSupervisedUser(bool create_shortcut,
const std::string& supervised_user_id,
@@ -191,7 +192,7 @@ class SigninCreateProfileHandlerTest : public BrowserWithTestWindowTest {
fake_signin_manager_->SetAuthenticatedAccountInfo(kTestGaiaId1,
kTestEmail1);
-#if defined(ENABLE_SUPERVISED_USERS)
+#if BUILDFLAG(ENABLE_SUPERVISED_USERS)
// Add supervised users to the custodian profile.
SupervisedUserSyncService* sync_service_ =
SupervisedUserSyncServiceFactory::GetForProfile(custodian_);
@@ -423,7 +424,7 @@ TEST_F(SigninCreateProfileHandlerTest, CreateProfileWithForceSignin) {
ASSERT_FALSE(show_confirmation);
}
-#if defined(ENABLE_SUPERVISED_USERS)
+#if BUILDFLAG(ENABLE_SUPERVISED_USERS)
TEST_F(SigninCreateProfileHandlerTest, CreateSupervisedUser) {
// Expect the call to create the profile.
« no previous file with comments | « chrome/browser/ui/webui/signin/signin_create_profile_handler.cc ('k') | chrome/common/BUILD.gn » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698