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

Unified Diff: android_webview/native/aw_autofill_client.cc

Issue 2603823002: Prohibit web payments on sites with bad SSL certificates (Closed)
Patch Set: Created 3 years, 11 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 side-by-side diff with in-line comments
Download patch
Index: android_webview/native/aw_autofill_client.cc
diff --git a/android_webview/native/aw_autofill_client.cc b/android_webview/native/aw_autofill_client.cc
index 11079829ab95b4ae58ec8bcec4c02098c2e6430d..57b0f2297bdf15de1d24e0ced8c340e243e200cd 100644
--- a/android_webview/native/aw_autofill_client.cc
+++ b/android_webview/native/aw_autofill_client.cc
@@ -186,22 +186,7 @@ void AwAutofillClient::OnFirstUserGestureObserved() {
}
bool AwAutofillClient::IsContextSecure(const GURL& form_origin) {
- content::SSLStatus ssl_status;
- content::NavigationEntry* navigation_entry =
- web_contents_->GetController().GetLastCommittedEntry();
- if (!navigation_entry)
- return false;
-
- ssl_status = navigation_entry->GetSSL();
- // Note: The implementation below is a copy of the one in
- // ChromeAutofillClient::IsContextSecure, and should be kept in sync
- // until crbug.com/505388 gets implemented.
- return navigation_entry->GetURL().SchemeIsCryptographic() &&
- ssl_status.certificate &&
- (!net::IsCertStatusError(ssl_status.cert_status) ||
- net::IsCertStatusMinorError(ssl_status.cert_status)) &&
- !(ssl_status.content_status &
- content::SSLStatus::RAN_INSECURE_CONTENT);
+ return web_contents_->IsContextSecure();
}
bool AwAutofillClient::ShouldShowSigninPromo() {

Powered by Google App Engine
This is Rietveld 408576698