Integrate credit card migration feedback files into AutofillClient

1. Modify autofill_client files and local_card_migration_manager to
wire up files in CL 1300473

2. Changed method name to ShowOfferDialog to match the
ShowFeedbackDialog.

Bug: 897998
Change-Id: I6490ab18e1e822da13d550b0fb7d30d3fcebdec6
Reviewed-on: https://chromium-review.googlesource.com/c/1307074Reviewed-by: Fabio Tirelo <ftirelo@chromium.org>
Reviewed-by: Tao Bai <michaelbai@chromium.org>
Reviewed-by: Eugene But <eugenebut@chromium.org>
Commit-Queue: Siyu An <siyua@chromium.org>
Cr-Commit-Position: refs/heads/master@{#606213}

android_webview/browser/aw_autofill_client.cc

@@ -313,6 +313,13 @@ void AwAutofillClient::ConfirmSaveCreditCardToCloud(
   NOTIMPLEMENTED();
 }
 
+void AwAutofillClient::ShowLocalCardMigrationResults(
+    const base::string16& tip_message,
+    const std::vector<autofill::MigratableCreditCard>&
+        migratable_credit_cards) {
+  NOTIMPLEMENTED();
+}
+
 void AwAutofillClient::ConfirmCreditCardFillAssist(
     const autofill::CreditCard& card,
     const base::Closure& callback) {

android_webview/browser/aw_autofill_client.h

@@ -84,6 +84,10 @@ class AwAutofillClient : public autofill::AutofillClient,
       const std::vector<autofill::MigratableCreditCard>&
           migratable_credit_cards,
       LocalCardMigrationCallback start_migrating_cards_callback) override;
+  void ShowLocalCardMigrationResults(
+      const base::string16& tip_message,
+      const std::vector<autofill::MigratableCreditCard>&
+          migratable_credit_cards) override;
   void ConfirmSaveAutofillProfile(const autofill::AutofillProfile& profile,
                                   base::OnceClosure callback) override;
   void ConfirmSaveCreditCardLocally(const autofill::CreditCard& card,

chrome/browser/ui/autofill/chrome_autofill_client.cc

@@ -261,13 +261,29 @@ void ChromeAutofillClient::ConfirmMigrateLocalCardToCloud(
   autofill::LocalCardMigrationDialogControllerImpl* controller =
       autofill::LocalCardMigrationDialogControllerImpl::FromWebContents(
           web_contents());
-  controller->ShowDialog(
+  controller->ShowOfferDialog(
       std::move(legal_message),
       CreateLocalCardMigrationDialogView(controller, web_contents()),
       migratable_credit_cards, std::move(start_migrating_cards_callback));
 #endif
 }
 
+void ChromeAutofillClient::ShowLocalCardMigrationResults(
+    const base::string16& tip_message,
+    const std::vector<MigratableCreditCard>& migratable_credit_cards) {
+#if !defined(OS_ANDROID)
+  autofill::LocalCardMigrationDialogControllerImpl::CreateForWebContents(
+      web_contents());
+  autofill::LocalCardMigrationDialogControllerImpl* controller =
+      autofill::LocalCardMigrationDialogControllerImpl::FromWebContents(
+          web_contents());
+  controller->ShowFeedbackDialog(
+      tip_message,
+      CreateLocalCardMigrationDialogView(controller, web_contents()),
+      migratable_credit_cards);
+#endif
+}
+
 void ChromeAutofillClient::ConfirmSaveAutofillProfile(
     const AutofillProfile& profile,
     base::OnceClosure callback) {

chrome/browser/ui/autofill/chrome_autofill_client.h

@@ -72,6 +72,9 @@ class ChromeAutofillClient
       std::unique_ptr<base::DictionaryValue> legal_message,
       const std::vector<MigratableCreditCard>& migratable_credit_cards,
       LocalCardMigrationCallback start_migrating_cards_callback) override;
+  void ShowLocalCardMigrationResults(const base::string16& tip_message,
+                                     const std::vector<MigratableCreditCard>&
+                                         migratable_credit_cards) override;
   void ConfirmSaveAutofillProfile(const AutofillProfile& profile,
                                   base::OnceClosure callback) override;
   void ConfirmSaveCreditCardLocally(const CreditCard& card,

chrome/browser/ui/autofill/local_card_migration_dialog_controller_impl.cc

@@ -41,7 +41,7 @@ LocalCardMigrationDialogControllerImpl::
     local_card_migration_dialog_->CloseDialog();
 }
 
-void LocalCardMigrationDialogControllerImpl::ShowDialog(
+void LocalCardMigrationDialogControllerImpl::ShowOfferDialog(
     std::unique_ptr<base::DictionaryValue> legal_message,
     LocalCardMigrationDialog* local_card_migration_dialog,
     const std::vector<MigratableCreditCard>& migratable_credit_cards,

chrome/browser/ui/autofill/local_card_migration_dialog_controller_impl.h

@@ -27,7 +27,7 @@ class LocalCardMigrationDialogControllerImpl
  public:
   ~LocalCardMigrationDialogControllerImpl() override;
 
-  void ShowDialog(
+  void ShowOfferDialog(
       std::unique_ptr<base::DictionaryValue> legal_message,
       LocalCardMigrationDialog* local_card_migration_dialog,
       const std::vector<MigratableCreditCard>& migratable_credit_cards,

components/autofill/core/browser/autofill_client.h

@@ -179,6 +179,12 @@ class AutofillClient : public RiskDataLoader {
       const std::vector<MigratableCreditCard>& migratable_credit_cards,
       LocalCardMigrationCallback start_migrating_cards_callback) = 0;
 
+  // Will show a dialog with all the selected card migration results. Invoked
+  // when the migration process is finished.
+  virtual void ShowLocalCardMigrationResults(
+      const base::string16& tip_message,
+      const std::vector<MigratableCreditCard>& migratable_credit_cards) = 0;
+
   // Runs |callback| if the |profile| should be imported as personal data.
   virtual void ConfirmSaveAutofillProfile(const AutofillProfile& profile,
                                           base::OnceClosure callback) = 0;

components/autofill/core/browser/local_card_migration_manager.cc

@@ -10,6 +10,7 @@
 #include <vector>
 
 #include "base/bind.h"
+#include "base/strings/utf_string_conversions.h"
 #include "components/autofill/core/browser/autofill_client.h"
 #include "components/autofill/core/browser/autofill_experiments.h"
 #include "components/autofill/core/browser/autofill_metrics.h"
@@ -216,7 +217,11 @@ void LocalCardMigrationManager::OnDidMigrateLocalCards(
   // Remove cards that were successfully migrated from local storage.
   personal_data_manager_->DeleteLocalCreditCards(migrated_cards);
 
-  // TODO(crbug.com/852904): Trigger the show result window.
+  if (base::FeatureList::IsEnabled(
+          features::kAutofillLocalCardMigrationShowFeedback)) {
+    client_->ShowLocalCardMigrationResults(base::UTF8ToUTF16(display_text),
+                                           migratable_credit_cards_);
+  }
 }
 
 void LocalCardMigrationManager::OnDidGetMigrationRiskData(

components/autofill/core/browser/test_autofill_client.cc

@@ -102,6 +102,10 @@ void TestAutofillClient::ConfirmMigrateLocalCardToCloud(
   std::move(start_migrating_cards_callback).Run(migration_card_selection_);
 }
 
+void TestAutofillClient::ShowLocalCardMigrationResults(
+    const base::string16& tip_message,
+    const std::vector<MigratableCreditCard>& migratable_credit_cards) {}
+
 void TestAutofillClient::ConfirmSaveAutofillProfile(
     const AutofillProfile& profile,
     base::OnceClosure callback) {

components/autofill/core/browser/test_autofill_client.h

@@ -54,6 +54,9 @@ class TestAutofillClient : public AutofillClient {
       std::unique_ptr<base::DictionaryValue> legal_message,
       const std::vector<MigratableCreditCard>& migratable_credit_cards,
       LocalCardMigrationCallback start_migrating_cards_callback) override;
+  void ShowLocalCardMigrationResults(const base::string16& tip_message,
+                                     const std::vector<MigratableCreditCard>&
+                                         migratable_credit_cards) override;
   void ConfirmSaveAutofillProfile(const AutofillProfile& profile,
                                   base::OnceClosure callback) override;
   void ConfirmSaveCreditCardLocally(const CreditCard& card,

ios/chrome/browser/ui/autofill/chrome_autofill_client_ios.h

@@ -66,6 +66,9 @@ class ChromeAutofillClientIOS : public AutofillClient {
       std::unique_ptr<base::DictionaryValue> legal_message,
       const std::vector<MigratableCreditCard>& migratable_credit_cards,
       LocalCardMigrationCallback start_migrating_cards_callback) override;
+  void ShowLocalCardMigrationResults(const base::string16& tip_message,
+                                     const std::vector<MigratableCreditCard>&
+                                         migratable_credit_cards) override;
   void ConfirmSaveAutofillProfile(const AutofillProfile& profile,
                                   base::OnceClosure callback) override;
   void ConfirmSaveCreditCardLocally(const CreditCard& card,

ios/chrome/browser/ui/autofill/chrome_autofill_client_ios.mm

@@ -216,6 +216,12 @@ void ChromeAutofillClientIOS::ConfirmMigrateLocalCardToCloud(
   NOTIMPLEMENTED();
 }
 
+void ChromeAutofillClientIOS::ShowLocalCardMigrationResults(
+    const base::string16& tip_message,
+    const std::vector<MigratableCreditCard>& migratable_credit_cards) {
+  NOTIMPLEMENTED();
+}
+
 void ChromeAutofillClientIOS::ConfirmSaveCreditCardToCloud(
     const CreditCard& card,
     std::unique_ptr<base::DictionaryValue> legal_message,

ios/web_view/internal/autofill/web_view_autofill_client_ios.h

@@ -60,6 +60,9 @@ class WebViewAutofillClientIOS : public AutofillClient {
       std::unique_ptr<base::DictionaryValue> legal_message,
       const std::vector<MigratableCreditCard>& migratable_credit_cards,
       LocalCardMigrationCallback start_migrating_cards_callback) override;
+  void ShowLocalCardMigrationResults(const base::string16& tip_message,
+                                     const std::vector<MigratableCreditCard>&
+                                         migratable_credit_cards) override;
   void ConfirmSaveAutofillProfile(const AutofillProfile& profile,
                                   base::OnceClosure callback) override;
   void ConfirmSaveCreditCardLocally(const CreditCard& card,

ios/web_view/internal/autofill/web_view_autofill_client_ios.mm

@@ -137,6 +137,12 @@ void WebViewAutofillClientIOS::ConfirmMigrateLocalCardToCloud(
   NOTIMPLEMENTED();
 }
 
+void WebViewAutofillClientIOS::ShowLocalCardMigrationResults(
+    const base::string16& tip_message,
+    const std::vector<MigratableCreditCard>& migratable_credit_cards) {
+  NOTIMPLEMENTED();
+}
+
 void WebViewAutofillClientIOS::ConfirmSaveAutofillProfile(
     const AutofillProfile& profile,
     base::OnceClosure callback) {