Commit bed4259f authored by Lucas Tenório's avatar Lucas Tenório Committed by Commit Bot

Only show the Supervision Onboarding to supervised accounts.

Bug: 958995
Change-Id: I115ba68ed9b58a6c0d50d1d94d52b1070f3837db
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1600133
Commit-Queue: Lucas Tenório <ltenorio@chromium.org>
Reviewed-by: default avatarToni Baržić <tbarzic@chromium.org>
Cr-Commit-Position: refs/heads/master@{#661108}
parent 887e9e6f
...@@ -7,6 +7,7 @@ ...@@ -7,6 +7,7 @@
#include "base/logging.h" #include "base/logging.h"
#include "chrome/browser/ui/webui/chromeos/login/supervision_onboarding_screen_handler.h" #include "chrome/browser/ui/webui/chromeos/login/supervision_onboarding_screen_handler.h"
#include "chromeos/constants/chromeos_features.h" #include "chromeos/constants/chromeos_features.h"
#include "components/user_manager/user_manager.h"
namespace chromeos { namespace chromeos {
namespace { namespace {
...@@ -31,10 +32,14 @@ SupervisionOnboardingScreen::~SupervisionOnboardingScreen() { ...@@ -31,10 +32,14 @@ SupervisionOnboardingScreen::~SupervisionOnboardingScreen() {
} }
void SupervisionOnboardingScreen::Show() { void SupervisionOnboardingScreen::Show() {
// TODO(ltenorio): Show this screen only for supervised accounts when the const user_manager::UserManager* user_manager =
// test support is improved by b/959244. user_manager::UserManager::Get();
if (view_ && base::FeatureList::IsEnabled( DCHECK(user_manager->IsUserLoggedIn());
features::kEnableSupervisionOnboardingScreens)) {
if (view_ &&
base::FeatureList::IsEnabled(
features::kEnableSupervisionOnboardingScreens) &&
user_manager->IsLoggedInAsChildUser()) {
view_->Show(); view_->Show();
return; return;
} }
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment