[sync] Check against nullptr web_data_backend->GetDatabase()

components/autofill/core/browser/webdata/autocomplete_sync_bridge_unittest.cc

Index: components/autofill/core/browser/webdata/autocomplete_sync_bridge_unittest.cc
diff --git a/components/autofill/core/browser/webdata/autocomplete_sync_bridge_unittest.cc b/components/autofill/core/browser/webdata/autocomplete_sync_bridge_unittest.cc
index 668ff4165a654fab97ae92d464a1d407b3fc05b7..322f48a8007c5afdad68b33a4232ef34e3a9ff6f 100644
--- a/components/autofill/core/browser/webdata/autocomplete_sync_bridge_unittest.cc
+++ b/components/autofill/core/browser/webdata/autocomplete_sync_bridge_unittest.cc
@@ -141,10 +141,10 @@ class AutocompleteSyncBridgeTest : public testing::Test {
   }
   ~AutocompleteSyncBridgeTest() override {}
 
-  void ResetBridge() {
+  void ResetBridge(bool expect_error = false) {
     bridge_.reset(new AutocompleteSyncBridge(
-        &backend_,
-        RecordingModelTypeChangeProcessor::FactoryForBridgeTest(&processor_)));
+        &backend_, RecordingModelTypeChangeProcessor::FactoryForBridgeTest(
+                       &processor_, expect_error)));
   }
 
   void SaveSpecificsToTable(
@@ -266,6 +266,8 @@ class AutocompleteSyncBridgeTest : public testing::Test {
 
   AutofillTable* table() { return &table_; }
 
+  FakeAutofillBackend* backend() { return &backend_; }
+
  private:
   ScopedTempDir temp_dir_;
   base::test::ScopedTaskEnvironment scoped_task_environment_;
@@ -592,6 +594,11 @@ TEST_F(AutocompleteSyncBridgeTest, LoadMetadataCalled) {
   EXPECT_EQ(1u, processor().metadata()->TakeAllMetadata().size());
 }
 
+TEST_F(AutocompleteSyncBridgeTest, LoadMetadataReportsErrorForMissingDB) {
+  backend()->SetWebDatabase(nullptr);
+  ResetBridge(true);

[skym, 2017/05/08 22:22:13]

It's fairly confusing who/what checked the expectation here. Can you add a
comment?

// Processor's destructor will verify an error was reported.

[Patrick Noland, 2017/05/08 22:36:33]

Done.

+}
+
 TEST_F(AutocompleteSyncBridgeTest, MergeSyncDataEmpty) {
   VerifyMerge(std::vector<AutofillSpecifics>());