OLD | NEW |
---|---|
1 // Copyright (c) 2012 The Chromium Authors. All rights reserved. | 1 // Copyright (c) 2012 The Chromium Authors. All rights reserved. |
2 // Use of this source code is governed by a BSD-style license that can be | 2 // Use of this source code is governed by a BSD-style license that can be |
3 // found in the LICENSE file. | 3 // found in the LICENSE file. |
4 | 4 |
5 #ifndef CHROME_BROWSER_UI_AUTOFILL_TAB_AUTOFILL_MANAGER_DELEGATE_H_ | 5 #ifndef CHROME_BROWSER_UI_AUTOFILL_TAB_AUTOFILL_MANAGER_DELEGATE_H_ |
6 #define CHROME_BROWSER_UI_AUTOFILL_TAB_AUTOFILL_MANAGER_DELEGATE_H_ | 6 #define CHROME_BROWSER_UI_AUTOFILL_TAB_AUTOFILL_MANAGER_DELEGATE_H_ |
7 | 7 |
8 #include "base/callback.h" | 8 #include "base/callback.h" |
9 #include "base/compiler_specific.h" | 9 #include "base/compiler_specific.h" |
10 #include "base/i18n/rtl.h" | 10 #include "base/i18n/rtl.h" |
(...skipping 57 matching lines...) Expand 10 before | Expand all | Expand 10 after Loading... | |
68 const std::vector<int>& identifiers, | 68 const std::vector<int>& identifiers, |
69 base::WeakPtr<AutofillPopupDelegate> delegate) OVERRIDE; | 69 base::WeakPtr<AutofillPopupDelegate> delegate) OVERRIDE; |
70 virtual void HideAutofillPopup() OVERRIDE; | 70 virtual void HideAutofillPopup() OVERRIDE; |
71 virtual void UpdateProgressBar(double value) OVERRIDE; | 71 virtual void UpdateProgressBar(double value) OVERRIDE; |
72 | 72 |
73 // content::WebContentsObserver implementation. | 73 // content::WebContentsObserver implementation. |
74 virtual void DidNavigateMainFrame( | 74 virtual void DidNavigateMainFrame( |
75 const content::LoadCommittedDetails& details, | 75 const content::LoadCommittedDetails& details, |
76 const content::FrameNavigateParams& params) OVERRIDE; | 76 const content::FrameNavigateParams& params) OVERRIDE; |
77 | 77 |
78 // Exposed for testing. | |
79 AutofillDialogControllerImpl* dialog_controller() { | |
80 return dialog_controller_.get(); | |
81 } | |
Ilya Sherman
2013/06/19 23:56:14
nit: If this is just for testing, please make this
Dan Beam
2013/06/20 00:26:57
the test interacts with the public interface and d
Ilya Sherman
2013/06/20 01:11:27
Seems like you should be able to use an instance o
Dan Beam
2013/06/20 01:47:09
we talked about this and landed on GetDialogContro
| |
82 | |
78 private: | 83 private: |
79 explicit TabAutofillManagerDelegate(content::WebContents* web_contents); | 84 explicit TabAutofillManagerDelegate(content::WebContents* web_contents); |
80 friend class content::WebContentsUserData<TabAutofillManagerDelegate>; | 85 friend class content::WebContentsUserData<TabAutofillManagerDelegate>; |
81 | 86 |
82 content::WebContents* const web_contents_; | 87 content::WebContents* const web_contents_; |
83 base::WeakPtr<AutofillDialogControllerImpl> dialog_controller_; | 88 base::WeakPtr<AutofillDialogControllerImpl> dialog_controller_; |
84 base::WeakPtr<AutocheckoutBubble> autocheckout_bubble_; | 89 base::WeakPtr<AutocheckoutBubble> autocheckout_bubble_; |
85 base::WeakPtr<AutofillPopupControllerImpl> popup_controller_; | 90 base::WeakPtr<AutofillPopupControllerImpl> popup_controller_; |
86 | 91 |
87 DISALLOW_COPY_AND_ASSIGN(TabAutofillManagerDelegate); | 92 DISALLOW_COPY_AND_ASSIGN(TabAutofillManagerDelegate); |
88 }; | 93 }; |
89 | 94 |
90 } // namespace autofill | 95 } // namespace autofill |
91 | 96 |
92 #endif // CHROME_BROWSER_UI_AUTOFILL_TAB_AUTOFILL_MANAGER_DELEGATE_H_ | 97 #endif // CHROME_BROWSER_UI_AUTOFILL_TAB_AUTOFILL_MANAGER_DELEGATE_H_ |
OLD | NEW |