Chromium Code Reviews| Index: components/autofill/browser/autocomplete_history_manager.h |
| diff --git a/components/autofill/browser/autocomplete_history_manager.h b/components/autofill/browser/autocomplete_history_manager.h |
| index bec877ad9670929651972503ebc425f686285b5c..d761a6641e2dc8a8bc8942326fdfefc92d083bdd 100644 |
| --- a/components/autofill/browser/autocomplete_history_manager.h |
| +++ b/components/autofill/browser/autocomplete_history_manager.h |
| @@ -8,7 +8,6 @@ |
| #include <vector> |
| #include "base/gtest_prod_util.h" |
| -#include "base/prefs/pref_member.h" |
| #include "components/autofill/browser/webdata/autofill_webdata_service.h" |
| #include "components/webdata/common/web_data_service_consumer.h" |
| @@ -21,6 +20,7 @@ namespace autofill { |
| class AutofillDriver; |
| class AutofillExternalDelegate; |
| +class AutofillManagerDelegate; |
| struct FormData; |
| // Per-tab Autocomplete history manager. Handles receiving form data |
| @@ -28,7 +28,8 @@ struct FormData; |
| // through WebDataServiceBase. |
| class AutocompleteHistoryManager : public WebDataServiceConsumer { |
| public: |
| - explicit AutocompleteHistoryManager(AutofillDriver* driver); |
| + AutocompleteHistoryManager(AutofillDriver* driver, |
| + autofill::AutofillManagerDelegate* delegate); |
|
Ilya Sherman
2013/06/19 00:38:27
nit: This declaration is already in the autofill::
sgurun-gerrit only
2013/06/19 17:56:23
Done.
|
| virtual ~AutocompleteHistoryManager(); |
| // WebDataServiceConsumer implementation. |
| @@ -46,7 +47,7 @@ class AutocompleteHistoryManager : public WebDataServiceConsumer { |
| const std::vector<base::string16>& autofill_labels, |
| const std::vector<base::string16>& autofill_icons, |
| const std::vector<int>& autofill_unique_ids); |
| - void OnFormSubmitted(const FormData& form); |
| + virtual void OnFormSubmitted(const FormData& form); |
| // Must be public for the external delegate to use. |
| void OnRemoveAutocompleteEntry(const base::string16& name, |
| @@ -73,8 +74,6 @@ class AutocompleteHistoryManager : public WebDataServiceConsumer { |
| AutofillDriver* driver_; |
| scoped_refptr<AutofillWebDataService> autofill_data_; |
| - BooleanPrefMember autofill_enabled_; |
| - |
| // When the manager makes a request from WebDataServiceBase, the database is |
| // queried on another thread, we record the query handle until we get called |
| // back. We also store the autofill results so we can send them together. |
| @@ -88,6 +87,7 @@ class AutocompleteHistoryManager : public WebDataServiceConsumer { |
| // Delegate to perform external processing (display, selection) on |
| // our behalf. Weak. |
| AutofillExternalDelegate* external_delegate_; |
| + autofill::AutofillManagerDelegate* const manager_delegate_; |
|
Ilya Sherman
2013/06/19 00:38:27
nit: Please leave a blank line above this member v
sgurun-gerrit only
2013/06/19 17:56:23
Done.
|
| // Whether IPC is sent. |
| bool send_ipc_; |