Commit d425d744 authored by Bettina's avatar Bettina Committed by Commit Bot

Remove use local blacklist experiment from Android

The experiment is no longer used anymore.

Bug: 1054179
Fixes: 1054179
Change-Id: Ie00127cd5327e3313e7684db25797cd14d58ca70
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2067563Reviewed-by: default avatarXinghui Lu <xinghuilu@chromium.org>
Reviewed-by: default avatarDaniel Rubery <drubery@chromium.org>
Commit-Queue: Bettina Dea <bdea@chromium.org>
Cr-Commit-Position: refs/heads/master@{#745271}
parent c7b0a365
...@@ -56,7 +56,7 @@ public final class SafeBrowsingApiBridge { ...@@ -56,7 +56,7 @@ public final class SafeBrowsingApiBridge {
SafeBrowsingApiBridgeJni.get().onUrlCheckDone( SafeBrowsingApiBridgeJni.get().onUrlCheckDone(
callbackId, resultStatus, metadata, checkDelta); callbackId, resultStatus, metadata, checkDelta);
} }
}, SafeBrowsingApiBridgeJni.get().areLocalBlacklistsEnabled()); });
return initSuccesssful ? handler : null; return initSuccesssful ? handler : null;
} }
...@@ -97,7 +97,6 @@ public final class SafeBrowsingApiBridge { ...@@ -97,7 +97,6 @@ public final class SafeBrowsingApiBridge {
@NativeMethods @NativeMethods
interface Natives { interface Natives {
boolean areLocalBlacklistsEnabled();
void onUrlCheckDone(long callbackId, int resultStatus, String metadata, long checkDelta); void onUrlCheckDone(long callbackId, int resultStatus, String metadata, long checkDelta);
} }
} }
...@@ -106,11 +106,6 @@ static PendingCallbacksMap* GetPendingCallbacksMapOnIOThread() { ...@@ -106,11 +106,6 @@ static PendingCallbacksMap* GetPendingCallbacksMapOnIOThread() {
} // namespace } // namespace
// Java->Native call, to check whether the feature to use local blacklists is
// enabled.
jboolean JNI_SafeBrowsingApiBridge_AreLocalBlacklistsEnabled(JNIEnv* env) {
return base::FeatureList::IsEnabled(kUseLocalBlacklistsV2);
}
// Respond to the URL reputation request by looking up the callback information // Respond to the URL reputation request by looking up the callback information
// stored in |pending_callbacks|. // stored in |pending_callbacks|.
......
...@@ -106,10 +106,6 @@ const base::Feature kThreatDomDetailsTagAndAttributeFeature{ ...@@ -106,10 +106,6 @@ const base::Feature kThreatDomDetailsTagAndAttributeFeature{
const base::Feature kTriggerThrottlerDailyQuotaFeature{ const base::Feature kTriggerThrottlerDailyQuotaFeature{
"SafeBrowsingTriggerThrottlerDailyQuota", "SafeBrowsingTriggerThrottlerDailyQuota",
base::FEATURE_DISABLED_BY_DEFAULT}; base::FEATURE_DISABLED_BY_DEFAULT};
const base::Feature kUseLocalBlacklistsV2{"SafeBrowsingUseLocalBlacklistsV2",
base::FEATURE_DISABLED_BY_DEFAULT};
const base::Feature kUseNewDownloadWarnings{"UseNewDownloadWarnings", const base::Feature kUseNewDownloadWarnings{"UseNewDownloadWarnings",
base::FEATURE_DISABLED_BY_DEFAULT}; base::FEATURE_DISABLED_BY_DEFAULT};
...@@ -141,7 +137,6 @@ constexpr struct { ...@@ -141,7 +137,6 @@ constexpr struct {
{&kSuspiciousSiteTriggerQuotaFeature, true}, {&kSuspiciousSiteTriggerQuotaFeature, true},
{&kThreatDomDetailsTagAndAttributeFeature, false}, {&kThreatDomDetailsTagAndAttributeFeature, false},
{&kTriggerThrottlerDailyQuotaFeature, false}, {&kTriggerThrottlerDailyQuotaFeature, false},
{&kUseLocalBlacklistsV2, true},
}; };
// Adds the name and the enabled/disabled status of a given feature. // Adds the name and the enabled/disabled status of a given feature.
......
...@@ -99,9 +99,6 @@ extern const base::Feature kThreatDomDetailsTagAndAttributeFeature; ...@@ -99,9 +99,6 @@ extern const base::Feature kThreatDomDetailsTagAndAttributeFeature;
// trials simultaneously. // trials simultaneously.
extern const base::Feature kTriggerThrottlerDailyQuotaFeature; extern const base::Feature kTriggerThrottlerDailyQuotaFeature;
// Controls whether Chrome on Android uses locally cached blacklists.
extern const base::Feature kUseLocalBlacklistsV2;
// Controls whether Chrome uses new download warning UX. // Controls whether Chrome uses new download warning UX.
extern const base::Feature kUseNewDownloadWarnings; extern const base::Feature kUseNewDownloadWarnings;
......
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