iOS: Mark HTTP pages with password fields with an omnibox icon.

ios/chrome/browser/ssl/ios_security_state_tab_helper.mm

Index: ios/chrome/browser/ssl/ios_security_state_tab_helper.mm
diff --git a/ios/chrome/browser/ssl/ios_security_state_tab_helper.mm b/ios/chrome/browser/ssl/ios_security_state_tab_helper.mm
index 0777b29078199a72aef8ae1b0c2aa48dc564ceb9..9225670592f5641be3f654c365b5d5594a25e86c 100644
--- a/ios/chrome/browser/ssl/ios_security_state_tab_helper.mm
+++ b/ios/chrome/browser/ssl/ios_security_state_tab_helper.mm
@@ -55,5 +55,9 @@ IOSSecurityStateTabHelper::GetVisibleSecurityState() const {
   state->displayed_mixed_content =
       (ssl.content_status & web::SSLStatus::DISPLAYED_INSECURE_CONTENT) ? true
                                                                         : false;
+  state->displayed_password_field_on_http =
+      (ssl.content_status & web::SSLStatus::DISPLAYED_PASSWORD_FIELD_ON_HTTP)
+          ? true

[Eugene But (OOO till 7-30), 2016/11/30 17:58:41]

nit: do you need ternary here? |ssl.content_status &
web::SSLStatus::DISPLAYED_PASSWORD_FIELD_ON_HTTP| perfectly converts to bool

[lgarron, 2016/12/02 01:21:52]

I copied state->displayed_mixed_content for consistency.
Should I be inconsistent (or convert state->displayed_mixed_content)?

[Eugene But (OOO till 7-30), 2016/12/02 01:53:11]

Sorry, did not notice the previous line :) Feel free to leave as it is or remove
ternary from both places. It's up to you.

+          : false;
   return state;
 }