WebCore:
authorcblu <cblu@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Fri, 19 Nov 2004 01:40:39 +0000 (01:40 +0000)
committercblu <cblu@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Fri, 19 Nov 2004 01:40:39 +0000 (01:40 +0000)
Fixed: <rdar://problem/3587481> Bug Reporter Login Page: Password AutoFill does not work reliably

        Reviewed by john.

        * kwq/KWQKHTMLPart.mm:
        (KWQKHTMLPart::currentForm): just return the current form, don't scan the entire document looking for a form if there is no current form

WebBrowser:

Fixed: <rdar://problem/3587481> Bug Reporter Login Page: Password AutoFill does not work reliably

        Reviewed by john.

        * FormCompletionController.m:
        (+[FormCompletionController firstFormInFrame:]): new, currentForm in WebCore used to do this work, but now currentForm just returns the current form
        (+[FormCompletionController currentOrFirstFrameAndForm:inWebView:]): renamed, if there is no current form, call firstFormInFrame
        (+[FormCompletionController autoFillInWebView:]): call renamed method
        (+[FormCompletionController autoFillPasswordInFrame:]): if there is no current form, find a password form from all the forms in the document rather than giving up when the first form is not a password form

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@8044 268f45cc-cd09-0410-ab3c-d52691b4dbfc

WebCore/ChangeLog-2005-08-23
WebCore/kwq/KWQKHTMLPart.mm

index 2652719..40ca217 100644 (file)
@@ -1,3 +1,12 @@
+2004-11-18  Chris Blumenberg  <cblu@apple.com>
+
+       Fixed: <rdar://problem/3587481> Bug Reporter Login Page: Password AutoFill does not work reliably
+
+        Reviewed by john.
+
+        * kwq/KWQKHTMLPart.mm:
+        (KWQKHTMLPart::currentForm): just return the current form, don't scan the entire document looking for a form if there is no current form
+
 2004-11-18  Maciej Stachowiak  <mjs@apple.com>
 
         Reviewed by Chris.
index 9868fc9..9f62ae2 100644 (file)
@@ -379,10 +379,7 @@ HTMLFormElementImpl *KWQKHTMLPart::currentForm() const
     }
 
     // try walking forward in the node tree to find a form element
-    if (!start) {
-        start = xmlDocImpl();
-    }
-    return scanForForm(start);
+    return start ? scanForForm(start) : 0;
 }
 
 // Either get cached regexp or build one that matches any of the labels.