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

Side by Side Diff: chrome/browser/ui/autofill/tab_autofill_manager_delegate.cc

Issue 257153003: We have a problem in the process on destroying WebContentsImpl because (Closed) Base URL: https://git.chromium.org/chromium/src.git@master
Patch Set: Rebased onto origin/lkgr Created 6 years, 7 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 (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 #include "chrome/browser/ui/autofill/tab_autofill_manager_delegate.h" 5 #include "chrome/browser/ui/autofill/tab_autofill_manager_delegate.h"
6 6
7 #include "base/logging.h" 7 #include "base/logging.h"
8 #include "base/prefs/pref_service.h" 8 #include "base/prefs/pref_service.h"
9 #include "chrome/browser/autofill/autofill_cc_infobar_delegate.h" 9 #include "chrome/browser/autofill/autofill_cc_infobar_delegate.h"
10 #include "chrome/browser/autofill/personal_data_manager_factory.h" 10 #include "chrome/browser/autofill/personal_data_manager_factory.h"
(...skipping 151 matching lines...) Expand 10 before | Expand all | Expand 10 after
162 bool TabAutofillManagerDelegate::IsAutocompleteEnabled() { 162 bool TabAutofillManagerDelegate::IsAutocompleteEnabled() {
163 // For browser, Autocomplete is always enabled as part of Autofill. 163 // For browser, Autocomplete is always enabled as part of Autofill.
164 return GetPrefs()->GetBoolean(prefs::kAutofillEnabled); 164 return GetPrefs()->GetBoolean(prefs::kAutofillEnabled);
165 } 165 }
166 166
167 void TabAutofillManagerDelegate::HideRequestAutocompleteDialog() { 167 void TabAutofillManagerDelegate::HideRequestAutocompleteDialog() {
168 if (dialog_controller_.get()) 168 if (dialog_controller_.get())
169 dialog_controller_->Hide(); 169 dialog_controller_->Hide();
170 } 170 }
171 171
172 void TabAutofillManagerDelegate::WebContentsDestroyed( 172 void TabAutofillManagerDelegate::WebContentsDestroyed() {
173 content::WebContents* web_contents) {
174 HideAutofillPopup(); 173 HideAutofillPopup();
175 } 174 }
176 175
177 void TabAutofillManagerDelegate::DetectAccountCreationForms( 176 void TabAutofillManagerDelegate::DetectAccountCreationForms(
178 const std::vector<autofill::FormStructure*>& forms) { 177 const std::vector<autofill::FormStructure*>& forms) {
179 password_manager::PasswordGenerationManager* manager = 178 password_manager::PasswordGenerationManager* manager =
180 ChromePasswordManagerClient::GetGenerationManagerFromWebContents( 179 ChromePasswordManagerClient::GetGenerationManagerFromWebContents(
181 web_contents_); 180 web_contents_);
182 if (manager) 181 if (manager)
183 manager->DetectAccountCreationForms(forms); 182 manager->DetectAccountCreationForms(forms);
184 } 183 }
185 184
186 void TabAutofillManagerDelegate::DidFillOrPreviewField( 185 void TabAutofillManagerDelegate::DidFillOrPreviewField(
187 const base::string16& autofilled_value, 186 const base::string16& autofilled_value,
188 const base::string16& profile_full_name) { 187 const base::string16& profile_full_name) {
189 #if defined(OS_ANDROID) 188 #if defined(OS_ANDROID)
190 AutofillLoggerAndroid::DidFillOrPreviewField( 189 AutofillLoggerAndroid::DidFillOrPreviewField(
191 autofilled_value, profile_full_name); 190 autofilled_value, profile_full_name);
192 #endif // defined(OS_ANDROID) 191 #endif // defined(OS_ANDROID)
193 } 192 }
194 193
195 } // namespace autofill 194 } // namespace autofill
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698