Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(55)

Side by Side Diff: chrome/browser/ui/autofill/chrome_autofill_client.h

Issue 796493004: Remove FrameDetached and FrameWillClose listeners from AutofillAgent. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: fix init order Created 5 years, 11 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View unified diff | Download patch
OLDNEW
1 // Copyright 2014 The Chromium Authors. All rights reserved. 1 // Copyright 2014 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_CHROME_AUTOFILL_CLIENT_H_ 5 #ifndef CHROME_BROWSER_UI_AUTOFILL_CHROME_AUTOFILL_CLIENT_H_
6 #define CHROME_BROWSER_UI_AUTOFILL_CHROME_AUTOFILL_CLIENT_H_ 6 #define CHROME_BROWSER_UI_AUTOFILL_CHROME_AUTOFILL_CLIENT_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 36 matching lines...) Expand 10 before | Expand all | Expand 10 after
47 scoped_refptr<AutofillWebDataService> GetDatabase() override; 47 scoped_refptr<AutofillWebDataService> GetDatabase() override;
48 PrefService* GetPrefs() override; 48 PrefService* GetPrefs() override;
49 void HideRequestAutocompleteDialog() override; 49 void HideRequestAutocompleteDialog() override;
50 void ShowAutofillSettings() override; 50 void ShowAutofillSettings() override;
51 void ShowUnmaskPrompt(const CreditCard& card, 51 void ShowUnmaskPrompt(const CreditCard& card,
52 base::WeakPtr<CardUnmaskDelegate> delegate) override; 52 base::WeakPtr<CardUnmaskDelegate> delegate) override;
53 void OnUnmaskVerificationResult(bool success) override; 53 void OnUnmaskVerificationResult(bool success) override;
54 void ConfirmSaveCreditCard(const base::Closure& save_card_callback) override; 54 void ConfirmSaveCreditCard(const base::Closure& save_card_callback) override;
55 bool HasCreditCardScanFeature() override; 55 bool HasCreditCardScanFeature() override;
56 void ScanCreditCard(const CreditCardScanCallback& callback) override; 56 void ScanCreditCard(const CreditCardScanCallback& callback) override;
57 void ShowRequestAutocompleteDialog(const FormData& form, 57 void ShowRequestAutocompleteDialog(
58 const GURL& source_url, 58 const FormData& form,
59 const ResultCallback& callback) override; 59 content::RenderFrameHost* render_frame_host,
60 const ResultCallback& callback) override;
60 void ShowAutofillPopup( 61 void ShowAutofillPopup(
61 const gfx::RectF& element_bounds, 62 const gfx::RectF& element_bounds,
62 base::i18n::TextDirection text_direction, 63 base::i18n::TextDirection text_direction,
63 const std::vector<autofill::Suggestion>& suggestions, 64 const std::vector<autofill::Suggestion>& suggestions,
64 base::WeakPtr<AutofillPopupDelegate> delegate) override; 65 base::WeakPtr<AutofillPopupDelegate> delegate) override;
65 void UpdateAutofillPopupDataListValues( 66 void UpdateAutofillPopupDataListValues(
66 const std::vector<base::string16>& values, 67 const std::vector<base::string16>& values,
67 const std::vector<base::string16>& labels) override; 68 const std::vector<base::string16>& labels) override;
68 void HideAutofillPopup() override; 69 void HideAutofillPopup() override;
69 bool IsAutocompleteEnabled() override; 70 bool IsAutocompleteEnabled() override;
70 void DetectAccountCreationForms( 71 void DetectAccountCreationForms(
71 content::RenderFrameHost* rfh, 72 content::RenderFrameHost* rfh,
72 const std::vector<autofill::FormStructure*>& forms) override; 73 const std::vector<autofill::FormStructure*>& forms) override;
73 void DidFillOrPreviewField(const base::string16& autofilled_value, 74 void DidFillOrPreviewField(const base::string16& autofilled_value,
74 const base::string16& profile_full_name) override; 75 const base::string16& profile_full_name) override;
75 void OnFirstUserGestureObserved() override; 76 void OnFirstUserGestureObserved() override;
76 77
77 // content::WebContentsObserver implementation. 78 // content::WebContentsObserver implementation.
79 void RenderFrameDeleted(content::RenderFrameHost* rfh) override;
80 void DidNavigateAnyFrame(
81 content::RenderFrameHost* render_frame_host,
82 const content::LoadCommittedDetails& details,
83 const content::FrameNavigateParams& params) override;
78 void WebContentsDestroyed() override; 84 void WebContentsDestroyed() override;
79 85
80 // ZoomObserver implementation. 86 // ZoomObserver implementation.
81 void OnZoomChanged( 87 void OnZoomChanged(
82 const ui_zoom::ZoomController::ZoomChangedEventData& data) override; 88 const ui_zoom::ZoomController::ZoomChangedEventData& data) override;
83 89
84 // Exposed for testing. 90 // Exposed for testing.
85 AutofillDialogController* GetDialogControllerForTesting() { 91 AutofillDialogController* GetDialogControllerForTesting() {
86 return dialog_controller_.get(); 92 return dialog_controller_.get();
87 } 93 }
(...skipping 23 matching lines...) Expand all
111 // Listens to Keystone notifications and passes relevant ones on to the 117 // Listens to Keystone notifications and passes relevant ones on to the
112 // PersonalDataManager. 118 // PersonalDataManager.
113 // 119 //
114 // The class of this member must remain a forward declaration, even in the 120 // The class of this member must remain a forward declaration, even in the
115 // .cc implementation file, since the class is defined in a Mac-only 121 // .cc implementation file, since the class is defined in a Mac-only
116 // implementation file. This means that the pointer cannot be wrapped in a 122 // implementation file. This means that the pointer cannot be wrapped in a
117 // scoped_ptr. 123 // scoped_ptr.
118 AutofillKeystoneBridgeWrapper* bridge_wrapper_; 124 AutofillKeystoneBridgeWrapper* bridge_wrapper_;
119 #endif // defined(OS_MACOSX) && !defined(OS_IOS) 125 #endif // defined(OS_MACOSX) && !defined(OS_IOS)
120 126
127 // The last render frame that called requestAutocomplete.
128 content::RenderFrameHost* last_rfh_to_rac_;
129
121 DISALLOW_COPY_AND_ASSIGN(ChromeAutofillClient); 130 DISALLOW_COPY_AND_ASSIGN(ChromeAutofillClient);
122 }; 131 };
123 132
124 } // namespace autofill 133 } // namespace autofill
125 134
126 #endif // CHROME_BROWSER_UI_AUTOFILL_CHROME_AUTOFILL_CLIENT_H_ 135 #endif // CHROME_BROWSER_UI_AUTOFILL_CHROME_AUTOFILL_CLIENT_H_
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698