Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(206)

Unified Diff: components/safe_browsing_db/android/java/src/org/chromium/components/safe_browsing/SafeBrowsingApiBridge.java

Issue 2810863003: Android: Remove GetApplicationContext: components/ (Closed)
Patch Set: Fix tests Created 3 years, 8 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: components/safe_browsing_db/android/java/src/org/chromium/components/safe_browsing/SafeBrowsingApiBridge.java
diff --git a/components/safe_browsing_db/android/java/src/org/chromium/components/safe_browsing/SafeBrowsingApiBridge.java b/components/safe_browsing_db/android/java/src/org/chromium/components/safe_browsing/SafeBrowsingApiBridge.java
index 17858996487014d35ca37df4da58e5dd1b1a21f5..de39ea272494585d285a07c1c6c5301ae9de0e6b 100644
--- a/components/safe_browsing_db/android/java/src/org/chromium/components/safe_browsing/SafeBrowsingApiBridge.java
+++ b/components/safe_browsing_db/android/java/src/org/chromium/components/safe_browsing/SafeBrowsingApiBridge.java
@@ -4,8 +4,7 @@
package org.chromium.components.safe_browsing;
-import android.content.Context;
-
+import org.chromium.base.ContextUtils;
import org.chromium.base.Log;
import org.chromium.base.annotations.CalledByNative;
import org.chromium.base.annotations.JNINamespace;
@@ -38,7 +37,7 @@ public final class SafeBrowsingApiBridge {
* @return the handler if it's usable, or null if the API is not supported.
*/
@CalledByNative
- private static SafeBrowsingApiHandler create(Context context) {
+ private static SafeBrowsingApiHandler create() {
SafeBrowsingApiHandler handler;
try {
handler = sHandler.newInstance();
@@ -46,12 +45,13 @@ public final class SafeBrowsingApiBridge {
Log.e(TAG, "Failed to init handler: " + e.getMessage());
return null;
}
- boolean initSuccesssful = handler.init(context, new SafeBrowsingApiHandler.Observer() {
- @Override
- public void onUrlCheckDone(long callbackId, int resultStatus, String metadata) {
- nativeOnUrlCheckDone(callbackId, resultStatus, metadata);
- }
- });
+ boolean initSuccesssful = handler.init(
+ ContextUtils.getApplicationContext(), new SafeBrowsingApiHandler.Observer() {
+ @Override
+ public void onUrlCheckDone(long callbackId, int resultStatus, String metadata) {
+ nativeOnUrlCheckDone(callbackId, resultStatus, metadata);
+ }
+ });
return initSuccesssful ? handler : null;
}

Powered by Google App Engine
This is Rietveld 408576698