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

Unified Diff: chrome/browser/ui/autofill/chrome_autofill_client.cc

Issue 2478043002: HTTP Bad: Add warning message to autofill dropdown for http sites (Closed)
Patch Set: add test to AutofillManagerTest 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: chrome/browser/ui/autofill/chrome_autofill_client.cc
diff --git a/chrome/browser/ui/autofill/chrome_autofill_client.cc b/chrome/browser/ui/autofill/chrome_autofill_client.cc
index 70354e9f7576431024ac355421e35e56277704e9..c9840bafc57127ab906ed3fa20c0ba0e6d672bb5 100644
--- a/chrome/browser/ui/autofill/chrome_autofill_client.cc
+++ b/chrome/browser/ui/autofill/chrome_autofill_client.cc
@@ -47,6 +47,7 @@
#include "content/public/browser/navigation_entry.h"
#include "content/public/browser/render_frame_host.h"
#include "content/public/browser/ssl_status.h"
+#include "content/public/common/origin_util.h"
#include "ui/gfx/geometry/rect.h"
#if BUILDFLAG(ANDROID_JAVA_UI)
@@ -365,6 +366,10 @@ bool ChromeAutofillClient::IsContextSecure(const GURL& form_origin) {
content::SSLStatus::RAN_INSECURE_CONTENT);
}
+bool ChromeAutofillClient::IsTopLevelURLSecure() {
+ return content::IsOriginSecure(web_contents()->GetVisibleURL());
+}
+
bool ChromeAutofillClient::ShouldShowSigninPromo() {
#if !defined(OS_ANDROID)
// Determine if we are in a valid context (on desktop platforms, we could be

Powered by Google App Engine
This is Rietveld 408576698