Reviewed by Beth Dakin.
authorsullivan <sullivan@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Mon, 1 Aug 2005 16:12:03 +0000 (16:12 +0000)
committersullivan <sullivan@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Mon, 1 Aug 2005 16:12:03 +0000 (16:12 +0000)
        Test cases added: none, this only affects Safari forms autofill

        Moved the recently-added isTextField method out of the public DOMExtensions.h
        and into the private DOMPrivate.h for now, and renamed it with a leading
        underscore. We want to make some sort of public API that covers this, but not
        necessarily this exact method, so we'll leave it out of the public API until we've
        thought about this more.

        * kwq/DOMExtensions.h:
        remove isTextField
        * kwq/DOMPrivate.h:
        add _isTextField. Also added some #imports that make this file more standalone.
        * kwq/DOMHTML.mm:
        (-[DOMHTMLInputElement _isTextField]):
        moved this method and added leading underscore.

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

WebCore/ChangeLog-2005-08-23
WebCore/kwq/DOMExtensions.h
WebCore/kwq/DOMHTML.mm
WebCore/kwq/DOMPrivate.h

index 45b70b1..b3a33c8 100644 (file)
@@ -1,3 +1,23 @@
+2005-08-01  John Sullivan  <sullivan@apple.com>
+
+        Reviewed by Beth Dakin.
+
+        Test cases added: none, this only affects Safari forms autofill
+        
+        Moved the recently-added isTextField method out of the public DOMExtensions.h
+        and into the private DOMPrivate.h for now, and renamed it with a leading
+        underscore. We want to make some sort of public API that covers this, but not
+        necessarily this exact method, so we'll leave it out of the public API until we've
+        thought about this more.
+
+        * kwq/DOMExtensions.h:
+        remove isTextField
+        * kwq/DOMPrivate.h:
+        add _isTextField. Also added some #imports that make this file more standalone.
+        * kwq/DOMHTML.mm:
+        (-[DOMHTMLInputElement _isTextField]):
+        moved this method and added leading underscore.
+
 2005-08-01  Eric Seidel  <eseidel@apple.com>
 
         Reviewed by darin.
index aa8211f..ef6049f 100644 (file)
 - (void)setWidth:(long)width;
 @end
 
-@interface DOMHTMLInputElement (DOMHTMLInputElementExtensions)
-- (BOOL)isTextField;
-@end
-
 @interface DOMRGBColor (DOMRGBColorExtensions)
 - (DOMCSSPrimitiveValue *)alpha;
 @end
index 870d090..33cf728 100644 (file)
@@ -1461,40 +1461,6 @@ using DOM::NodeImpl;
 
 @end
 
-@implementation DOMHTMLInputElement (DOMHTMLInputElementExtensions)
-
-- (BOOL)isTextField
-{
-    static NSArray *textInputTypes = nil;
-#ifndef NDEBUG
-    static NSArray *nonTextInputTypes = nil;
-#endif
-    
-    NSString *type = [self type];
-    
-    // No type at all is treated as text type
-    if ([type length] == 0)
-        return YES;
-    
-    if (textInputTypes == nil)
-        textInputTypes = [[NSSet alloc] initWithObjects:@"text", @"password", @"search", nil];
-    
-    BOOL isText = [textInputTypes containsObject:[type lowercaseString]];
-    
-#ifndef NDEBUG
-    if (nonTextInputTypes == nil)
-        nonTextInputTypes = [[NSSet alloc] initWithObjects:@"isindex", @"checkbox", @"radio", @"submit", @"reset", @"file", @"hidden", @"image", @"button", @"range", nil];
-    
-    // Catch cases where a new input type has been added that's not in these lists.
-    ASSERT(isText || [nonTextInputTypes containsObject:[type lowercaseString]]);
-#endif    
-    
-    return isText;
-}
-
-@end
-
-
 @implementation DOMHTMLTextAreaElement
 
 - (HTMLTextAreaElementImpl *)_textAreaElementImpl
@@ -4024,6 +3990,37 @@ static NSView *viewForElement(DOMElement *element)
     return [(NSTextField *)viewForElement(self) stringValue];
 }
 
+- (BOOL)_isTextField
+{
+    // We could make this public API as-is, or we could change it into a method that returns whether
+    // the element is a text field or a button or ... ?
+    static NSArray *textInputTypes = nil;
+#ifndef NDEBUG
+    static NSArray *nonTextInputTypes = nil;
+#endif
+    
+    NSString *type = [self type];
+    
+    // No type at all is treated as text type
+    if ([type length] == 0)
+        return YES;
+    
+    if (textInputTypes == nil)
+        textInputTypes = [[NSSet alloc] initWithObjects:@"text", @"password", @"search", nil];
+    
+    BOOL isText = [textInputTypes containsObject:[type lowercaseString]];
+    
+#ifndef NDEBUG
+    if (nonTextInputTypes == nil)
+        nonTextInputTypes = [[NSSet alloc] initWithObjects:@"isindex", @"checkbox", @"radio", @"submit", @"reset", @"file", @"hidden", @"image", @"button", @"range", nil];
+    
+    // Catch cases where a new input type has been added that's not in these lists.
+    ASSERT(isText || [nonTextInputTypes containsObject:[type lowercaseString]]);
+#endif    
+    
+    return isText;
+}
+
 - (void)_setDisplayedValue:(NSString *)newValue
 {
     // This method is used by autofill and needs to work even when the field is currently being edited.
index 26c1863..446cafc 100644 (file)
@@ -23,6 +23,8 @@
  * OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE. 
  */
 
+#import <WebCore/DOMCSS.h>
+#import <WebCore/DOMHTML.h>
 #import <WebCore/DOMRange.h>
 
 @interface DOMRange (WebPrivate)
@@ -56,6 +58,7 @@
 // and these will be deleted.
 @interface DOMHTMLInputElement(FormsAutoFillTransition)
 - (NSString *)_displayedValue; // the string currently displayed in the field, even when the field is being edited
+- (BOOL)_isTextField;
 - (void)_setDisplayedValue:(NSString *)newValue; // set the value displayed, even when the field is being edited
 - (NSRect)_rectOnScreen; // bounding box of the text field, in screen coordinates
 - (void)_replaceCharactersInRange:(NSRange)targetRange withString:(NSString *)replacementString selectingFromIndex:(int)index;