diff --git a/components/safe_browsing/android/java/src/org/chromium/components/safe_browsing/BraveSafeBrowsingApiHandler.java b/components/safe_browsing/android/java/src/org/chromium/components/safe_browsing/BraveSafeBrowsingApiHandler.java index 7d6d0df6460c..53e6b7dc03ad 100644 --- a/components/safe_browsing/android/java/src/org/chromium/components/safe_browsing/BraveSafeBrowsingApiHandler.java +++ b/components/safe_browsing/android/java/src/org/chromium/components/safe_browsing/BraveSafeBrowsingApiHandler.java @@ -20,12 +20,18 @@ import org.chromium.base.ContextUtils; import org.chromium.base.Log; +/** + * Brave implementation of SafetyNetApiHandler for Safe Browsing + */ public class BraveSafeBrowsingApiHandler implements SafetyNetApiHandler { public static final long SAFE_BROWSING_INIT_INTERVAL_MS = 30000; private static final long DEFAULT_CHECK_DELTA = 10; private static final String SAFE_METADATA = "{}"; private static final String TAG = "BraveSafeBrowsingApiHandler"; + /** + *This is a delegate that is implemented in the object where the handler is created + */ public interface BraveSafeBrowsingApiHandlerDelegate { default void turnSafeBrowsingOff() {} default boolean isSafeBrowsingEnabled() { @@ -39,16 +45,16 @@ default boolean isSafeBrowsingEnabled() { private boolean mInitialized; private int mTriesCount; private BraveSafeBrowsingApiHandlerDelegate mBraveSafeBrowsingApiHandlerDelegate; - private static final Object lock = new Object(); - private static BraveSafeBrowsingApiHandler instance; + private static final Object sLock = new Object(); + private static BraveSafeBrowsingApiHandler sInstance; public static BraveSafeBrowsingApiHandler getInstance() { - synchronized (lock) { - if (instance == null) { - instance = new BraveSafeBrowsingApiHandler(); + synchronized (sLock) { + if (sInstance == null) { + sInstance = new BraveSafeBrowsingApiHandler(); } } - return instance; + return sInstance; } public void setDelegate(String apiKey, @@ -142,12 +148,17 @@ public void startUriLookup(final long callbackId, String uri, int[] threatsOfInt == SafetyNetStatusCodes.SAFE_BROWSING_API_NOT_INITIALIZED) { initSafeBrowsing(); startUriLookup(callbackId, uri, threatsOfInterest); + } else { + mObserver.onUrlCheckDone(callbackId, SafeBrowsingResult.TIMEOUT, "{}", + DEFAULT_CHECK_DELTA); } } else { // A different, unknown type of error occurred. if (isDebuggable()) { Log.d(TAG, "Error: " + e.getMessage()); } + mObserver.onUrlCheckDone( + callbackId, SafeBrowsingResult.TIMEOUT, "{}", DEFAULT_CHECK_DELTA); } mTriesCount = 0; });