Deflake some credit card tests

Some credit card tests may start executing before the
PersonalDataManager they depend on is fully loaded. This CL introduces
waiting for that to be completed.

Bug: 1163102,1162474
Change-Id: I394b690fe0e98925b0c6f1e8deb8be9b5388944a
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2628947
Commit-Queue: Evan Stade <estade@chromium.org>
Auto-Submit: Dominic Battré <battre@chromium.org>
Reviewed-by: Evan Stade <estade@chromium.org>
Reviewed-by: Matthias Körber <koerber@google.com>
Reviewed-by: Dominic Battré <battre@chromium.org>
Cr-Commit-Position: refs/heads/master@{#844826}

chrome/browser/ui/views/autofill/payments/credit_card_access_manager_browsertest.cc

@@ -30,6 +30,10 @@ class CreditCardAccessManagerBrowserTest : public InProcessBrowserTest {
     embedded_test_server()->ServeFilesFromSourceDirectory(
         "components/test/data/autofill");
     embedded_test_server()->StartAcceptingConnections();
+
+    // Wait for Personal Data Manager to be fully loaded to prevent that
+    // spurious notifications deceive the tests.
+    WaitForPersonalDataManagerToBeLoaded(browser()->profile());
   }
 
   CreditCardAccessManager* GetCreditCardAccessManager() {

chrome/browser/ui/views/autofill/payments/local_card_migration_browsertest.cc

@@ -214,6 +214,10 @@ class LocalCardMigrationBrowserTest
     personal_data_ =
         PersonalDataManagerFactory::GetForProfile(browser()->profile());
 
+    // Wait for Personal Data Manager to be fully loaded to prevent that
+    // spurious notifications deceive the tests.
+    WaitForPersonalDataManagerToBeLoaded(browser()->profile());
+
     // Set up the fake geolocation data.
     geolocation_overrider_ =
         std::make_unique<device::ScopedGeolocationOverrider>(

chrome/browser/ui/views/autofill/payments/save_card_bubble_views_browsertest.cc

@@ -212,6 +212,10 @@ class SaveCardBubbleViewsFullFormBrowserTest
         ->GetPaymentsClient()
         ->set_url_loader_factory_for_testing(test_shared_loader_factory_);
 
+    // Wait for Personal Data Manager to be fully loaded to prevent that
+    // spurious notifications deceive the tests.
+    WaitForPersonalDataManagerToBeLoaded(browser()->profile());
+
     // Set up this class as the ObserverForTest implementation.
     credit_card_save_manager_ = ContentAutofillDriver::GetForRenderFrameHost(
                                     GetActiveWebContents()->GetMainFrame())