[PwdCheckAndroid] Add delete password to check and bridge API

This CL adds the wiring of the delete call to the native side. The
actual deleting does not happen yet.

Bug: 1108358, 1092444
Change-Id: Ibe312ca2eb5218188e8eefe3a064ad005a75b0d0
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2332825Reviewed-by: Ioana Pandele <ioanap@chromium.org>
Commit-Queue: Friedrich [CET] <fhorschig@chromium.org>
Cr-Commit-Position: refs/heads/master@{#795381}

chrome/browser/password_check/android/internal/java/src/org/chromium/chrome/browser/password_check/PasswordCheckBridge.java

@@ -124,6 +124,10 @@ class PasswordCheckBridge {
                 mNativePasswordCheckBridge, credentials);
     }
 
+    void removeCredential(CompromisedCredential credential) {
+        PasswordCheckBridgeJni.get().removeCredential(mNativePasswordCheckBridge, credential);
+    }
+
     /**
      * Destroys its C++ counterpart.
      */
@@ -146,6 +150,7 @@ class PasswordCheckBridge {
         int getSavedPasswordsCount(long nativePasswordCheckBridge);
         void getCompromisedCredentials(
                 long nativePasswordCheckBridge, CompromisedCredential[] credentials);
+        void removeCredential(long nativePasswordCheckBridge, CompromisedCredential credentials);
         void destroy(long nativePasswordCheckBridge);
     }
 }

chrome/browser/password_check/android/internal/java/src/org/chromium/chrome/browser/password_check/PasswordCheckImpl.java

@@ -76,7 +76,7 @@ class PasswordCheckImpl implements PasswordCheck, PasswordCheckObserver {
 
     @Override
     public void removeCredential(CompromisedCredential credential) {
-        // TODO(crbug.com/1106726): Call native method through bridge.
+        mPasswordCheckBridge.removeCredential(credential);
     }
 
     @Override

chrome/browser/password_check/android/password_check_bridge.cc

@@ -43,6 +43,12 @@ void PasswordCheckBridge::GetCompromisedCredentials(
   // TODO(crbug.com/1102025): implement this.
 }
 
+void PasswordCheckBridge::RemoveCredential(
+    JNIEnv* env,
+    const base::android::JavaParamRef<jobject>& credential) {
+  // TODO(crbug.com/1108358): implement this.
+}
+
 void PasswordCheckBridge::Destroy(JNIEnv* env) {
   delete this;
 }

chrome/browser/password_check/android/password_check_bridge.h

@@ -35,6 +35,12 @@ class PasswordCheckBridge {
       JNIEnv* env,
       const base::android::JavaParamRef<jobjectArray>& credentials);
 
+  // Called by Java to remove a single compromised credentials from the password
+  // store.
+  void RemoveCredential(
+      JNIEnv* env,
+      const base::android::JavaParamRef<jobject>& credentials);
+
   // Called by Java when the bridge is no longer needed. Destructs itself.
   void Destroy(JNIEnv* env);