Index: components/autofill/content/browser/autofill_driver_impl.h |
diff --git a/components/autofill/content/browser/autofill_driver_impl.h b/components/autofill/content/browser/autofill_driver_impl.h |
index b57729328b22dd0871937934037d9396aa5274f7..6c43f9ba62505d87cca460d9c09fd612aa5cf28d 100644 |
--- a/components/autofill/content/browser/autofill_driver_impl.h |
+++ b/components/autofill/content/browser/autofill_driver_impl.h |
@@ -54,9 +54,9 @@ class AutofillDriverImpl : public AutofillDriver, |
void SetAutofillExternalDelegate( |
scoped_ptr<AutofillExternalDelegate> delegate); |
- AutofillManager* autofill_manager() { return &autofill_manager_; } |
+ AutofillManager* autofill_manager() { return autofill_manager_.get(); } |
- private: |
+ protected: |
AutofillDriverImpl( |
content::WebContents* web_contents, |
autofill::AutofillManagerDelegate* delegate, |
@@ -71,13 +71,18 @@ class AutofillDriverImpl : public AutofillDriver, |
const content::FrameNavigateParams& params) OVERRIDE; |
virtual bool OnMessageReceived(const IPC::Message& message) OVERRIDE; |
+ // Sets the manager to |manager|. Takes ownership of |manager|. |
Ilya Sherman
2013/06/21 20:19:24
nit: Please document that the manager's external d
blundell
2013/06/22 06:29:06
Done.
|
+ void SetAutofillManager(scoped_ptr<AutofillManager> manager); |
+ |
+ private: |
+ |
Ilya Sherman
2013/06/21 20:19:24
nit: Spurious newline.
blundell
2013/06/22 06:29:06
Done.
|
// AutofillExternalDelegate instance that this object instantiates in the |
// case where the autofill native UI is enabled. |
scoped_ptr<AutofillExternalDelegate> autofill_external_delegate_; |
// AutofillManager instance via which this object drives the shared Autofill |
// code. |
- AutofillManager autofill_manager_; |
+ scoped_ptr<AutofillManager> autofill_manager_; |
}; |
} // namespace autofill |