LayoutTests:

Adele Peterson · Adele Peterson · commit 450652c736be · 2006-08-26T00:33:22.000Z
Updated results for addition of -webkit-text-security. * fast/css/computed-style-expected.txt: WebCore: Patch by Francisco, Reviewed by me. Preparation for switch to new text field implementation of password field. Added -webkit-text-security property. Tests: updated fast/css/computed-style-expected.txt * css/CSSPropertyNames.in: Added -webkit-text-security. * css/CSSComputedStyleDeclaration.cpp: (WebCore::): (WebCore::CSSComputedStyleDeclaration::getPropertyCSSValue): Added cases for textSecurity. * css/cssparser.cpp: (WebCore::CSSParser::parseValue): ditto. * css/cssstyleselector.cpp: (WebCore::CSSStyleSelector::applyProperty): ditto. * css/html4.css: Added style for password field to use -webkit-text-security. * html/HTMLInputElement.cpp: Check appearance property to decide which renderer to use. (WebCore::HTMLInputElement::selectionStart): (WebCore::HTMLInputElement::selectionEnd): (WebCore::HTMLInputElement::setSelectionStart): (WebCore::HTMLInputElement::setSelectionEnd): (WebCore::HTMLInputElement::select): (WebCore::HTMLInputElement::setSelectionRange): (WebCore::HTMLInputElement::createRenderer): * html/HTMLInputElement.h: (WebCore::HTMLInputElement::isNonWidgetTextField): Added check for password. * platform/StringImpl.cpp: (WebCore::StringImpl::secure): Added. Converts a string to replace characters with one character, like a bullet. * platform/StringImpl.h: * rendering/RenderStyle.cpp: (WebCore::StyleCSS3InheritedData::StyleCSS3InheritedData): Initialize textSecurity. (WebCore::RenderStyle::diff): Added case for textSecurity. * rendering/RenderStyle.h: (WebCore::): (WebCore::RenderStyle::textSecurity): Added. (WebCore::RenderStyle::setTextSecurity): Added. (WebCore::RenderStyle::initialTextSecurity): Added. * rendering/RenderText.cpp: (WebCore::RenderText::setStyle): Added case for textSecurity. (WebCore::RenderText::setText): ditto. Canonical link: https://commits.webkit.org/13441@main git-svn-id: https://svn.webkit.org/repository/webkit/trunk@16045 268f45cc-cd09-0410-ab3c-d52691b4dbfc
diff --git a/LayoutTests/ChangeLog b/LayoutTests/ChangeLog
@@ -1,3 +1,9 @@
+2006-08-25  Adele Peterson  <adele@apple.com>
+
+        Updated results for addition of -webkit-text-security.
+
+        * fast/css/computed-style-expected.txt:
+
 2006-08-25  Adele Peterson  <adele@apple.com>
 
         Reviewed by Hyatt.
diff --git a/LayoutTests/fast/css/computed-style-expected.txt b/LayoutTests/fast/css/computed-style-expected.txt
@@ -92,6 +92,7 @@ text-decoration: none;
 -webkit-text-decorations-in-effect: none;
 text-indent: 0px;
 text-shadow: none;
+-webkit-text-security: none;
 text-transform: none;
 top: auto;
 unicode-bidi: normal;
diff --git a/WebCore/ChangeLog b/WebCore/ChangeLog
@@ -1,3 +1,47 @@
+2006-08-25  Adele Peterson  <adele@apple.com>
+
+        Patch by Francisco, Reviewed by me.
+
+        Preparation for switch to new text field implementation of password field.
+
+        Added -webkit-text-security property.
+
+        Tests: updated fast/css/computed-style-expected.txt
+
+        * css/CSSPropertyNames.in: Added -webkit-text-security.
+        * css/CSSComputedStyleDeclaration.cpp:
+        (WebCore::):
+        (WebCore::CSSComputedStyleDeclaration::getPropertyCSSValue): Added cases for textSecurity.
+        * css/cssparser.cpp: (WebCore::CSSParser::parseValue): ditto.
+        * css/cssstyleselector.cpp: (WebCore::CSSStyleSelector::applyProperty): ditto.
+        * css/html4.css: Added style for password field to use -webkit-text-security.
+
+        * html/HTMLInputElement.cpp: Check appearance property to decide which renderer to use.
+        (WebCore::HTMLInputElement::selectionStart):
+        (WebCore::HTMLInputElement::selectionEnd):
+        (WebCore::HTMLInputElement::setSelectionStart):
+        (WebCore::HTMLInputElement::setSelectionEnd):
+        (WebCore::HTMLInputElement::select):
+        (WebCore::HTMLInputElement::setSelectionRange):
+        (WebCore::HTMLInputElement::createRenderer):
+        * html/HTMLInputElement.h: (WebCore::HTMLInputElement::isNonWidgetTextField): Added check for password.
+
+        * platform/StringImpl.cpp: (WebCore::StringImpl::secure): Added. Converts a string to replace 
+         characters with one character, like a bullet.
+        * platform/StringImpl.h:
+
+        * rendering/RenderStyle.cpp:
+        (WebCore::StyleCSS3InheritedData::StyleCSS3InheritedData): Initialize textSecurity.
+        (WebCore::RenderStyle::diff): Added case for textSecurity.
+        * rendering/RenderStyle.h:
+        (WebCore::):
+        (WebCore::RenderStyle::textSecurity): Added.
+        (WebCore::RenderStyle::setTextSecurity): Added.
+        (WebCore::RenderStyle::initialTextSecurity): Added.
+        * rendering/RenderText.cpp:
+        (WebCore::RenderText::setStyle): Added case for textSecurity.
+        (WebCore::RenderText::setText): ditto.
+
 2006-08-25  Adele Peterson  <adele@apple.com>
 
         Reviewed by Hyatt.
diff --git a/WebCore/css/CSSComputedStyleDeclaration.cpp b/WebCore/css/CSSComputedStyleDeclaration.cpp
@@ -136,6 +136,7 @@ static const int computedProperties[] = {
     CSS_PROP__WEBKIT_TEXT_DECORATIONS_IN_EFFECT,
     CSS_PROP_TEXT_INDENT,
     CSS_PROP_TEXT_SHADOW,
+    CSS_PROP__WEBKIT_TEXT_SECURITY,
     CSS_PROP_TEXT_TRANSFORM,
     CSS_PROP_TOP,
     CSS_PROP_UNICODE_BIDI,
@@ -1110,6 +1111,21 @@ PassRefPtr<CSSValue> CSSComputedStyleDeclaration::getPropertyCSSValue(int proper
         return valueForLength(style->textIndent());
     case CSS_PROP_TEXT_SHADOW:
         return valueForShadow(style->textShadow());
+    case CSS_PROP__WEBKIT_TEXT_SECURITY:
+    {
+        switch (style->textSecurity()) {
+            case TSNONE:
+                return new CSSPrimitiveValue(CSS_VAL_NONE);
+            case TSDISC:
+                return new CSSPrimitiveValue(CSS_VAL_DISC);
+            case TSCIRCLE:
+                return new CSSPrimitiveValue(CSS_VAL_CIRCLE);
+            case TSSQUARE:
+                return new CSSPrimitiveValue(CSS_VAL_SQUARE);
+        }
+        ASSERT_NOT_REACHED();
+        return 0;
+    }
     case CSS_PROP__WEBKIT_TEXT_SIZE_ADJUST:
         if (style->textSizeAdjust()) 
             return new CSSPrimitiveValue(CSS_VAL_AUTO);
diff --git a/WebCore/css/CSSPropertyNames.in b/WebCore/css/CSSPropertyNames.in
@@ -138,6 +138,7 @@ text-overline-color
 text-overline-mode
 text-overline-style
 text-overline-width
+-webkit-text-security
 text-shadow
 text-transform
 text-underline
diff --git a/WebCore/css/cssparser.cpp b/WebCore/css/cssparser.cpp
@@ -1175,6 +1175,12 @@ bool CSSParser::parseValue(int propId, bool important)
             valid_primitive = true;
         break;
 
+    case CSS_PROP__WEBKIT_TEXT_SECURITY:
+        // disc | circle | square | none | inherit
+        if (id == CSS_VAL_DISC || id == CSS_VAL_CIRCLE || id == CSS_VAL_SQUARE|| id == CSS_VAL_NONE)
+            valid_primitive = true;
+        break;
+
 #if __APPLE__
     case CSS_PROP__WEBKIT_DASHBOARD_REGION:                 // <dashboard-region> | <dashboard-region> 
         if (value->unit == Value::Function || id == CSS_VAL_NONE)
diff --git a/WebCore/css/cssstyleselector.cpp b/WebCore/css/cssstyleselector.cpp
@@ -4003,6 +4003,21 @@ void CSSStyleSelector::applyProperty(int id, CSSValue *value)
         fontDirty = true;
         break;
     }
+    case CSS_PROP__WEBKIT_TEXT_SECURITY: {
+        HANDLE_INHERIT_AND_INITIAL(textSecurity, TextSecurity)
+        if (!primitiveValue) break;
+        ETextSecurity textSecurity= TSNONE;
+        switch(primitiveValue->getIdent())
+        {
+            case CSS_VAL_DISC:      textSecurity= TSDISC;
+                                    break;
+            case CSS_VAL_CIRCLE:    textSecurity= TSCIRCLE;
+                                    break;
+            case CSS_VAL_SQUARE:    textSecurity= TSSQUARE;
+                                    break;
+        }
+        style->setTextSecurity(textSecurity);
+    }
 #if __APPLE__
     case CSS_PROP__WEBKIT_DASHBOARD_REGION: {
         HANDLE_INHERIT_AND_INITIAL(dashboardRegions, DashboardRegions)
diff --git a/WebCore/css/html4.css b/WebCore/css/html4.css
@@ -330,7 +330,17 @@ textarea {
     resize: auto;
 }
 
-input[type="password"], input[type="search"], input[type="hidden"], input[type="image"], input[type="file"] {
+input[type="password"] {
+    // FIXME: Remove these when switching over to the new implementation for password fields.    
+    -webkit-appearance:none;
+    background-color: initial;
+    border:initial;
+    padding:initial;
+
+    -webkit-text-security: disc;
+}
+
+input[type="search"], input[type="hidden"], input[type="image"], input[type="file"] {
     -webkit-appearance: initial;
     padding: initial;
     background-color: initial;
@@ -518,7 +528,6 @@ input[type="checkbox"]:focus,
 input[type="file"]:focus,
 input[type="hidden"]:focus,
 input[type="image"]:focus,
-input[type="password"]:focus,
 input[type="radio"]:focus,
 input[type="reset"]:focus,
 input[type="search"]:focus,
diff --git a/WebCore/html/HTMLInputElement.cpp b/WebCore/html/HTMLInputElement.cpp
@@ -474,13 +474,17 @@ int HTMLInputElement::selectionStart() const
         case RESET:
         case SUBMIT:
             break;
-        case PASSWORD:
         case SEARCH:
             return static_cast<RenderLineEdit*>(renderer())->selectionStart();
         case TEXT:
             if (document()->focusNode() != this && cachedSelStart >= 0)
                 return cachedSelStart;
             return static_cast<RenderTextControl*>(renderer())->selectionStart();
+        case PASSWORD:
+            if (renderer()->style()->appearance() == TextFieldAppearance)
+                return static_cast<RenderTextControl*>(renderer())->selectionStart();
+            else
+                return static_cast<RenderLineEdit*>(renderer())->selectionStart();
     }
     return 0;
 }
@@ -502,13 +506,17 @@ int HTMLInputElement::selectionEnd() const
         case RESET:
         case SUBMIT:
             break;
-        case PASSWORD:
         case SEARCH:
             return static_cast<RenderLineEdit*>(renderer())->selectionEnd();
         case TEXT:
             if (document()->focusNode() != this && cachedSelEnd >= 0)
                 return cachedSelEnd;
             return static_cast<RenderTextControl*>(renderer())->selectionEnd();
+        case PASSWORD:
+            if (renderer()->style()->appearance() == TextFieldAppearance)
+                return static_cast<RenderTextControl*>(renderer())->selectionEnd();
+            else
+                return static_cast<RenderLineEdit*>(renderer())->selectionEnd();
     }
     return 0;
 }
@@ -530,13 +538,18 @@ void HTMLInputElement::setSelectionStart(int start)
         case RESET:
         case SUBMIT:
             break;
-        case PASSWORD:
         case SEARCH:
             static_cast<RenderLineEdit*>(renderer())->setSelectionStart(start);
             break;
         case TEXT:
             static_cast<RenderTextControl*>(renderer())->setSelectionStart(start);
             break;
+        case PASSWORD:
+            if (renderer()->style()->appearance() == TextFieldAppearance)
+                static_cast<RenderTextControl*>(renderer())->setSelectionStart(start);
+            else
+                static_cast<RenderLineEdit*>(renderer())->setSelectionStart(start);
+            break;
     }
 }
 
@@ -557,13 +570,18 @@ void HTMLInputElement::setSelectionEnd(int end)
         case RESET:
         case SUBMIT:
             break;
-        case PASSWORD:
         case SEARCH:
             static_cast<RenderLineEdit*>(renderer())->setSelectionEnd(end);
             break;
         case TEXT:
             static_cast<RenderTextControl*>(renderer())->setSelectionEnd(end);
             break;
+        case PASSWORD:
+            if (renderer()->style()->appearance() == TextFieldAppearance)
+                static_cast<RenderTextControl*>(renderer())->setSelectionEnd(end);
+            else
+                static_cast<RenderLineEdit*>(renderer())->setSelectionEnd(end);
+            break;
     }
 }
 
@@ -586,13 +604,18 @@ void HTMLInputElement::select()
         case FILE:
             static_cast<RenderFileButton*>(renderer())->select();
             break;
-        case PASSWORD:
         case SEARCH:
             static_cast<RenderLineEdit*>(renderer())->select();
             break;
         case TEXT:
             static_cast<RenderTextControl*>(renderer())->select();
             break;
+        case PASSWORD:
+            if (renderer()->style()->appearance() == TextFieldAppearance)
+                static_cast<RenderTextControl*>(renderer())->select();
+            else
+                static_cast<RenderLineEdit*>(renderer())->select();
+            break;
     }
 }
 
@@ -613,13 +636,18 @@ void HTMLInputElement::setSelectionRange(int start, int end)
         case RESET:
         case SUBMIT:
             break;
-        case PASSWORD:
         case SEARCH:
             static_cast<RenderLineEdit*>(renderer())->setSelectionRange(start, end);
             break;
         case TEXT:
             static_cast<RenderTextControl*>(renderer())->setSelectionRange(start, end);
             break;
+        case PASSWORD:
+            if (renderer()->style()->appearance() == TextFieldAppearance)
+                static_cast<RenderTextControl*>(renderer())->setSelectionRange(start, end);
+            else
+                static_cast<RenderLineEdit*>(renderer())->setSelectionRange(start, end);
+            break;
     }
 }
 
@@ -836,13 +864,17 @@ RenderObject *HTMLInputElement::createRenderer(RenderArena *arena, RenderStyle *
         case IMAGE:
             return new (arena) RenderImage(this);
         case ISINDEX:
-        case PASSWORD:
         case SEARCH:
             return new (arena) RenderLineEdit(this);
         case RANGE:
             return new (arena) DeprecatedSlider(this);
         case TEXT:
             return new (arena) RenderTextControl(this, false);
+        case PASSWORD:
+            if (style->appearance() == TextFieldAppearance)
+                return new (arena) RenderTextControl(this, false);
+            else
+                return new (arena) RenderLineEdit(this);                
     }
     assert(false);
     return 0;
diff --git a/WebCore/html/HTMLInputElement.h b/WebCore/html/HTMLInputElement.h
@@ -28,6 +28,10 @@
 
 #include "HTMLGenericFormElement.h"
 
+// FIXME: Remove these when converting the password field
+#include "RenderStyle.h"
+#include "RenderObject.h"
+
 namespace WebCore {
 
 class HTMLImageLoader;
@@ -81,7 +85,8 @@ class HTMLInputElement : public HTMLGenericFormElement
     bool isTextField() const { return m_type == TEXT || m_type == PASSWORD || m_type == SEARCH; }
     // FIXME: When other text fields switch to the non-NSView implementation, we should add them here.
     // Once all text fields switch over, we should merge this with isTextField.
-    bool isNonWidgetTextField() const { return m_type == TEXT; }
+    bool isNonWidgetTextField() const { return m_type == TEXT || (m_type == PASSWORD /* FIXME: Remove this style check when converting password */ 
+                                        && renderer() && renderer()->style()->appearance() == TextFieldAppearance); }
 
     bool checked() const { return m_checked; }
     void setChecked(bool, bool sendChangeEvent = false);
diff --git a/WebCore/platform/StringImpl.cpp b/WebCore/platform/StringImpl.cpp
@@ -461,6 +461,16 @@ StringImpl* StringImpl::upper() const
     return c;
 }
 
+StringImpl* StringImpl::secure(UChar aChar) const
+{
+    StringImpl* c= new StringImpl;
+    c->m_data = newUCharVector(c->m_length= m_length);
+    
+    u_memset(c->m_data, aChar, m_length);
+    
+    return c;
+}
+
 StringImpl* StringImpl::foldCase() const
 {
     StringImpl* c = new StringImpl;
diff --git a/WebCore/platform/StringImpl.h b/WebCore/platform/StringImpl.h
@@ -101,6 +101,7 @@ class StringImpl : public Shared<StringImpl>, Noncopyable {
     bool isLower() const;
     StringImpl* lower() const;
     StringImpl* upper() const;
+    StringImpl* secure(UChar aChar) const;
     StringImpl* capitalize(UChar previousCharacter) const;
     StringImpl* foldCase() const;
 
diff --git a/WebCore/rendering/RenderStyle.cpp b/WebCore/rendering/RenderStyle.cpp
@@ -457,6 +457,7 @@ bool StyleCSS3NonInheritedData::operator==(const StyleCSS3NonInheritedData& o) c
 
 StyleCSS3InheritedData::StyleCSS3InheritedData()
     : textShadow(0)
+    , textSecurity(RenderStyle::initialTextSecurity())
     , userModify(READ_ONLY)
     , wordWrap(WBNORMAL)
     , nbspMode(NBNORMAL)
@@ -471,6 +472,7 @@ StyleCSS3InheritedData::StyleCSS3InheritedData(const StyleCSS3InheritedData& o)
     : Shared<StyleCSS3InheritedData>()
     , textShadow(o.textShadow ? new ShadowData(*o.textShadow) : 0)
     , highlight(o.highlight)
+    , textSecurity(o.textSecurity)
     , userModify(o.userModify)
     , wordWrap(o.wordWrap)
     , nbspMode(o.nbspMode)
@@ -835,7 +837,8 @@ RenderStyle::Diff RenderStyle::diff( const RenderStyle *other ) const
          visual->colspan != other->visual->colspan ||
          visual->counter_increment != other->visual->counter_increment ||
          visual->counter_reset != other->visual->counter_reset ||
-         css3NonInheritedData->textOverflow != other->css3NonInheritedData->textOverflow)
+         css3NonInheritedData->textOverflow != other->css3NonInheritedData->textOverflow ||
+         (css3InheritedData->textSecurity != other->css3InheritedData->textSecurity))
         return Layout;
    
     // changes causing Layout changes:
diff --git a/WebCore/rendering/RenderStyle.h b/WebCore/rendering/RenderStyle.h
diff --git a/WebCore/rendering/RenderText.cpp b/WebCore/rendering/RenderText.cpp
