Moved WebDataService to ProfileKeyedService

chrome/browser/autofill/autofill_metrics_unittest.cc

Index: chrome/browser/autofill/autofill_metrics_unittest.cc
diff --git a/chrome/browser/autofill/autofill_metrics_unittest.cc b/chrome/browser/autofill/autofill_metrics_unittest.cc
index df71b842ad52b45e9fd923e567aeece1d892c7b3..d6abecc49ad640c2474f67e2de0a34d81486fd75 100644
--- a/chrome/browser/autofill/autofill_metrics_unittest.cc
+++ b/chrome/browser/autofill/autofill_metrics_unittest.cc
@@ -930,6 +930,7 @@ TEST_F(AutofillMetricsTest, AutofillIsEnabledAtStartup) {
   EXPECT_CALL(*personal_data_.metric_logger(),
               LogIsAutofillEnabledAtStartup(true)).Times(1);
   personal_data_.Init(profile());
+  personal_data_.Shutdown();

[rpetterson, 2012/03/27 01:15:06]

Why is this needed now (as opposed to before)?

[GeorgeY, 2012/03/30 19:20:12]

It is either this or make an empty service like for some other tests. Init() add
notification if service is not NULL, Shutdown() removes it. If you call Init()
twice in a row with non-null data service you get an assert for adding the same
notification twice. I thought this is a better way for fixing this problem.

 
   personal_data_.set_autofill_enabled(false);
   EXPECT_CALL(*personal_data_.metric_logger(),