Commit 9883c54b authored by Owen Min's avatar Owen Min Committed by Commit Bot

Delete unused policy code

Some Java code in policy components are kept due to internal code base
references. Those references are removed hence those code can be clean
up now.

Bug: 1127469
Change-Id: I36021a4e73c988ccc391e89a28815224a98bba06
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2414890
Auto-Submit: Owen Min <zmin@chromium.org>
Commit-Queue: Michael Thiessen <mthiesse@chromium.org>
Reviewed-by: default avatarMichael Thiessen <mthiesse@chromium.org>
Cr-Commit-Position: refs/heads/master@{#808167}
parent 138be98c
...@@ -262,13 +262,6 @@ public abstract class AppHooks { ...@@ -262,13 +262,6 @@ public abstract class AppHooks {
new AppRestrictionsProvider(ContextUtils.getApplicationContext())); new AppRestrictionsProvider(ContextUtils.getApplicationContext()));
} }
// TODO(zmin): Delete once the internal code is migrated.
public void registerPolicyProviders(
org.chromium.policy.CombinedPolicyProvider combinedProvider) {
combinedProvider.registerProvider(
new AppRestrictionsProvider(ContextUtils.getApplicationContext()));
}
/** /**
* TODO(crbug.com/1102812) : Remove this method after updating the downstream to use the new * TODO(crbug.com/1102812) : Remove this method after updating the downstream to use the new
* method {@link getOfflinePagesCctAllowlist} instead. * method {@link getOfflinePagesCctAllowlist} instead.
......
...@@ -27,9 +27,6 @@ android_library("policy_java") { ...@@ -27,9 +27,6 @@ android_library("policy_java") {
annotation_processor_deps = [ "//base/android/jni_generator:jni_processor" ] annotation_processor_deps = [ "//base/android/jni_generator:jni_processor" ]
sources = _jni_sources + [ sources = _jni_sources + [
"java/src/org/chromium/policy/AppRestrictionsProvider.java",
"java/src/org/chromium/policy/CombinedPolicyProvider.java",
"java/src/org/chromium/policy/PolicyProvider.java",
"java/src/org/chromium/components/policy/AbstractAppRestrictionsProvider.java", "java/src/org/chromium/components/policy/AbstractAppRestrictionsProvider.java",
"java/src/org/chromium/components/policy/AppRestrictionsProvider.java", "java/src/org/chromium/components/policy/AppRestrictionsProvider.java",
"java/src/org/chromium/components/policy/PolicyProvider.java", "java/src/org/chromium/components/policy/PolicyProvider.java",
......
// Copyright 2015 The Chromium Authors. All rights reserved.
// Use of this source code is governed by a BSD-style license that can be
// found in the LICENSE file.
package org.chromium.policy;
import android.content.Context;
/**
* Concrete app restriction provider, that uses the default android mechanism to retrieve the
* restrictions.
* TODO(zmin): Delete once the internal code is migrated.
*/
public class AppRestrictionsProvider
extends org.chromium.components.policy.AppRestrictionsProvider {
public AppRestrictionsProvider(Context context) {
super(context);
}
}
// Copyright 2015 The Chromium Authors. All rights reserved.
// Use of this source code is governed by a BSD-style license that can be
// found in the LICENSE file.
package org.chromium.policy;
/**
* Reads enterprise policies from one or more policy providers and plumbs them through to the policy
* subsystem.
*
* TODO(zmin): Delete once the internal code is migrated.
*/
public class CombinedPolicyProvider extends org.chromium.components.policy.CombinedPolicyProvider {}
// Copyright 2015 The Chromium Authors. All rights reserved.
// Use of this source code is governed by a BSD-style license that can be
// found in the LICENSE file.
package org.chromium.policy;
/**
* Base class for Policy providers.
* TODO(zmin): Delete once the internal code is migrated.
*/
public abstract class PolicyProvider extends org.chromium.components.policy.PolicyProvider {}
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