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

Unified Diff: components/autofill/core/browser/autofill_external_delegate.h

Issue 2478043002: HTTP Bad: Add warning message to autofill dropdown for http sites (Closed)
Patch Set: nit change Created 4 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 side-by-side diff with in-line comments
Download patch
Index: components/autofill/core/browser/autofill_external_delegate.h
diff --git a/components/autofill/core/browser/autofill_external_delegate.h b/components/autofill/core/browser/autofill_external_delegate.h
index 22a002f7e0d40c1441aa7b3c78d16868c2c78d04..5431b31fe6c664b874dd3cfe27922bb479164275 100644
--- a/components/autofill/core/browser/autofill_external_delegate.h
+++ b/components/autofill/core/browser/autofill_external_delegate.h
@@ -100,8 +100,12 @@ class AutofillExternalDelegate : public AutofillPopupDelegate {
// this data.
void FillAutofillFormData(int unique_id, bool is_preview);
- // Handle applying any Autofill warnings to the Autofill popup.
- void ApplyAutofillWarnings(std::vector<Suggestion>* suggestions);
+ // Will remove Autofill warnings from |suggestions| if there are also
+ // autocomplete entries in the vector. Note: at this point, it is assumed that
+ // if there are Autofill warnings, they will be at the head of the vector and
+ // any entry that is not an Autofill warning is considered an Autocomplete
+ // entry.
+ void PossiblyRemoveAutofillWarnings(std::vector<Suggestion>* suggestions);
// Handle applying any Autofill option listings to the Autofill popup.
// This function should only get called when there is at least one

Powered by Google App Engine
This is Rietveld 408576698