Fix newly introduced flake in ExtensionPreferenceApiTest.Standard

chrome/browser/extensions/api/autofill_private/autofill_private_event_router.cc

Index: chrome/browser/extensions/api/autofill_private/autofill_private_event_router.cc
diff --git a/chrome/browser/extensions/api/autofill_private/autofill_private_event_router.cc b/chrome/browser/extensions/api/autofill_private/autofill_private_event_router.cc
index 6f0a87f2ffd673db41e741f2e6ead3b156e8cdc8..816ed0addcc36e7a4ec0abe1786e530bd97fee58 100644
--- a/chrome/browser/extensions/api/autofill_private/autofill_private_event_router.cc
+++ b/chrome/browser/extensions/api/autofill_private/autofill_private_event_router.cc
@@ -51,7 +51,11 @@ void AutofillPrivateEventRouter::Shutdown() {
 }
 
 void AutofillPrivateEventRouter::OnPersonalDataChanged() {
-  DCHECK(personal_data_ && personal_data_->IsDataLoaded());
+  DCHECK(personal_data_);

[stevenjb, 2016/06/08 00:25:53]

Looking at personal_data_manager.cc, it is not clear to me that this is
guaranteed to be true. I think it would be better to early exit if null.

[hcarmona, 2016/06/08 14:21:06]

Done.

+
+  // Ignore any updates before data is loaded. This can happen in tests.
+  if (!personal_data_->IsDataLoaded())
+    return;
 
   autofill_util::AddressEntryList addressList =
       extensions::autofill_util::GenerateAddressList(*personal_data_);