Commit d609b38e authored by Xinghui Lu's avatar Xinghui Lu Committed by Commit Bot

Remove safebrowsing enableLocalBlacklists interface.

The implementations in the internal repository was removed, so it is
safe to remove the
"init(Observer result, boolean enableLocalBlacklists)" interface now.

Bug: 1054179
Change-Id: I6756c7d551fa57fb7e36fe841025a0c3003d9e05
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2151286Reviewed-by: default avatarNate Fischer <ntfschr@chromium.org>
Reviewed-by: default avatarYaron Friedman <yfriedman@chromium.org>
Reviewed-by: default avatarDaniel Rubery <drubery@chromium.org>
Commit-Queue: Xinghui Lu <xinghuilu@chromium.org>
Cr-Commit-Position: refs/heads/master@{#760733}
parent d60868c2
...@@ -154,11 +154,6 @@ public class SafeBrowsingTest { ...@@ -154,11 +154,6 @@ public class SafeBrowsingTest {
@Override @Override
public boolean init(Observer result) { public boolean init(Observer result) {
return init(result, false);
}
@Override
public boolean init(Observer result, boolean enableLocalBlacklists) {
mObserver = result; mObserver = result;
return true; return true;
} }
......
...@@ -41,11 +41,6 @@ public class MockSafeBrowsingApiHandler implements SafeBrowsingApiHandler { ...@@ -41,11 +41,6 @@ public class MockSafeBrowsingApiHandler implements SafeBrowsingApiHandler {
@Override @Override
public boolean init(Observer observer) { public boolean init(Observer observer) {
return init(observer, false);
}
@Override
public boolean init(Observer observer, boolean enableLocalBlacklists) {
mObserver = observer; mObserver = observer;
mResponseMap = new HashMap<String, String>(sResponseMap); mResponseMap = new HashMap<String, String>(sResponseMap);
return true; return true;
......
...@@ -46,21 +46,6 @@ public interface SafeBrowsingApiHandler { ...@@ -46,21 +46,6 @@ public interface SafeBrowsingApiHandler {
*/ */
public boolean init(Observer result); public boolean init(Observer result);
/**
* Verifies that SafeBrowsingApiHandler can operate and initializes if feasible.
* Should be called on the same sequence as |startUriLookup|.
*
* @param observer The object on which to call the callback functions when URL checking
* is complete.
* @param enableLocalBlacklists specifies if the feature to use local blacklists is enabled.
*
* @return whether Safe Browsing is supported for this installation.
*/
// TODO(crbug.com/1054179): Remove this interface once upstream implementation is removed.
public default boolean init(Observer result, boolean enableLocalBlacklists) {
return init(result);
}
/** /**
* Returns the Safety Net ID of the device. Checks to make sure that the feature to report * Returns the Safety Net ID of the device. Checks to make sure that the feature to report
* telemetry for APK downloads is enabled. This should not be called for AW. * telemetry for APK downloads is enabled. This should not be called for AW.
......
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