LayoutTests:
authormjs <mjs@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Fri, 1 Jun 2007 00:19:05 +0000 (00:19 +0000)
committermjs <mjs@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Fri, 1 Jun 2007 00:19:05 +0000 (00:19 +0000)
        Reviewed by Hyatt.

        - fix layout test editing/style/table-selection.html

        * editing/style/table-selection-expected.txt: Updated selection delegate output.

WebCore:

        Reviewed by Hyatt.

        - fix layout test editing/style/table-selection.html

        This was broken by the change to keep whitespace text nodes in tables. They safely didn't render,
        but the font delta style change command would wrap them in spans, creating extra anonymous table
        cells.

        * editing/ApplyStyleCommand.cpp:
        (WebCore::ApplyStyleCommand::applyRelativeFontStyleChange): Don't wrap in a span if the text node
        is not rendered.

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

LayoutTests/ChangeLog
LayoutTests/editing/style/table-selection-expected.txt
WebCore/ChangeLog
WebCore/editing/ApplyStyleCommand.cpp

index 0b9c851..883b6e8 100644 (file)
@@ -1,3 +1,13 @@
+<<<<<<< .mine
+2007-05-30  Maciej Stachowiak  <mjs@apple.com>
+
+        Reviewed by Hyatt.
+        
+        - fix layout test editing/style/table-selection.html
+
+        * editing/style/table-selection-expected.txt: Updated selection delegate output.
+
+=======
 2007-05-30  Sam Weinig  <sam@webkit.org>
 
         Reviewed by Oliver.
@@ -62,6 +72,7 @@
         * fast/dom/Window/window-properties-expected.txt: adds new constructors
         * fast/dom/wrapper-classes-expected.txt: CSSStyleRule now passes!!
 
+>>>>>>> .r21912
 2007-05-30  Oliver Hunt  <oliver@apple.com>
 
         Reviewed by Mitz.
index 5e0c3db..0f42e49 100644 (file)
@@ -24,5 +24,5 @@ layer at (0,0) size 800x600
               RenderTableCell {TD} at (2,26) size 26x22 [r=1 c=0 rs=1 cs=1]
                 RenderText {#text} at (1,1) size 23x20
                   text run at (1,1) width 23: "bar"
-selection start: position 1 of child 0 {#text} of child 0 {TD} of child 0 {TR} of child 1 {TBODY} of child 1 {TABLE} of child 1 {DIV} of child 1 {BODY} of child 0 {HTML} of document
-selection end:   position 4 of child 0 {#text} of child 0 {TD} of child 1 {TR} of child 1 {TBODY} of child 1 {TABLE} of child 1 {DIV} of child 1 {BODY} of child 0 {HTML} of document
+selection start: position 1 of child 0 {#text} of child 1 {TD} of child 0 {TR} of child 1 {TBODY} of child 1 {TABLE} of child 1 {DIV} of child 1 {BODY} of child 0 {HTML} of document
+selection end:   position 4 of child 0 {#text} of child 1 {TD} of child 2 {TR} of child 1 {TBODY} of child 1 {TABLE} of child 1 {DIV} of child 1 {BODY} of child 0 {HTML} of document
index 5d19491..c2f3a9a 100644 (file)
@@ -1,3 +1,17 @@
+2007-05-30  Maciej Stachowiak  <mjs@apple.com>
+
+        Reviewed by Hyatt.
+
+        - fix layout test editing/style/table-selection.html
+        
+        This was broken by the change to keep whitespace text nodes in tables. They safely didn't render,
+        but the font delta style change command would wrap them in spans, creating extra anonymous table
+        cells.
+
+        * editing/ApplyStyleCommand.cpp:
+        (WebCore::ApplyStyleCommand::applyRelativeFontStyleChange): Don't wrap in a span if the text node
+        is not rendered.
+
 2007-05-31  Alp Toker  <alp.toker@collabora.co.uk>
 
         Reviewed by Eric Seidel.
index dac31b0..83cbd04 100644 (file)
@@ -507,7 +507,7 @@ void ApplyStyleCommand::applyRelativeFontStyleChange(CSSMutableStyleDeclaration
             if (!nodeFullySelected(node, start, end))
                 continue;
             elem = static_cast<HTMLElement *>(node);
-        } else if (node->isTextNode() && node->parentNode() != lastStyledNode) {
+        } else if (node->isTextNode() && node->renderer() && node->parentNode() != lastStyledNode) {
             // Last styled node was not parent node of this text node, but we wish to style this
             // text node. To make this possible, add a style span to surround this text node.
             RefPtr<HTMLElement> span = createStyleSpanElement(document());