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

Side by Side Diff: components/autofill/core/browser/autofill_manager.cc

Issue 707173004: Refactor Autofill for out of process iframes (OOPIF). (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: update tests, respond to review comments, cleanup Created 6 years, 1 month 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 2013 The Chromium Authors. All rights reserved. 1 // Copyright 2013 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 "components/autofill/core/browser/autofill_manager.h" 5 #include "components/autofill/core/browser/autofill_manager.h"
6 6
7 #include <stddef.h> 7 #include <stddef.h>
8 8
9 #include <limits> 9 #include <limits>
10 #include <map> 10 #include <map>
(...skipping 757 matching lines...) Expand 10 before | Expand all | Expand 10 after
768 768
769 void AutofillManager::OnLoadedServerPredictions( 769 void AutofillManager::OnLoadedServerPredictions(
770 const std::string& response_xml) { 770 const std::string& response_xml) {
771 // Parse and store the server predictions. 771 // Parse and store the server predictions.
772 FormStructure::ParseQueryResponse(response_xml, 772 FormStructure::ParseQueryResponse(response_xml,
773 form_structures_.get(), 773 form_structures_.get(),
774 *metric_logger_); 774 *metric_logger_);
775 775
776 // Forward form structures to the password generation manager to detect 776 // Forward form structures to the password generation manager to detect
777 // account creation forms. 777 // account creation forms.
778 client_->DetectAccountCreationForms(form_structures_.get()); 778 driver_->DetectAccountCreationForms(form_structures_.get());
779 779
780 // If the corresponding flag is set, annotate forms with the predicted types. 780 // If the corresponding flag is set, annotate forms with the predicted types.
781 driver_->SendAutofillTypePredictionsToRenderer(form_structures_.get()); 781 driver_->SendAutofillTypePredictionsToRenderer(form_structures_.get());
782 } 782 }
783 783
784 void AutofillManager::OnDidEndTextFieldEditing() { 784 void AutofillManager::OnDidEndTextFieldEditing() {
785 external_delegate_->DidEndTextFieldEditing(); 785 external_delegate_->DidEndTextFieldEditing();
786 } 786 }
787 787
788 bool AutofillManager::IsAutofillEnabled() const { 788 bool AutofillManager::IsAutofillEnabled() const {
(...skipping 462 matching lines...) Expand 10 before | Expand all | Expand 10 after
1251 return false; 1251 return false;
1252 1252
1253 // Disregard forms that we wouldn't ever autofill in the first place. 1253 // Disregard forms that we wouldn't ever autofill in the first place.
1254 if (!form.ShouldBeParsed()) 1254 if (!form.ShouldBeParsed())
1255 return false; 1255 return false;
1256 1256
1257 return true; 1257 return true;
1258 } 1258 }
1259 1259
1260 } // namespace autofill 1260 } // namespace autofill
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698