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

Unified Diff: ios/web/web_state/web_state_impl.mm

Issue 2466143002: iOS: Mark HTTP pages with password fields with an omnibox icon. (Closed)
Patch Set: IsOriginSecure() Created 4 years 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: ios/web/web_state/web_state_impl.mm
diff --git a/ios/web/web_state/web_state_impl.mm b/ios/web/web_state/web_state_impl.mm
index 2be723464c9b10db360e5099e61e6be65c25390e..399760f0c77d34d8ef45fe09b96a12f7615ec7bb 100644
--- a/ios/web/web_state/web_state_impl.mm
+++ b/ios/web/web_state/web_state_impl.mm
@@ -288,6 +288,10 @@ void WebStateImpl::OnSignInFailed(int request_id, const GURL& source_url) {
observer.SignInFailed(request_id, source_url);
}
+void WebStateImpl::OnPasswordInputShownOnHttp() {
Eugene But (OOO till 7-30) 2016/12/02 01:53:12 nit: Could you please try keeping the same order f
lgarron 2016/12/06 01:38:36 Whoops, fixed.
+ [web_controller_ didShowPasswordInputOnHTTP];
+}
+
void WebStateImpl::OnDocumentSubmitted(const std::string& form_name,
bool user_initiated) {
for (auto& observer : observers_)
« ios/web/web_state/ui/crw_web_controller.mm ('K') | « ios/web/web_state/web_state_impl.h ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698