Index: components/autofill/content/renderer/password_autofill_agent.cc |
diff --git a/components/autofill/content/renderer/password_autofill_agent.cc b/components/autofill/content/renderer/password_autofill_agent.cc |
index f9a0b25a8aec430e14cd9a335db6c59a6a95f953..2bcee9225a407b909e7dd08065143e1a581ef625 100644 |
--- a/components/autofill/content/renderer/password_autofill_agent.cc |
+++ b/components/autofill/content/renderer/password_autofill_agent.cc |
@@ -467,14 +467,10 @@ void PasswordAutofillAgent::FirstUserGestureObserved() { |
void PasswordAutofillAgent::SendPasswordForms(blink::WebFrame* frame, |
bool only_visible) { |
scoped_ptr<RendererSavePasswordProgressLogger> logger; |
- // From the perspective of saving passwords, only calls with |only_visible| |
- // being true are important -- the decision whether to save the password is |
- // only made after visible forms are known, for failed login detection. Calls |
- // with |only_visible| false are important for password form autofill, which |
- // is currently not part of the logging. |
- if (only_visible && logging_state_active_) { |
+ if (logging_state_active_) { |
logger.reset(new RendererSavePasswordProgressLogger(this, routing_id())); |
logger->LogMessage(Logger::STRING_SEND_PASSWORD_FORMS_METHOD); |
+ logger->LogBoolean(Logger::STRING_ONLY_VISIBLE, only_visible); |
} |
// Make sure that this security origin is allowed to use password manager. |