Move UI dependency from WebDataService to WebDataServiceFactory.

components/autofill/browser/autocomplete_history_manager_unittest.cc

Index: components/autofill/browser/autocomplete_history_manager_unittest.cc
diff --git a/components/autofill/browser/autocomplete_history_manager_unittest.cc b/components/autofill/browser/autocomplete_history_manager_unittest.cc
index 2d2bb4dbbbcdbf49125b630b21e917a711bddd3f..fe2967de8a9b0dbb9f8578b3f8456069f8151af9 100644
--- a/components/autofill/browser/autocomplete_history_manager_unittest.cc
+++ b/components/autofill/browser/autocomplete_history_manager_unittest.cc
@@ -32,7 +32,8 @@ namespace {
 
 class MockWebDataService : public WebDataService {
  public:
-  MockWebDataService() {
+  MockWebDataService()
+      : WebDataService(WebDataServiceBase::ProfileErrorCallback()) {

[Cait (Slow), 2013/03/18 16:16:37]

I wonder if it's worth having a default ctor on WDS that constructs the WDS with
a NULL callback (and  eventually a NULL WebDatabaseService), to avoid
continually having to touch these same unit_tests?

[Jói, 2013/03/18 23:21:23]

Added such a protected constructor, so mock versions of WebDataService don't
need to specify this.

     current_mock_web_data_service_ = this;
   }