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

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

Issue 2622773002: Componentize SafeBrowsingApi{Bridge,Handler} (Closed)
Patch Set: Rebase off master and fix merge conflict Created 3 years, 11 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/chrome/android/java/src/org/chromium/chrome/browser/safe_browsing/SafeBrowsingApiBridge.java b/components/safe_browsing_db/android/java/src/org/chromium/components/safe_browsing/SafeBrowsingApiBridge.java
similarity index 92%
rename from chrome/android/java/src/org/chromium/chrome/browser/safe_browsing/SafeBrowsingApiBridge.java
rename to components/safe_browsing_db/android/java/src/org/chromium/components/safe_browsing/SafeBrowsingApiBridge.java
index cd4668aef2735b4dd95d332832426bf2d20c43f1..c5bf24c906edb1a8c6dd27a2f3e33751ee9cc2b4 100644
--- a/chrome/android/java/src/org/chromium/chrome/browser/safe_browsing/SafeBrowsingApiBridge.java
+++ b/components/safe_browsing_db/android/java/src/org/chromium/components/safe_browsing/SafeBrowsingApiBridge.java
@@ -2,7 +2,7 @@
// Use of this source code is governed by a BSD-style license that can be
// found in the LICENSE file.
-package org.chromium.chrome.browser.safe_browsing;
+package org.chromium.components.safe_browsing;
import android.content.Context;
@@ -56,8 +56,8 @@ public final class SafeBrowsingApiBridge {
}
@CalledByNative
- private static void startUriLookup(SafeBrowsingApiHandler handler, long callbackId,
- String uri, int[] threatsOfInterest) {
+ private static void startUriLookup(
+ SafeBrowsingApiHandler handler, long callbackId, String uri, int[] threatsOfInterest) {
vakh (use Gerrit instead) 2017/01/11 22:31:47 Similarly: ApiHandler here
Nate Fischer 2017/01/11 23:39:17 Ack. Putting this off to do renaming in another CL
handler.startUriLookup(callbackId, uri, threatsOfInterest);
Log.d(TAG, "Done starting request");
}

Powered by Google App Engine
This is Rietveld 408576698