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

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/master again, resolved conflicts 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 149 matching lines...) Expand 10 before | Expand all | Expand 10 after
160 bool TabAutofillManagerDelegate::IsAutocompleteEnabled() { 160 bool TabAutofillManagerDelegate::IsAutocompleteEnabled() {
161 // For browser, Autocomplete is always enabled as part of Autofill. 161 // For browser, Autocomplete is always enabled as part of Autofill.
162 return GetPrefs()->GetBoolean(prefs::kAutofillEnabled); 162 return GetPrefs()->GetBoolean(prefs::kAutofillEnabled);
163 } 163 }
164 164
165 void TabAutofillManagerDelegate::HideRequestAutocompleteDialog() { 165 void TabAutofillManagerDelegate::HideRequestAutocompleteDialog() {
166 if (dialog_controller_.get()) 166 if (dialog_controller_.get())
167 dialog_controller_->Hide(); 167 dialog_controller_->Hide();
168 } 168 }
169 169
170 void TabAutofillManagerDelegate::WebContentsDestroyed( 170 void TabAutofillManagerDelegate::WebContentsDestroyed() {
171 content::WebContents* web_contents) {
172 HideAutofillPopup(); 171 HideAutofillPopup();
173 } 172 }
174 173
175 void TabAutofillManagerDelegate::DetectAccountCreationForms( 174 void TabAutofillManagerDelegate::DetectAccountCreationForms(
176 const std::vector<autofill::FormStructure*>& forms) { 175 const std::vector<autofill::FormStructure*>& forms) {
177 password_manager::PasswordGenerationManager* manager = 176 password_manager::PasswordGenerationManager* manager =
178 ChromePasswordManagerClient::GetGenerationManagerFromWebContents( 177 ChromePasswordManagerClient::GetGenerationManagerFromWebContents(
179 web_contents_); 178 web_contents_);
180 if (manager) 179 if (manager)
181 manager->DetectAccountCreationForms(forms); 180 manager->DetectAccountCreationForms(forms);
182 } 181 }
183 182
184 void TabAutofillManagerDelegate::DidFillOrPreviewField( 183 void TabAutofillManagerDelegate::DidFillOrPreviewField(
185 const base::string16& autofilled_value, 184 const base::string16& autofilled_value,
186 const base::string16& profile_full_name) { 185 const base::string16& profile_full_name) {
187 #if defined(OS_ANDROID) 186 #if defined(OS_ANDROID)
188 AutofillLoggerAndroid::DidFillOrPreviewField( 187 AutofillLoggerAndroid::DidFillOrPreviewField(
189 autofilled_value, profile_full_name); 188 autofilled_value, profile_full_name);
190 #endif // defined(OS_ANDROID) 189 #endif // defined(OS_ANDROID)
191 } 190 }
192 191
193 } // namespace autofill 192 } // namespace autofill
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698