Display a warning when autofill is disabled for a website.

chrome/renderer/autofill_helper.cc

 // Copyright (c) 2010 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "chrome/renderer/autofill_helper.h"
 
 #include "app/l10n_util.h"
 #include "base/utf_string_conversions.h"
 #include "chrome/common/chrome_constants.h"
 #include "chrome/renderer/form_manager.h"
@@ skipping 36 matching lines @@
 }  // namespace
 
 AutoFillHelper::AutoFillHelper(RenderView* render_view)
     : render_view_(render_view),
       autofill_query_id_(0),
       autofill_action_(AUTOFILL_NONE),
       suggestions_clear_index_(-1),
       suggestions_options_index_(-1) {
 }
 
-void AutoFillHelper::QueryAutoFillSuggestions(const WebNode& node) {
+void AutoFillHelper::QueryAutoFillSuggestions(const WebNode& node,
+                                              bool autofill_disabled) {
   static int query_counter = 0;
   autofill_query_id_ = query_counter++;
   autofill_query_node_ = node;
 
   const WebFormControlElement& element = node.toConst<WebFormControlElement>();
   webkit_glue::FormField field;
   FormManager::WebFormControlElementToFormField(
       element, FormManager::EXTRACT_VALUE, &field);
 
   // WebFormControlElementToFormField does not scrape the DOM for the field
   // label, so find the label here.
   // TODO(jhawkins): Add form and field identities so we can use the cached form
   // data in FormManager.
   field.set_label(FormManager::LabelForElement(element));
 
   bool field_autofilled = NodeIsAutoFilled(node);
   render_view_->Send(new ViewHostMsg_QueryFormFieldAutoFill(
-      render_view_->routing_id(), autofill_query_id_, field_autofilled, field));
+      render_view_->routing_id(), autofill_query_id_, field_autofilled, field,
+      autofill_disabled));
 }
 
 void AutoFillHelper::RemoveAutocompleteSuggestion(
     const WebKit::WebString& name, const WebKit::WebString& value) {
   // The index of clear & options will have shifted down.
   if (suggestions_clear_index_ != -1)
     suggestions_clear_index_--;
   if (suggestions_options_index_ != -1)
     suggestions_options_index_--;
 
@@ skipping 30 matching lines @@
     i.push_back(string16());
     ids.push_back(0);
     suggestions_clear_index_ = v.size() - 1;
     separator_index = values.size();
   }
 
   // Only include "AutoFill Options" special menu item if we have AutoFill
   // items, identified by |unique_ids| having at least one valid value.
   bool show_options = false;
   for (size_t i = 0; i < ids.size(); ++i) {
-    if (ids[i] != 0) {
+    if (ids[i] > 0) {
       show_options = true;
       break;
     }
   }
   if (show_options) {
     // Append the 'Chrome Autofill options' menu item;
     v.push_back(l10n_util::GetStringFUTF16(IDS_AUTOFILL_OPTIONS_POPUP,
         WideToUTF16(chrome::kBrowserAppName)));
     l.push_back(string16());
     i.push_back(string16());
@@ skipping 82 matching lines @@
 
 void AutoFillHelper::FrameWillClose(WebFrame* frame) {
   form_manager_.ResetFrame(frame);
 }
 
 void AutoFillHelper::FrameDetached(WebFrame* frame) {
   form_manager_.ResetFrame(frame);
 }
 
 void AutoFillHelper::TextDidChangeInTextField(const WebInputElement& element) {
-  ShowSuggestions(element, false, true);
+  ShowSuggestions(element, false, true, false);
 }
 
 bool AutoFillHelper::InputElementClicked(const WebInputElement& element,
                                          bool was_focused,
                                          bool is_focused) {
   if (was_focused)
-    ShowSuggestions(element, true, false);
+    ShowSuggestions(element, true, false, true);
   return false;
 }
 
+void AutoFillHelper::ShowSuggestions(const WebInputElement& element,
+                                     bool autofill_on_empty_values,
+                                     bool requires_caret_at_end,
+                                     bool display_warning_if_disabled) {
+  bool autofill_disabled = !element.isEnabledFormControl() ||
+      !element.isText() || element.isPasswordField() ||
+      !element.autoComplete() || element.isReadOnly();
+  if (autofill_disabled && !display_warning_if_disabled)
+    return;
 
-void AutoFillHelper::ShowSuggestions(
-    const WebInputElement& const_element,
-    bool autofill_on_empty_values,
-    bool requires_caret_at_end) {
-  // We need to call non-const methods.
-  WebInputElement element(const_element);
-  if (!element.isEnabledFormControl() ||
-      !element.isText() ||
-      element.isPasswordField() ||
-      !element.autoComplete() || element.isReadOnly()) {
-    return;
-  }
-
-  WebString name = element.nameForAutofill();
-  if (name.isEmpty())  // If the field has no name, then we won't have values.
+  // If the field has no name, then we won't have values.
+  if (element.nameForAutofill().isEmpty())
     return;
 
   // Don't attempt to autofill with values that are too large.
   WebString value = element.value();
   if (value.length() > kMaximumTextSizeForAutoFill)
     return;
 
   if (!autofill_on_empty_values && value.isEmpty())
     return;
 
   if (requires_caret_at_end &&
       (element.selectionStart() != element.selectionEnd() ||
-       element.selectionEnd() != static_cast<int>(value.length()))) {
+       element.selectionEnd() != static_cast<int>(value.length())))
     return;
-  }
 
-  QueryAutoFillSuggestions(element);
+  QueryAutoFillSuggestions(element, autofill_disabled);

[dhollowa, 2010/11/12 01:52:03]

Could we avoid the IPC call here in the case that |autofill_disabled| is true? 
It seems strange that we'd call over to the browser just to fetch the disabled
string.

[Ilya Sherman, 2010/11/12 05:15:38]

The thing is, we don't want to show a warning if there are no matching profiles
or credit cards; and only the browser knows that.  We also don't want to show a
warning if Autofill is disabled; again, only the browser knows that.

 }
 
 void AutoFillHelper::QueryAutoFillFormData(const WebNode& node,
                                            int unique_id,
                                            AutoFillAction action) {
   static int query_counter = 0;
   autofill_query_id_ = query_counter++;
 
   webkit_glue::FormData form;
   const WebInputElement element = node.toConst<WebInputElement>();
@@ skipping 22 matching lines @@
             FormManager::EXTRACT_NONE, &form)) {
       forms.push_back(form);
     }
   }
 
   if (!forms.empty()) {
     render_view_->Send(new ViewHostMsg_FormsSeen(render_view_->routing_id(),
                                                  forms));
   }
 }