PasswordAutofillAgent: only check visibility if needed

components/autofill/content/renderer/password_autofill_agent.cc

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 db5d26d027eeef8203b7b5a745c162e348230f6c..0f642204976bdf0169fb66af846ba9bde4e36b19 100644
--- a/components/autofill/content/renderer/password_autofill_agent.cc
+++ b/components/autofill/content/renderer/password_autofill_agent.cc
@@ -893,16 +893,18 @@ void PasswordAutofillAgent::SendPasswordForms(bool only_visible) {
   std::vector<PasswordForm> password_forms;
   for (size_t i = 0; i < forms.size(); ++i) {
     const blink::WebFormElement& form = forms[i];
-    bool is_form_visible = IsWebNodeVisible(form);
-    if (logger) {
-      LogHTMLForm(logger.get(), Logger::STRING_FORM_FOUND_ON_PAGE, form);
-      logger->LogBoolean(Logger::STRING_FORM_IS_VISIBLE, is_form_visible);
-    }
+    if (only_visible) {
+      bool is_form_visible = IsWebNodeVisible(form);
+      if (logger) {
+        LogHTMLForm(logger.get(), Logger::STRING_FORM_FOUND_ON_PAGE, form);
+        logger->LogBoolean(Logger::STRING_FORM_IS_VISIBLE, is_form_visible);
+      }
 
-    // If requested, ignore non-rendered forms, e.g. those styled with
-    // display:none.
-    if (only_visible && !is_form_visible)
-      continue;
+      // If requested, ignore non-rendered forms, e.g., those styled with
+      // display:none.
+      if (!is_form_visible)
+        continue;
+    }
 
     scoped_ptr<PasswordForm> password_form(CreatePasswordForm(form, nullptr));
     if (password_form.get()) {