Fix for autofill bug 8627

webkit/glue/editor_client_impl.cc

 // Copyright (c) 2006-2008 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.
 
 // The Mac interface forwards most of these commands to the application layer,
 // and I'm not really sure what to do about most of them.
 
 #include "config.h"
 #include "webkit/glue/editor_client_impl.h"
 
@@ skipping 10 matching lines @@
 #include "KeyboardEvent.h"
 #include "PlatformKeyboardEvent.h"
 #include "PlatformString.h"
 #include "RenderObject.h"
 
 #undef LOG
 #include "base/message_loop.h"
 #include "base/string_util.h"
 #include "third_party/WebKit/WebKit/chromium/public/WebKit.h"
 #include "webkit/glue/autofill_form.h"
+#include "webkit/glue/dom_operations.h"
 #include "webkit/glue/editor_client_impl.h"
 #include "webkit/glue/glue_util.h"
 #include "webkit/glue/webkit_glue.h"
 #include "webkit/glue/webview.h"
 #include "webkit/glue/webview_impl.h"
 
 // Arbitrary depth limit for the undo stack, to keep it from using
 // unbounded memory.  This is the maximum number of distinct undoable
 // actions -- unbroken stretches of typed characters are coalesced
 // into a single action.
@@ skipping 616 matching lines @@
   // Hide any showing popup.
   web_view_->HideAutoCompletePopup();
 }
 
 void EditorClientImpl::textDidChangeInTextField(WebCore::Element* element) {
   DCHECK(element->hasLocalName(WebCore::HTMLNames::inputTag));
   Autofill(static_cast<WebCore::HTMLInputElement*>(element), false);
 }
 
 void EditorClientImpl::ShowAutofillForNode(WebCore::Node* node) {
-  if (node->nodeType() == WebCore::Node::ELEMENT_NODE) {
-    WebCore::Element* element = static_cast<WebCore::Element*>(node);
-    if (element->hasLocalName(WebCore::HTMLNames::inputTag)) {
-      WebCore::HTMLInputElement* input_element =
-          static_cast<WebCore::HTMLInputElement*>(element);
-      Autofill(input_element, true);
-    }
-  }
+  WebCore::HTMLInputElement* input_element =
+      webkit_glue::NodeToHTMLInputElement(node);
+  if (input_element)
+    Autofill(input_element, true);
 }
 
 void EditorClientImpl::Autofill(WebCore::HTMLInputElement* input_element,
                                 bool autofill_on_empty_value) {
   // Cancel any pending DoAutofill calls.
   autofill_factory_.RevokeAll();
 
   // Let's try to trigger autofill for that field, if applicable.
   if (!input_element->isEnabled() || !input_element->isTextField() ||
       input_element->isPasswordField() || !input_element->autoComplete()) {
@@ skipping 223 matching lines @@
   }
   return L"(UNKNOWN AFFINITY)";
 }
 
 std::wstring EditorClientImpl::Describe(WebCore::CSSStyleDeclaration* style) {
   // TODO(pamg): Implement me.  It's not clear what WebKit produces for this
   // (their [style description] method), and none of the layout tests provide
   // an example.  But because none of them use it, it's not yet important.
   return std::wstring();
 }