Tree was closed. Rolling out.
authorkocienda <kocienda@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Mon, 21 Feb 2005 22:02:34 +0000 (22:02 +0000)
committerkocienda <kocienda@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Mon, 21 Feb 2005 22:02:34 +0000 (22:02 +0000)
git-svn-id: https://svn.webkit.org/repository/webkit/trunk@8652 268f45cc-cd09-0410-ab3c-d52691b4dbfc

LayoutTests/editing/execCommand/modifyForeColorByCharacter-expected.txt
LayoutTests/editing/style/block-style-004-expected.txt
LayoutTests/editing/style/block-style-005-expected.txt
LayoutTests/editing/style/block-style-006-expected.txt
LayoutTests/editing/style/smoosh-styles-001-expected.txt
LayoutTests/editing/style/smoosh-styles-002-expected.txt
WebCore/ChangeLog-2005-08-23
WebCore/khtml/editing/htmlediting.cpp
WebCore/khtml/editing/htmlediting.h

index d0aa8d39e053e07a89e07a455aa24c618452d1f0..fa5c476c3fabc6125e976a571cf2425712b9fb0d 100644 (file)
@@ -6,7 +6,7 @@ layer at (0,0) size 800x600
       RenderBlock {DIV} at (0,0) size 784x356 [border: (2px solid #FF0000)]
         RenderBlock (anonymous) at (14,14) size 756x56
           RenderInline {SPAN} at (0,0) size 687x56
-            RenderInline {FONT} at (0,0) size 10x28 [color=#008000]
+            RenderInline {SPAN} at (0,0) size 10x28 [color=#008000]
               RenderText {TEXT} at (0,0) size 10x28
                 text run at (0,0) width 10: "\""
             RenderText {TEXT} at (10,0) size 687x56
@@ -39,9 +39,9 @@ layer at (0,0) size 800x600
           RenderInline {SPAN} at (0,0) size 0x0
           RenderText {TEXT} at (0,0) size 0x0
 selection is RANGE:
-start:      position 0 of child 1 {TEXT} of child 1 {FONT} of child 2 {SPAN} of root {DIV}
+start:      position 0 of child 1 {TEXT} of child 1 {SPAN} of child 2 {SPAN} of root {DIV}
 upstream:   position 0 of  of root {DIV}
-downstream: position 0 of child 1 {TEXT} of child 1 {FONT} of child 2 {SPAN} of root {DIV}
-end:        position 1 of child 1 {TEXT} of child 1 {FONT} of child 2 {SPAN} of root {DIV}
-upstream:   position 1 of child 1 {TEXT} of child 1 {FONT} of child 2 {SPAN} of root {DIV}
+downstream: position 0 of child 1 {TEXT} of child 1 {SPAN} of child 2 {SPAN} of root {DIV}
+end:        position 1 of child 1 {TEXT} of child 1 {SPAN} of child 2 {SPAN} of root {DIV}
+upstream:   position 1 of child 1 {TEXT} of child 1 {SPAN} of child 2 {SPAN} of root {DIV}
 downstream: position 0 of child 2 {TEXT} of child 2 {SPAN} of root {DIV}
index 580ca67bdf22fa0b70ea1ccc2e71cc812f1f9474..cb023f11c2784a14b796ed1f232175ba8d053965 100644 (file)
@@ -20,24 +20,20 @@ layer at (0,0) size 800x600
           text run at (14,98) width 557: "REGRESSION (Mail): Style not preserved on blank lines"
       RenderBlock {DIV} at (0,164) size 784x276
         RenderBlock {DIV} at (0,0) size 784x69
-          RenderInline {FONT} at (0,0) size 30x55
-            RenderInline {SPAN} at (0,0) size 30x69
-              RenderText {TEXT} at (0,0) size 30x69
-                text run at (0,0) width 30: "x"
+          RenderInline {SPAN} at (0,0) size 30x69
+            RenderText {TEXT} at (0,0) size 30x69
+              text run at (0,0) width 30: "x"
         RenderBlock {DIV} at (0,69) size 784x69
-          RenderInline {FONT} at (0,0) size 0x55
-            RenderInline {SPAN} at (0,0) size 0x69
-              RenderBR {BR} at (0,0) size 0x69
+          RenderInline {SPAN} at (0,0) size 0x69
+            RenderBR {BR} at (0,0) size 0x69
         RenderBlock {DIV} at (0,138) size 784x69
-          RenderInline {FONT} at (0,0) size 0x55
-            RenderInline {SPAN} at (0,0) size 0x69
-              RenderBR {BR} at (0,0) size 0x69
+          RenderInline {SPAN} at (0,0) size 0x69
+            RenderBR {BR} at (0,0) size 0x69
         RenderBlock {DIV} at (0,207) size 784x69
-          RenderInline {FONT} at (0,0) size 30x55
-            RenderInline {SPAN} at (0,0) size 30x69
-              RenderText {TEXT} at (0,0) size 30x69
-                text run at (0,0) width 30: "x"
+          RenderInline {SPAN} at (0,0) size 30x69
+            RenderText {TEXT} at (0,0) size 30x69
+              text run at (0,0) width 30: "x"
 selection is CARET:
-start:      position 0 of child 1 {BR} of child 1 {SPAN} of child 1 {FONT} of child 3 {DIV} of root {DIV}
+start:      position 0 of child 1 {BR} of child 1 {SPAN} of child 3 {DIV} of root {DIV}
 upstream:   position 0 of child 3 {DIV} of root {DIV}
-downstream: position 0 of child 1 {BR} of child 1 {SPAN} of child 1 {FONT} of child 3 {DIV} of root {DIV}
+downstream: position 0 of child 1 {BR} of child 1 {SPAN} of child 3 {DIV} of root {DIV}
index bdd33c3e2d3a9d174cadbd2f0cd1203870abad21..9ae916bf58d084d9d729114f1854d2a413b64f5e 100644 (file)
@@ -22,19 +22,16 @@ layer at (0,0) size 800x600
           text run at (14,98) width 106: "blank lines"
       RenderBlock {DIV} at (0,164) size 784x207
         RenderBlock {DIV} at (0,0) size 784x69
-          RenderInline {FONT} at (0,0) size 30x55
-            RenderInline {SPAN} at (0,0) size 30x69
-              RenderText {TEXT} at (0,0) size 30x69
-                text run at (0,0) width 30: "x"
+          RenderInline {SPAN} at (0,0) size 30x69
+            RenderText {TEXT} at (0,0) size 30x69
+              text run at (0,0) width 30: "x"
         RenderBlock {DIV} at (0,69) size 784x69
-          RenderInline {FONT} at (0,0) size 0x55
-            RenderInline {SPAN} at (0,0) size 0x69
-              RenderBR {BR} at (0,0) size 0x69
+          RenderInline {SPAN} at (0,0) size 0x69
+            RenderBR {BR} at (0,0) size 0x69
         RenderBlock {DIV} at (0,138) size 784x69
-          RenderInline {FONT} at (0,0) size 0x55
-            RenderInline {SPAN} at (0,0) size 0x69
-              RenderBR {BR} at (0,0) size 0x69
+          RenderInline {SPAN} at (0,0) size 0x69
+            RenderBR {BR} at (0,0) size 0x69
 selection is CARET:
-start:      position 0 of child 1 {BR} of child 1 {SPAN} of child 1 {FONT} of child 3 {DIV} of root {DIV}
+start:      position 0 of child 1 {BR} of child 1 {SPAN} of child 3 {DIV} of root {DIV}
 upstream:   position 0 of child 3 {DIV} of root {DIV}
-downstream: position 0 of child 1 {BR} of child 1 {SPAN} of child 1 {FONT} of child 3 {DIV} of root {DIV}
+downstream: position 0 of child 1 {BR} of child 1 {SPAN} of child 3 {DIV} of root {DIV}
index 0a899f316890f0d6def969fbbbf1bd1f6210ce58..b7113d010c411bc1ab6baf3b3c5d65a982961c56 100644 (file)
@@ -22,24 +22,20 @@ layer at (0,0) size 800x600
           text run at (14,98) width 273: "not preserved on blank lines"
       RenderBlock {DIV} at (0,164) size 784x276
         RenderBlock {DIV} at (0,0) size 784x69
-          RenderInline {FONT} at (0,0) size 30x55
-            RenderInline {SPAN} at (0,0) size 30x69
-              RenderText {TEXT} at (0,0) size 30x69
-                text run at (0,0) width 30: "x"
+          RenderInline {SPAN} at (0,0) size 30x69
+            RenderText {TEXT} at (0,0) size 30x69
+              text run at (0,0) width 30: "x"
         RenderBlock {DIV} at (0,69) size 784x69
-          RenderInline {FONT} at (0,0) size 0x55
-            RenderInline {SPAN} at (0,0) size 0x69
-              RenderBR {BR} at (0,0) size 0x69
+          RenderInline {SPAN} at (0,0) size 0x69
+            RenderBR {BR} at (0,0) size 0x69
         RenderBlock {DIV} at (0,138) size 784x69
-          RenderInline {FONT} at (0,0) size 0x55
-            RenderInline {SPAN} at (0,0) size 0x69
-              RenderBR {BR} at (0,0) size 0x69
+          RenderInline {SPAN} at (0,0) size 0x69
+            RenderBR {BR} at (0,0) size 0x69
         RenderBlock {DIV} at (0,207) size 784x69
-          RenderInline {FONT} at (0,0) size 30x55
-            RenderInline {SPAN} at (0,0) size 30x69
-              RenderText {TEXT} at (0,0) size 30x69
-                text run at (0,0) width 30: "x"
+          RenderInline {SPAN} at (0,0) size 30x69
+            RenderText {TEXT} at (0,0) size 30x69
+              text run at (0,0) width 30: "x"
 selection is CARET:
-start:      position 0 of child 1 {TEXT} of child 1 {SPAN} of child 1 {FONT} of child 4 {DIV} of root {DIV}
+start:      position 0 of child 1 {TEXT} of child 1 {SPAN} of child 4 {DIV} of root {DIV}
 upstream:   position 0 of child 4 {DIV} of root {DIV}
-downstream: position 0 of child 1 {TEXT} of child 1 {SPAN} of child 1 {FONT} of child 4 {DIV} of root {DIV}
+downstream: position 0 of child 1 {TEXT} of child 1 {SPAN} of child 4 {DIV} of root {DIV}
index be92187f644f454b37678ab5591cb8b6a7b0b870..ca773c69423211322ee47852d53f42d06eaaf7b9 100644 (file)
@@ -31,12 +31,12 @@ layer at (0,0) size 800x600
           RenderInline {SPAN} at (0,0) size 77x28 [color=#FF0000]
             RenderText {TEXT} at (2,2) size 23x28
               text run at (2,2) width 23: "ab"
-            RenderInline {FONT} at (0,0) size 34x28 [color=#000000]
+            RenderInline {SPAN} at (0,0) size 34x28 [color=#000000]
               RenderText {TEXT} at (25,2) size 34x28
                 text run at (25,2) width 34: "cde"
             RenderText {TEXT} at (59,2) size 20x28
               text run at (59,2) width 20: "fg"
 selection is CARET:
-start:      position 3 of child 1 {TEXT} of child 2 {FONT} of child 2 {SPAN} of child 1 {DIV} of root {DIV}
-upstream:   position 3 of child 1 {TEXT} of child 2 {FONT} of child 2 {SPAN} of child 1 {DIV} of root {DIV}
+start:      position 3 of child 1 {TEXT} of child 2 {SPAN} of child 2 {SPAN} of child 1 {DIV} of root {DIV}
+upstream:   position 3 of child 1 {TEXT} of child 2 {SPAN} of child 2 {SPAN} of child 1 {DIV} of root {DIV}
 downstream: position 0 of child 3 {TEXT} of child 2 {SPAN} of child 1 {DIV} of root {DIV}
index 586a995ee9f13eb4316320e7c357619db68a676a..cfb1a37cf4797abad647b0d25bde2f7156e57b65 100644 (file)
@@ -30,16 +30,15 @@ layer at (0,0) size 800x600
           RenderInline {SPAN} at (0,0) size 99x28 [color=#FF0000]
             RenderText {TEXT} at (2,8) size 29x28
               text run at (2,8) width 29: "ab"
-            RenderInline {FONT} at (0,0) size 46x37 [color=#000000]
-              RenderInline {SPAN} at (0,0) size 46x37
-                RenderInline {B} at (0,0) size 46x37
-                  RenderText {TEXT} at (31,2) size 46x37
-                    text run at (31,2) width 46: "cde"
+            RenderInline {SPAN} at (0,0) size 46x37 [color=#000000]
+              RenderInline {B} at (0,0) size 46x37
+                RenderText {TEXT} at (31,2) size 46x37
+                  text run at (31,2) width 46: "cde"
             RenderText {TEXT} at (77,8) size 24x28
               text run at (77,8) width 24: "fg"
           RenderText {TEXT} at (0,0) size 0x0
         RenderBlock {H1} at (0,62) size 784x0
 selection is CARET:
-start:      position 3 of child 1 {TEXT} of child 1 {B} of child 1 {SPAN} of child 2 {FONT} of child 2 {SPAN} of child 1 {DIV} of root {DIV}
-upstream:   position 3 of child 1 {TEXT} of child 1 {B} of child 1 {SPAN} of child 2 {FONT} of child 2 {SPAN} of child 1 {DIV} of root {DIV}
+start:      position 3 of child 1 {TEXT} of child 1 {B} of child 2 {SPAN} of child 2 {SPAN} of child 1 {DIV} of root {DIV}
+upstream:   position 3 of child 1 {TEXT} of child 1 {B} of child 2 {SPAN} of child 2 {SPAN} of child 1 {DIV} of root {DIV}
 downstream: position 0 of child 3 {TEXT} of child 2 {SPAN} of child 1 {DIV} of root {DIV}
index cb37136b8975d775ab130875605bff195f1ce056..24d32cae07a7dc9ece972a85eb03101934f403c9 100644 (file)
@@ -1,41 +1,3 @@
-2005-02-21  Ken Kocienda  <kocienda@apple.com>
-
-        Reviewed by Hyatt
-
-        Fix for this bug:
-        
-        <rdar://problem/4003463> Mail.app HTML uses inline styling markup not understood by Entourage and Eudora
-
-        * khtml/editing/htmlediting.cpp:
-        (khtml::isEmptyFontTag): Helper for removing <font> tags.
-        (khtml::StyleChange::styleModeForParseMode): Helper to map a document parse mode to a use/don't use
-        legacy-html-styles value.
-        (khtml::StyleChange::checkForLegacyHTMLStyleChange): Add support for colors, font faces, and font sizes.
-        (khtml::ApplyStyleCommand::isHTMLStyleNode):
-        (khtml::ApplyStyleCommand::removeHTMLFontStyle):
-        (khtml::ApplyStyleCommand::applyTextDecorationStyle): Now call styleModeForParseMode to determine
-        whether to use legacy html styles or not.
-        (khtml::ApplyStyleCommand::removeInlineStyle): Now properly removes <font> styles.
-        (khtml::ApplyStyleCommand::addBlockStyleIfNeeded): Ditto.
-        (khtml::ApplyStyleCommand::addInlineStyleIfNeeded): Ditto.
-        (khtml::createFontElement): Helper for applying <font> elements.
-        * khtml/editing/htmlediting.h: All the following support the new bits of data we need to store.
-        (khtml::StyleChange::applyFontColor)
-        (khtml::StyleChange::applyFontFace)
-        (khtml::StyleChange::applyFontSize)
-        (khtml::StyleChange::fontColor)
-        (khtml::StyleChange::fontFace)
-        (khtml::StyleChange::fontSize)
-
-        Test results updated now that we will write out <font> tags for quirks mode documents.        
-        
-        * layout-tests/editing/execCommand/modifyForeColorByCharacter-expected.txt
-        * layout-tests/editing/style/block-style-004-expected.txt
-        * layout-tests/editing/style/block-style-005-expected.txt
-        * layout-tests/editing/style/block-style-006-expected.txt
-        * layout-tests/editing/style/smoosh-styles-001-expected.txt
-        * layout-tests/editing/style/smoosh-styles-002-expected.txt
-
 2005-02-21  Darin Adler  <darin@apple.com>
 
         - fixed Panther deployment build
index 85d2df3d8bb7eb2850cae4857ab79fd7f8530a2b..491c7c4bfaa03cb59f7ef004862d2936a4ef7f6c 100644 (file)
@@ -48,7 +48,6 @@
 #include "khtml_part.h"
 #include "khtml_part.h"
 #include "khtmlview.h"
-#include "qcolor.h"
 #include "qptrlist.h"
 #include "render_object.h"
 #include "render_style.h"
@@ -196,16 +195,6 @@ static bool isStyleSpan(const NodeImpl *node)
     return elem->id() == ID_SPAN && elem->getAttribute(ATTR_CLASS) == styleSpanClassString();
 }
 
-static bool isEmptyFontTag(const NodeImpl *node)
-{
-    if (!node || node->id() != ID_FONT)
-        return false;
-
-    const ElementImpl *elem = static_cast<const ElementImpl *>(node);
-    NamedAttrMapImpl *map = elem->attributes(true); // true for read-only
-    return (!map || map->length() == 1) && elem->getAttribute(ATTR_CLASS) == styleSpanClassString();
-}
-
 static DOMString &blockPlaceholderClassString()
 {
     static DOMString blockPlaceholderClassString = "khtml-block-placeholder";
@@ -451,12 +440,7 @@ void StyleChange::init(CSSStyleDeclarationImpl *style, const Position &position)
     m_cssStyle = styleText.stripWhiteSpace();
 }
 
-StyleChange::ELegacyHTMLStyles StyleChange::styleModeForParseMode(bool isQuirksMode)
-{
-    return isQuirksMode ? UseLegacyHTMLStyles : DoNotUseLegacyHTMLStyles;
-}
-
-bool StyleChange::checkForLegacyHTMLStyleChange(const CSSProperty *property)
+bool StyleChange::checkForLegacyHTMLStyleChange(const DOM::CSSProperty *property)
 {
     DOMString valueText(property->value()->cssText());
     switch (property->id()) {
@@ -472,41 +456,6 @@ bool StyleChange::checkForLegacyHTMLStyleChange(const CSSProperty *property)
                 return true;
             }
             break;
-        case CSS_PROP_COLOR: {
-            QColor color(CSSParser::parseColor(valueText));
-            m_applyFontColor = color.name();
-            return true;
-        }
-        case CSS_PROP_FONT_FAMILY:
-            m_applyFontFace = valueText;
-            return true;
-        case CSS_PROP_FONT_SIZE:
-            if (property->value()->cssValueType() == CSSValue::CSS_PRIMITIVE_VALUE) {
-                CSSPrimitiveValueImpl *value = static_cast<CSSPrimitiveValueImpl *>(property->value());
-                float number = value->getFloatValue(CSSPrimitiveValue::CSS_PX);
-                if (number <= 9)
-                    m_applyFontSize = "1";
-                else if (number <= 10)
-                    m_applyFontSize = "2";
-                else if (number <= 13)
-                    m_applyFontSize = "3";
-                else if (number <= 16)
-                    m_applyFontSize = "4";
-                else if (number <= 18)
-                    m_applyFontSize = "5";
-                else if (number <= 24)
-                    m_applyFontSize = "6";
-                else
-                    m_applyFontSize = "7";
-                // Huge quirk in Microsft Entourage is that they understand CSS font-size, but also write 
-                // out legacy 1-7 values in font tags (I guess for mailers that are not CSS-savvy at all, 
-                // like Eudora). Yes, they write out *both*. We need to write out both as well. Return false.
-                return false; 
-            }
-            else {
-                // Can't make sense of the number. Put no font size.
-                return true;
-            }
     }
     return false;
 }
@@ -1562,6 +1511,7 @@ bool ApplyStyleCommand::isHTMLStyleNode(CSSMutableStyleDeclarationImpl *style, H
             case CSS_PROP_FONT_STYLE:
                 if (elem->id() == ID_I)
                     return true;
+                break;
         }
     }
 
@@ -1578,37 +1528,6 @@ void ApplyStyleCommand::removeHTMLStyleNode(HTMLElementImpl *elem)
     removeNodePreservingChildren(elem);
 }
 
-void ApplyStyleCommand::removeHTMLFontStyle(CSSMutableStyleDeclarationImpl *style, HTMLElementImpl *elem)
-{
-    ASSERT(style);
-    ASSERT(elem);
-
-    if (elem->id() != ID_FONT)
-        return;
-
-    int exceptionCode = 0;
-    QValueListConstIterator<CSSProperty> end;
-    for (QValueListConstIterator<CSSProperty> it = style->valuesIterator(); it != end; ++it) {
-        switch ((*it).id()) {
-            case CSS_PROP_COLOR:
-                elem->removeAttribute(ATTR_COLOR, exceptionCode);
-                ASSERT(exceptionCode == 0);
-                break;
-            case CSS_PROP_FONT_FAMILY:
-                elem->removeAttribute(ATTR_FACE, exceptionCode);
-                ASSERT(exceptionCode == 0);
-                break;
-            case CSS_PROP_FONT_SIZE:
-                elem->removeAttribute(ATTR_SIZE, exceptionCode);
-                ASSERT(exceptionCode == 0);
-                break;
-        }
-    }
-
-    if (isEmptyFontTag(elem))
-        removeNodePreservingChildren(elem);
-}
-
 void ApplyStyleCommand::removeCSSStyle(CSSMutableStyleDeclarationImpl *style, HTMLElementImpl *elem)
 {
     ASSERT(style);
@@ -1760,7 +1679,7 @@ void ApplyStyleCommand::applyTextDecorationStyle(NodeImpl *node, CSSMutableStyle
 
     HTMLElementImpl *element = static_cast<HTMLElementImpl *>(node);
         
-    StyleChange styleChange(style, Position(element, 0), StyleChange::styleModeForParseMode(document()->inCompatMode()));
+    StyleChange styleChange(style, Position(element, 0), StyleChange::DoNotUseLegacyHTMLStyles);
     if (styleChange.cssStyle().length() > 0) {
         DOMString cssText = styleChange.cssStyle();
         CSSMutableStyleDeclarationImpl *decl = element->inlineStyleDecl();
@@ -1842,13 +1761,10 @@ void ApplyStyleCommand::removeInlineStyle(CSSMutableStyleDeclarationImpl *style,
         NodeImpl *next = node->traverseNextNode();
         if (node->isHTMLElement() && nodeFullySelected(node, start, end)) {
             HTMLElementImpl *elem = static_cast<HTMLElementImpl *>(node);
-            if (isHTMLStyleNode(style, elem)) {
+            if (isHTMLStyleNode(style, elem))
                 removeHTMLStyleNode(elem);
-            }
-            else {
-                removeHTMLFontStyle(style, elem);
+            else
                 removeCSSStyle(style, elem);
-            }
         }
         if (node == end.node())
             break;
@@ -2139,7 +2055,7 @@ void ApplyStyleCommand::addBlockStyleIfNeeded(CSSMutableStyleDeclarationImpl *st
     if (!block)
         return;
         
-    StyleChange styleChange(style, Position(block, 0), StyleChange::styleModeForParseMode(document()->inCompatMode()));
+    StyleChange styleChange(style, Position(block, 0), StyleChange::DoNotUseLegacyHTMLStyles);
     if (styleChange.cssStyle().length() > 0) {
         moveParagraphContentsToNewBlockIfNecessary(Position(node, 0));
         block = static_cast<HTMLElementImpl *>(node->enclosingBlockFlowElement());
@@ -2153,25 +2069,9 @@ void ApplyStyleCommand::addBlockStyleIfNeeded(CSSMutableStyleDeclarationImpl *st
 
 void ApplyStyleCommand::addInlineStyleIfNeeded(CSSMutableStyleDeclarationImpl *style, NodeImpl *startNode, NodeImpl *endNode)
 {
-    StyleChange styleChange(style, Position(startNode, 0), StyleChange::styleModeForParseMode(document()->inCompatMode()));
+    StyleChange styleChange(style, Position(startNode, 0));
     int exceptionCode = 0;
     
-    //
-    // Font tags need to go outside of CSS so that CSS font sizes override leagcy font sizes.
-    //
-    if (styleChange.applyFontColor() || styleChange.applyFontFace() || styleChange.applyFontSize()) {
-        ElementImpl *fontElement = createFontElement(document());
-        ASSERT(exceptionCode == 0);
-        insertNodeBefore(fontElement, startNode);
-        if (styleChange.applyFontColor())
-            fontElement->setAttribute(ATTR_COLOR, styleChange.fontColor());
-        if (styleChange.applyFontFace())
-            fontElement->setAttribute(ATTR_FACE, styleChange.fontFace());
-        if (styleChange.applyFontSize())
-            fontElement->setAttribute(ATTR_SIZE, styleChange.fontSize());
-        surroundNodeRangeWithElement(startNode, endNode, fontElement);
-    }
-
     if (styleChange.cssStyle().length() > 0) {
         ElementImpl *styleElement = createStyleSpanElement(document());
         styleElement->setAttribute(ATTR_STYLE, styleChange.cssStyle());
@@ -5572,15 +5472,6 @@ ElementImpl *createBreakElement(DocumentImpl *document)
     return breakNode;
 }
 
-ElementImpl *createFontElement(DocumentImpl *document)
-{
-    int exceptionCode = 0;
-    ElementImpl *fontNode = document->createHTMLElement("font", exceptionCode);
-    ASSERT(exceptionCode == 0);
-    fontNode->setAttribute(ATTR_CLASS, styleSpanClassString());
-    return fontNode;
-}
-
 ElementImpl *createStyleSpanElement(DocumentImpl *document)
 {
     int exceptionCode = 0;
index 5bd86f794f88e827e4dbc39222ec649e2e263cba..eb7b3cd98c4474b43afa2a1de11f501512c74bcd 100644 (file)
@@ -105,19 +105,9 @@ public:
     explicit StyleChange(DOM::CSSStyleDeclarationImpl *, ELegacyHTMLStyles usesLegacyStyles=UseLegacyHTMLStyles);
     StyleChange(DOM::CSSStyleDeclarationImpl *, const DOM::Position &, ELegacyHTMLStyles usesLegacyStyles=UseLegacyHTMLStyles);
 
-    static ELegacyHTMLStyles styleModeForParseMode(bool);
-
     DOM::DOMString cssStyle() const { return m_cssStyle; }
     bool applyBold() const { return m_applyBold; }
     bool applyItalic() const { return m_applyItalic; }
-    bool applyFontColor() const { return m_applyFontColor.length() > 0; }
-    bool applyFontFace() const { return m_applyFontFace.length() > 0; }
-    bool applyFontSize() const { return m_applyFontSize.length() > 0; }
-
-    DOM::DOMString fontColor() { return m_applyFontColor; }
-    DOM::DOMString fontFace() { return m_applyFontFace; }
-    DOM::DOMString fontSize() { return m_applyFontSize; }
-
     bool usesLegacyStyles() const { return m_usesLegacyStyles; }
 
 private:
@@ -128,9 +118,6 @@ private:
     DOM::DOMString m_cssStyle;
     bool m_applyBold;
     bool m_applyItalic;
-    DOM::DOMString m_applyFontColor;
-    DOM::DOMString m_applyFontFace;
-    DOM::DOMString m_applyFontSize;
     bool m_usesLegacyStyles;
 };
 
@@ -294,7 +281,6 @@ private:
     // style-removal helpers
     bool isHTMLStyleNode(DOM::CSSMutableStyleDeclarationImpl *, DOM::HTMLElementImpl *);
     void removeHTMLStyleNode(DOM::HTMLElementImpl *);
-    void removeHTMLFontStyle(DOM::CSSMutableStyleDeclarationImpl *, DOM::HTMLElementImpl *);
     void removeCSSStyle(DOM::CSSMutableStyleDeclarationImpl *, DOM::HTMLElementImpl *);
     void removeBlockStyle(DOM::CSSMutableStyleDeclarationImpl *, const DOM::Position &start, const DOM::Position &end);
     void removeInlineStyle(DOM::CSSMutableStyleDeclarationImpl *, const DOM::Position &start, const DOM::Position &end);
@@ -932,7 +918,6 @@ private:
 DOM::ElementImpl *createDefaultParagraphElement(DOM::DocumentImpl *document);
 DOM::ElementImpl *createBlockPlaceholderElement(DOM::DocumentImpl *document);
 DOM::ElementImpl *createBreakElement(DOM::DocumentImpl *document);
-DOM::ElementImpl *createFontElement(DOM::DocumentImpl *document);
 DOM::ElementImpl *createStyleSpanElement(DOM::DocumentImpl *document);
 
 bool isNodeRendered(const DOM::NodeImpl *);