New issue
Advanced search Search tips
Note: Color blocks (like or ) mean that a user may not be available. Tooltip shows the reason.

Issue 866578 link

Starred by 1 user

Issue metadata

Status: Fixed
Owner:
Closed: Jul 24
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Linux , Windows , Mac
Pri: 1
Type: Bug



Sign in to add a comment

Disable legacy supervised users on desktop

Project Member Reported by jochen@chromium.org, Jul 23

Issue description

after showing an info bar in M69 announcing the end of life in M70 ( issue 865157 ), we should hide legacy supervised user profiles.

After a few milestones, we should GC them (will file a separate issue for this)
 
Project Member

Comment 1 by bugdroid1@chromium.org, Jul 23

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/d84bb42a2147fde3a3aaacd3ff64baf9bc28c914

commit d84bb42a2147fde3a3aaacd3ff64baf9bc28c914
Author: Jochen Eisinger <jochen@chromium.org>
Date: Mon Jul 23 22:39:03 2018

Stop opening legacy supervised users profiles

Omit them from the profile info cache, and show a profile picker if
the default profile is supervised.

BUG= 866578 
R=bauerb@chromium.org,tschumann@chromium.org,msarda@chromium.org

Change-Id: Ice583daee39eb1efc987af6a16de256d5bb1fc65
Reviewed-on: https://chromium-review.googlesource.com/1145271
Commit-Queue: Jochen Eisinger <jochen@chromium.org>
Reviewed-by: Carlos IL <carlosil@chromium.org>
Reviewed-by: Bernhard Bauer <bauerb@chromium.org>
Cr-Commit-Position: refs/heads/master@{#577307}
[modify] https://crrev.com/d84bb42a2147fde3a3aaacd3ff64baf9bc28c914/chrome/browser/chromeos/accessibility/accessibility_manager_browsertest.cc
[modify] https://crrev.com/d84bb42a2147fde3a3aaacd3ff64baf9bc28c914/chrome/browser/extensions/api/webstore_private/webstore_private_apitest.cc
[modify] https://crrev.com/d84bb42a2147fde3a3aaacd3ff64baf9bc28c914/chrome/browser/extensions/webstore_startup_installer_browsertest.cc
[modify] https://crrev.com/d84bb42a2147fde3a3aaacd3ff64baf9bc28c914/chrome/browser/profiles/profile_impl.cc
[modify] https://crrev.com/d84bb42a2147fde3a3aaacd3ff64baf9bc28c914/chrome/browser/profiles/profile_info_cache.cc
[modify] https://crrev.com/d84bb42a2147fde3a3aaacd3ff64baf9bc28c914/chrome/browser/profiles/profile_info_cache_unittest.cc
[modify] https://crrev.com/d84bb42a2147fde3a3aaacd3ff64baf9bc28c914/chrome/browser/profiles/profile_list_desktop_unittest.cc
[modify] https://crrev.com/d84bb42a2147fde3a3aaacd3ff64baf9bc28c914/chrome/browser/profiles/profile_manager_unittest.cc
[modify] https://crrev.com/d84bb42a2147fde3a3aaacd3ff64baf9bc28c914/chrome/browser/supervised_user/supervised_user_browsertest.cc
[modify] https://crrev.com/d84bb42a2147fde3a3aaacd3ff64baf9bc28c914/chrome/browser/supervised_user/supervised_user_interstitial.cc
[modify] https://crrev.com/d84bb42a2147fde3a3aaacd3ff64baf9bc28c914/chrome/browser/supervised_user/supervised_user_navigation_throttle_browsertest.cc
[modify] https://crrev.com/d84bb42a2147fde3a3aaacd3ff64baf9bc28c914/chrome/browser/supervised_user/supervised_user_service_browsertest.cc
[modify] https://crrev.com/d84bb42a2147fde3a3aaacd3ff64baf9bc28c914/chrome/browser/ui/cocoa/profiles/profile_menu_controller_unittest.mm
[modify] https://crrev.com/d84bb42a2147fde3a3aaacd3ff64baf9bc28c914/chrome/browser/ui/startup/startup_browser_creator_browsertest.cc
[delete] https://crrev.com/ea0ed30f2e414844d319851ccc332c3f70d25b2a/chrome/test/data/extensions/api_test/webstore_private/begin_install_fail_supervised.html
[modify] https://crrev.com/d84bb42a2147fde3a3aaacd3ff64baf9bc28c914/content/browser/frame_host/interstitial_page_impl.cc
[modify] https://crrev.com/d84bb42a2147fde3a3aaacd3ff64baf9bc28c914/content/public/browser/interstitial_page.h

Status: Fixed (was: Assigned)

Sign in to add a comment