LayoutTests:
authorjusting <justing@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Thu, 13 Apr 2006 01:31:24 +0000 (01:31 +0000)
committerjusting <justing@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Thu, 13 Apr 2006 01:31:24 +0000 (01:31 +0000)
commit7a6a34f2c89285b390a97921bff597614ac7951b
treef2c9eac12bb202f563aad7f4b8063d011cdf87a2
parentc93409b1ead96b4ccf96ebb80d4bd9686a9195ee
LayoutTests:

         Reviewed by hyatt

         - updated test results for http://bugzilla.opendarwin.org/show_bug.cgi?id=4855
           List item's bullets fail to redraw correctly after their style is set with JavaScript

         * css1/basic/class_as_selector-expected.txt:
         * css1/basic/containment-expected.txt:
         * css1/basic/contextual_selectors-expected.txt:
         * css1/basic/id_as_selector-expected.txt:
         * css1/box_properties/border_bottom-expected.txt:
         * css1/box_properties/border_left-expected.txt:
         * css1/box_properties/border_right_inline-expected.txt:
         * css1/box_properties/border_top-expected.txt:
         * css1/box_properties/clear_float-expected.txt:
         * css1/box_properties/margin-expected.txt:
         * css1/box_properties/margin_bottom-expected.txt:
         * css1/box_properties/margin_left-expected.txt:
         * css1/box_properties/margin_right-expected.txt:
         * css1/box_properties/margin_top-expected.txt:
         * css1/box_properties/padding_left-expected.txt:
         * css1/box_properties/padding_right-expected.txt:
         * css1/cascade/cascade_order-expected.txt:
         * css1/classification/display-expected.txt:
         * css1/classification/list_style-expected.txt:
         * css1/classification/list_style_image-expected.txt:
         * css1/classification/list_style_position-expected.txt:
         * css1/classification/list_style_type-expected.txt:
         * css1/conformance/forward_compatible_parsing-expected.txt:
         * css1/pseudo/anchor-expected.txt:
         * css2.1/t0402-c71-fwd-parsing-02-f-expected.txt:
         * css2.1/t0505-c16-descendant-01-e-expected.txt:
         * css2.1/t050803-c14-classes-00-e-expected.txt:
         * css2.1/t0509-c15-ids-01-e-expected.txt:
         * css2.1/t0805-c5518-brdr-t-01-e-expected.txt:
         * css2.1/t0805-c5519-brdr-r-02-e-expected.txt:
         * css2.1/t0805-c5520-brdr-b-01-e-expected.txt:
         * css2.1/t0805-c5521-brdr-l-02-e-expected.txt:
         * css2.1/t1205-c561-list-displ-00-b-expected.txt:
         * css2.1/t1205-c563-list-type-00-b-expected.txt:
         * css2.1/t1205-c563-list-type-01-b-expected.txt:
         * css2.1/t1205-c564-list-img-00-b-g-expected.txt:
         * css2.1/t1205-c565-list-pos-00-b-expected.txt:
         * css2.1/t1205-c566-list-stl-00-e-ag-expected.txt:
         * css2.1/t1205-c566-list-stl-01-c-g-expected.txt:
         * editing/deleting/delete-listitem-002-expected.txt:
         * editing/execCommand/insertHTML-expected.txt:
         * editing/pasteboard/paste-4039777-fix-expected.txt:
         * editing/pasteboard/paste-list-001-expected.txt:
         * editing/selection/extend-by-word-002-expected.txt:
         * editing/selection/move-by-line-002-expected.txt:
         * editing/selection/select-all-iframe-expected.txt:
         * editing/selection/selectNode-expected.txt:
         * editing/selection/selectNodeContents-expected.txt:
         * editing/unsupported-content/list-type-after-expected.txt:
         * editing/unsupported-content/list-type-before-expected.txt:
         * fast/block/float/014-expected.txt:
         * fast/css-generated-content/009-expected.txt:
         * fast/css/001-expected.txt:
         * fast/css/001-expected.png:
         * fast/css/001-expected.checksum:
         * fast/css/continuationCrash-expected.txt:
         * fast/css/list-outline-expected.txt:
         * fast/doctypes/001-expected.txt:
         * fast/doctypes/002-expected.txt:
         * fast/doctypes/003-expected.txt:
         * fast/doctypes/004-expected.txt:
         * fast/dom/window-onFocus-expected.txt:
         * fast/forms/form-hides-table-expected.txt:
         * fast/forms/plaintext-mode-2-expected.txt:
         * fast/lists/001-expected.checksum:
         * fast/lists/001-expected.png:
         * fast/lists/001-expected.txt:
         * fast/lists/002-expected.txt:
         * fast/lists/003-expected.txt:
         * fast/lists/004-expected.txt:
         * fast/lists/005-expected.txt:
         * fast/lists/006-expected.txt:
         * fast/lists/007-expected.txt:
         * fast/lists/008-expected.txt:
         * fast/lists/009-expected.txt:
         * fast/lists/drag-into-marker-expected.txt:
         * fast/lists/li-br-expected.txt:
         * fast/lists/ol-display-types-expected.txt:
         * fast/lists/olstart-expected.txt:
         * fast/lists/scrolled-marker-paint-expected.txt:
         * fast/overflow/overflow-rtl-expected.txt:
         * fast/parser/entities-in-xhtml-expected.txt:
         * fast/selectors/001-expected.txt:
         * fast/selectors/013-expected.txt:
         * fast/selectors/015-expected.txt:
         * fast/selectors/166-expected.txt:
         * fast/table/018-expected.txt:
         * fast/table/form-with-table-style-expected.txt:
         * fast/text/whitespace/tab-character-basics-expected.txt:
         * fast/tokenizer/004-expected.txt:
         * tables/mozilla/bugs/bug23235-expected.txt:
         * tables/mozilla/bugs/bug30692-expected.txt:
         * tables/mozilla/bugs/bug3191-expected.txt:
         * tables/mozilla/bugs/bug32205-2-expected.txt:
         * tables/mozilla/marvin/backgr_index-expected.txt:
         * tables/mozilla/marvin/backgr_layers-opacity-expected.txt:
         * tables/mozilla/marvin/x_table-expected.txt:
         * tables/mozilla/other/wa_table_thtd_rowspan-expected.txt:
         * tables/mozilla/other/wa_table_tr_align-expected.txt:
         * tables/mozilla_expected_failures/bugs/bug1010-expected.txt:
         * tables/mozilla_expected_failures/marvin/backgr_fixed-bg-expected.txt:
         * tables/mozilla_expected_failures/marvin/backgr_layers-show-expected.txt:

WebCore:

         Reviewed by hyatt

         - fix http://bugzilla.opendarwin.org/show_bug.cgi?id=4855
           List item's bullets fail to redraw correctly after their style is set with JavaScript

         * manual-tests/list-marker-repaint.html: Added.
         * rendering/RenderBlock.cpp:
         (WebCore::RenderBlock::layoutBlock): Call positionListMarker() after laying out
         the children.
         (WebCore::RenderBlock::calcInlineMinMaxWidth): Call calcWidth() on the child
         if we are going to use its marginLeft() or marginRight(), which is if they are
         given as a percentage or if the child is a list marker.
         * rendering/RenderBlock.h:
         (WebCore::RenderBlock::positionListMarker): Added this virtual function which
         RenderListItem implements and which is called from layoutBlock().
         * rendering/RenderFlow.cpp:
         (WebCore::RenderFlow::addFocusRingRects): Avoid adding focus rings around outside list
         markers. Previously it did not matter since the markers had zero width.
         * rendering/bidi.cpp:
         (WebCore::RenderBlock::findNextLineBreak): Outside list markers should not contribute
         to the line width, even now that they have width.
         * rendering/render_line.cpp:
         (WebCore::InlineFlowBox::placeBoxesHorizontally): Skip outside list markers.
         * rendering/render_list.cpp:
         (WebCore::RenderListItem::positionListMarker): Added.
         (WebCore::RenderListMarker::paint): Removed code that was used to right-align outside
         text markers, since that is achieved by margins now.
         (WebCore::RenderListMarker::calcMinMaxWidth): Changed the marker height to be the font height.
         Made the width of outside list markers non-zero and equal to the width of inside markers.
         Changed the width to include only the marker and not any padding. Increased the width of bullets
         by 2 to contain spillage due to antialiasing.
         (WebCore::RenderListMarker::calcWidth): Calculate horizontal margins. Padding that was
         previously included in the width is now part of the margins.
         (WebCore::RenderListMarker::getRelativeMarkerRect): Adjusted for the changes to width and
         margins.
         * rendering/render_list.h:

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@13853 268f45cc-cd09-0410-ab3c-d52691b4dbfc
110 files changed:
LayoutTests/ChangeLog
LayoutTests/css1/basic/class_as_selector-expected.txt
LayoutTests/css1/basic/containment-expected.txt
LayoutTests/css1/basic/contextual_selectors-expected.txt
LayoutTests/css1/basic/id_as_selector-expected.txt
LayoutTests/css1/box_properties/border_bottom-expected.txt
LayoutTests/css1/box_properties/border_left-expected.txt
LayoutTests/css1/box_properties/border_right_inline-expected.txt
LayoutTests/css1/box_properties/border_top-expected.txt
LayoutTests/css1/box_properties/clear_float-expected.txt
LayoutTests/css1/box_properties/margin-expected.txt
LayoutTests/css1/box_properties/margin_bottom-expected.txt
LayoutTests/css1/box_properties/margin_left-expected.txt
LayoutTests/css1/box_properties/margin_right-expected.txt
LayoutTests/css1/box_properties/margin_top-expected.txt
LayoutTests/css1/box_properties/padding_left-expected.txt
LayoutTests/css1/box_properties/padding_right-expected.txt
LayoutTests/css1/cascade/cascade_order-expected.txt
LayoutTests/css1/classification/display-expected.txt
LayoutTests/css1/classification/list_style-expected.txt
LayoutTests/css1/classification/list_style_image-expected.txt
LayoutTests/css1/classification/list_style_position-expected.txt
LayoutTests/css1/classification/list_style_type-expected.txt
LayoutTests/css1/conformance/forward_compatible_parsing-expected.txt
LayoutTests/css1/pseudo/anchor-expected.txt
LayoutTests/css2.1/t0402-c71-fwd-parsing-02-f-expected.txt
LayoutTests/css2.1/t0505-c16-descendant-01-e-expected.txt
LayoutTests/css2.1/t050803-c14-classes-00-e-expected.txt
LayoutTests/css2.1/t0509-c15-ids-01-e-expected.txt
LayoutTests/css2.1/t0805-c5518-brdr-t-01-e-expected.txt
LayoutTests/css2.1/t0805-c5519-brdr-r-02-e-expected.txt
LayoutTests/css2.1/t0805-c5520-brdr-b-01-e-expected.txt
LayoutTests/css2.1/t0805-c5521-brdr-l-02-e-expected.txt
LayoutTests/css2.1/t1205-c561-list-displ-00-b-expected.txt
LayoutTests/css2.1/t1205-c563-list-type-00-b-expected.txt
LayoutTests/css2.1/t1205-c563-list-type-01-b-expected.txt
LayoutTests/css2.1/t1205-c564-list-img-00-b-g-expected.txt
LayoutTests/css2.1/t1205-c565-list-pos-00-b-expected.txt
LayoutTests/css2.1/t1205-c566-list-stl-00-e-ag-expected.txt
LayoutTests/css2.1/t1205-c566-list-stl-01-c-g-expected.txt
LayoutTests/editing/deleting/delete-listitem-002-expected.txt
LayoutTests/editing/deleting/merge-unrendered-space.html
LayoutTests/editing/execCommand/insertHTML-expected.txt
LayoutTests/editing/pasteboard/paste-4039777-fix-expected.txt
LayoutTests/editing/pasteboard/paste-list-001-expected.txt
LayoutTests/editing/selection/extend-by-word-002-expected.txt
LayoutTests/editing/selection/move-by-line-002-expected.txt
LayoutTests/editing/selection/select-all-iframe-expected.txt
LayoutTests/editing/selection/selectNode-expected.txt
LayoutTests/editing/selection/selectNodeContents-expected.txt
LayoutTests/editing/unsupported-content/list-type-after-expected.txt
LayoutTests/editing/unsupported-content/list-type-before-expected.txt
LayoutTests/fast/block/float/014-expected.txt
LayoutTests/fast/css-generated-content/009-expected.txt
LayoutTests/fast/css/001-expected.txt
LayoutTests/fast/css/continuationCrash-expected.txt
LayoutTests/fast/css/list-outline-expected.txt
LayoutTests/fast/doctypes/001-expected.txt
LayoutTests/fast/doctypes/002-expected.txt
LayoutTests/fast/doctypes/003-expected.txt
LayoutTests/fast/doctypes/004-expected.txt
LayoutTests/fast/dom/window-onFocus-expected.txt
LayoutTests/fast/forms/form-hides-table-expected.txt
LayoutTests/fast/forms/plaintext-mode-2-expected.txt
LayoutTests/fast/lists/001-expected.checksum
LayoutTests/fast/lists/001-expected.png
LayoutTests/fast/lists/001-expected.txt
LayoutTests/fast/lists/002-expected.txt
LayoutTests/fast/lists/003-expected.txt
LayoutTests/fast/lists/004-expected.txt
LayoutTests/fast/lists/005-expected.txt
LayoutTests/fast/lists/006-expected.txt
LayoutTests/fast/lists/007-expected.txt
LayoutTests/fast/lists/008-expected.txt
LayoutTests/fast/lists/009-expected.txt
LayoutTests/fast/lists/drag-into-marker-expected.txt
LayoutTests/fast/lists/li-br-expected.txt
LayoutTests/fast/lists/ol-display-types-expected.txt
LayoutTests/fast/lists/olstart-expected.txt
LayoutTests/fast/lists/scrolled-marker-paint-expected.txt
LayoutTests/fast/overflow/overflow-rtl-expected.txt
LayoutTests/fast/parser/entities-in-xhtml-expected.txt
LayoutTests/fast/selectors/001-expected.txt
LayoutTests/fast/selectors/013-expected.txt
LayoutTests/fast/selectors/015-expected.txt
LayoutTests/fast/selectors/166-expected.txt
LayoutTests/fast/table/018-expected.txt
LayoutTests/fast/table/form-with-table-style-expected.txt
LayoutTests/fast/text/whitespace/tab-character-basics-expected.txt
LayoutTests/fast/tokenizer/004-expected.txt
LayoutTests/tables/mozilla/bugs/bug23235-expected.txt
LayoutTests/tables/mozilla/bugs/bug30692-expected.txt
LayoutTests/tables/mozilla/bugs/bug3191-expected.txt
LayoutTests/tables/mozilla/bugs/bug32205-2-expected.txt
LayoutTests/tables/mozilla/marvin/backgr_index-expected.txt
LayoutTests/tables/mozilla/marvin/backgr_layers-opacity-expected.txt
LayoutTests/tables/mozilla/marvin/x_table-expected.txt
LayoutTests/tables/mozilla/other/wa_table_thtd_rowspan-expected.txt
LayoutTests/tables/mozilla/other/wa_table_tr_align-expected.txt
LayoutTests/tables/mozilla_expected_failures/bugs/bug1010-expected.txt
LayoutTests/tables/mozilla_expected_failures/marvin/backgr_fixed-bg-expected.txt
LayoutTests/tables/mozilla_expected_failures/marvin/backgr_layers-show-expected.txt
WebCore/ChangeLog
WebCore/rendering/RenderBlock.cpp
WebCore/rendering/RenderBlock.h
WebCore/rendering/RenderFlow.cpp
WebCore/rendering/bidi.cpp
WebCore/rendering/render_line.cpp
WebCore/rendering/render_list.cpp
WebCore/rendering/render_list.h