Commit d9af708a authored by Timothy Loh's avatar Timothy Loh Committed by Commit Bot

Remove kCrostiniAppUninstallGui feature

The kCrostiniAppUninstallGui feature was turned on by default in M76 so we no
longer need this feature.

Bug: 955797
Change-Id: I559149a37b8415ef67dc82cd2bac92456ad65817
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1743297
Auto-Submit: Timothy Loh <timloh@chromium.org>
Commit-Queue: Nic Hollingum <hollingum@google.com>
Reviewed-by: default avatarNic Hollingum <hollingum@google.com>
Cr-Commit-Position: refs/heads/master@{#685451}
parent 277db56f
......@@ -10,9 +10,7 @@
#include "chrome/browser/chromeos/crostini/crostini_registry_service.h"
#include "chrome/browser/chromeos/crostini/crostini_registry_service_factory.h"
#include "chrome/browser/chromeos/crostini/crostini_util.h"
#include "chrome/common/chrome_features.h"
#include "chrome/grit/generated_resources.h"
#include "ui/base/ui_base_features.h"
CrostiniAppContextMenu::CrostiniAppContextMenu(
Profile* profile,
......@@ -33,10 +31,6 @@ bool CrostiniAppContextMenu::IsUninstallable() const {
return true;
}
if (!base::FeatureList::IsEnabled(features::kCrostiniAppUninstallGui)) {
return false;
}
crostini::CrostiniRegistryService* registry_service =
crostini::CrostiniRegistryServiceFactory::GetForProfile(profile());
base::Optional<crostini::CrostiniRegistryService::Registration> registration =
......
......@@ -185,12 +185,6 @@ const base::Feature kCrostiniAdditionalEnterpriseReporting{
const base::Feature kCrostiniAdvancedAccessControls{
"CrostiniAdvancedAccessControls", base::FEATURE_DISABLED_BY_DEFAULT};
// Enables an uninstall option in the right-click menu of Crostini (Linux)
// applications.
// TODO(crbug.com/955797): Remove this flag entirely.
const base::Feature kCrostiniAppUninstallGui{"CrostiniAppUninstallGui",
base::FEATURE_ENABLED_BY_DEFAULT};
// Enables infrastructure for applying Ansible playbook to default Crostini
// container.
const base::Feature kCrostiniAnsibleInfrastructure{
......
......@@ -114,8 +114,6 @@ extern const base::Feature kCrostiniAdditionalEnterpriseReporting;
COMPONENT_EXPORT(CHROME_FEATURES)
extern const base::Feature kCrostiniAdvancedAccessControls;
COMPONENT_EXPORT(CHROME_FEATURES)
extern const base::Feature kCrostiniAppUninstallGui;
COMPONENT_EXPORT(CHROME_FEATURES)
extern const base::Feature kCrostiniAnsibleInfrastructure;
COMPONENT_EXPORT(CHROME_FEATURES)
extern const base::Feature kCupsPrintersUiOverhaul;
......
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