Skip to site navigation (Press enter)

[webkit-changes] [88175] trunk

yael . aharon Mon, 06 Jun 2011 11:33:25 -0700

Title: [88175] trunk

Revision: 88175
Author: [email protected]
Date: 2011-06-06 11:32:54 -0700 (Mon, 06 Jun 2011)

Log Message

2011-06-06  Yael Aharon  <[email protected]>

        Reviewed by Eric Seidel.


        Inline elements are wrapped prematurely with non-zero border, margin or padding.
        https://bugs.webkit.org/show_bug.cgi?id=60459

        * fast/inline/inline-wrap-with-parent-padding.html: Added.
        * platform/mac/fast/inline/inline-wrap-with-parent-padding-expected.png: Added.
        * platform/mac/fast/inline/inline-wrap-with-parent-padding-expected.txt: Added.
2011-06-06  Yael Aharon  <[email protected]>

        Reviewed by Eric Seidel.

        Inline elements are wrapped prematurely with non-zero border, margin or padding.
        https://bugs.webkit.org/show_bug.cgi?id=60459

        When an inline element has a right border/margin/padding and it has more than one descendant with no siblings,
        the width of the right border/margin/padding should be included in line breaking calculation only once,
        and not for each descendant.

        Test: fast/inline/inline-wrap-with-parent-padding.html

        * rendering/RenderBlockLineLayout.cpp:
        (WebCore::RenderBlock::LineBreaker::nextLineBreak):

Modified Paths

trunk/LayoutTests/ChangeLog
trunk/Source/WebCore/ChangeLog
trunk/Source/WebCore/rendering/RenderBlockLineLayout.cpp

Added Paths

trunk/LayoutTests/fast/inline/inline-wrap-with-parent-padding.html
trunk/LayoutTests/platform/mac/fast/inline/inline-wrap-with-parent-padding-expected.png
trunk/LayoutTests/platform/mac/fast/inline/inline-wrap-with-parent-padding-expected.txt

Diff

Modified: trunk/LayoutTests/ChangeLog (88174 => 88175)


--- trunk/LayoutTests/ChangeLog	2011-06-06 18:03:05 UTC (rev 88174)
+++ trunk/LayoutTests/ChangeLog	2011-06-06 18:32:54 UTC (rev 88175)
@@ -1,3 +1,14 @@
+2011-06-06  Yael Aharon  <[email protected]>
+
+        Reviewed by Eric Seidel.
+
+        Inline elements are wrapped prematurely with non-zero border, margin or padding.
+        https://bugs.webkit.org/show_bug.cgi?id=60459
+
+        * fast/inline/inline-wrap-with-parent-padding.html: Added.
+        * platform/mac/fast/inline/inline-wrap-with-parent-padding-expected.png: Added.
+        * platform/mac/fast/inline/inline-wrap-with-parent-padding-expected.txt: Added.
+
 2011-06-06  Shishir Agrawal  <[email protected]>
 
         Reviewed by Tony Gentilcore.

Added: trunk/LayoutTests/fast/inline/inline-wrap-with-parent-padding.html (0 => 88175)


--- trunk/LayoutTests/fast/inline/inline-wrap-with-parent-padding.html	                        (rev 0)
+++ trunk/LayoutTests/fast/inline/inline-wrap-with-parent-padding.html	2011-06-06 18:32:54 UTC (rev 88175)
@@ -0,0 +1,13 @@
+<!DOCTYPE html>
+<html>
+<head>
+<style>
+div { width: 320px; }
+</style>
+</head>
+<body>
+<div style="border: 1px solid red">
+<span style="border: 1px solid green; padding-right: 15px">-- <span>-- <span>-- <span>-- <span>-- <span>-- <span>-- <span>-- <span>-- <span>-- <span>-- <span>-- <span>-- <span>-- <span>-- <span>-- <span>-- <span>-- <span>-- <span>-- <span>-- <span>-- <span>-- <span>--</span></span></span></span></span></span></span></span></span></span></span></span></span></span></span></span></span></span></span></span></span></span></span></span>
+</div>
+</body>
+</html>
\ No newline at end of file

Added: trunk/LayoutTests/platform/mac/fast/inline/inline-wrap-with-parent-padding-expected.png (0 => 88175)


--- trunk/LayoutTests/platform/mac/fast/inline/inline-wrap-with-parent-padding-expected.png	                        (rev 0)
+++ trunk/LayoutTests/platform/mac/fast/inline/inline-wrap-with-parent-padding-expected.png	2011-06-06 18:32:54 UTC (rev 88175)
@@ -0,0 +1,11 @@
+\x89PNG
+
+
+IHDR X\x9Av\x82p)tEXtchecksum7c887338f071fa5651edec9073daa043K\xB1\xDAl\xEEiCCPICC Profilex\x85T\xCFkA\xFE6n\xA9\xD0"Zk\xB2x\x90"IY\xABhE\xD46\xFDbk\xDB\xB6E\x90d3I\xD6n6\xEB\xEE&\xB5\xA5\x88\xE4\xE2\xD1*\xDEE\xED\xA1\xFF\x80z\xF0d/J\x85ZE(ޫ(b\xA1-\xF1\xCDnL\xB6\xA5\xEA\xC0\xCE~\xF3\xDE7\xEF}ov\xDF
+r\xD24\xF5\x80\xE4
+\xC7R\xA2il|Bj\xFC\x88\x8E\xA2	A4%U\xDB\xECN$A\x83s\xF9{\xE7\xD8z\x81[V\xC3{\xFBw\xB2w\xAD\x9AҶ\x9A\x84\xFD@\xE0G\x9A\xD9*\xB0\xEFq
+Y\x88<ߡ)\xC7t\xDF\xE3\xD8\xF2\xEC\x8F9Nyx\xC1\xB5+=\xC4Y"|@5-\xCEM\xB8S\xCD%\xD3@\x83H8\x94\xF5qR>\x9C׋\x94\xD7inf\xC6\xC8\xBDO\x90\xA6\xBB\xCC\xEE\xABb\xA1\x9CN\xF6\x90\xBD\x9D\xF4~N\xB3\xDE>\xC2!\xC2\xAD?F\xB8\x8D\x9E\xF5\x8C\xD5?\xE2a\xE1\xA4\xE6Ć=5\xF4\xF8`\xB7\xA9\xF85\xC2_M'\xA2Tq\xD9.
+\xF1\x98\xAE\xFDV\xF2J\x82p\x908\xCAda\x80sZHO\xD7Ln\xF8\xBA\x87}&\xD7\xAF\xE2wVQ\xE1y\x8Fg\xDE\xD4\xDD\xEFE\xDA\xAF0\x9A
+HPEa\x98\xB0P@\x86<14\xB2r?#\xAB\x93{2u$j\xBBtbD\xB1A{6\xDC=\xB7Q\xA4\xDD<\xFE("q\x94C\xB5\x92\xFCA\xFE*\xAF\xC9O\xE5y\xF9\xCB\\xB0\xD8V\x{1D4B5B}\x9A\xBA\xF2\xE0;\xC5噹\xD7\xD3\xC8\xE3sM^|\x95\xD4v\x93WG\x96\xACyz\xBC\x9A\xEC?\xECW\x971\xE6\x825\x8F\xC4s\xB0\xFB\xF1-_\x95̗)\x8C\xC5\xE3U\xF3\xEAK\x84uZ17ߟl;=\xE2.\xCF.\xB5\xD6s\xAD\x89\x8B7V\x9B\x97g\xFDjH\xFB\x93\xE6U\xF9O^\xF5\xF1\xFCg\xCD\xC4c\xE2)1&v\x8A\xE7!\x89\x97\xC5.\xF1\x92\xD8K\xAB\xE2`m\xC7\x9D\x95\x86)\xD2m\x91\xFA$\xD5``\x9A\xBC\xF5/]?[x\xBDF\xF5Q\x94\xCC\xD2T\x89\xF7\xC2*d4\x9D\xB9o\xFA\x8F\xDB\xC7\xFC\xE4\xF7\x8A\xE7\x9F(/l\xE0ș\xBAmSq\xEF\xA1e\xA5ns\xAE\xBF\xD1}\xF0\xB6nk\xA3~8\xFCX<\xAB\xADR5\x9F \xBCv\x87z\xE8)\x98Ӗ\x96\xCD9R\x87,\x9F\x93\xBA\xE9\xCAbR\xCCP\xDBCRR\xD7%\xD7eK\xB3\x99Ub\xE9vؙ\xD3n\xA19B\xF7ħJe\x93\xFA\xAF\xF1\xB0\xFD\xB0R\xF9\xF9\xACR\xD9~Nց\x97\xFAo\xC0\xBC\xFDE\xC0x\x8B\x89 IDATx\xED\xDD\xD1Q\x9CFљ\xAD̀\x80\x88NrD\xA4@<\x92\x96\x8B\xCB\xC5Ly_,\xBEÃ\x8B4\xAD>
+X\xD7e1\xE7\x97ח\x93 @\x80\xFF	fA\x80 @\x80\x81\xDF\x88O @\x802\x92QD\x80 @\x80\x80\xF19@\x80 @\x80@& @2j\x83 @\x80 > @\x80\xC8HFm @\x80\xFF\xBDHp>\x9F\xCE\xDF/~\x94 @\x80\xC0\x97x\xF9\xFE\xC7J\x9E.\xE9\xCB\xDD\xD6B \xF0y\x97\xE4\x8F\xDF\xDB\xCB7\xCFW\xF8yg2\x99\x81O\xF8\xF6:\xF5\xF5?\xC2x!@\x80\x8E\xF0\xBF`g\xE9\x91 @\x80\xB8  @.y7 @\x80\xC7	\x90\xE3,= @\x80\xC8 \xEF&@\x80 @\xE08r\x9C\xA5G"@\x80 @\xE0\x82\x80\xB9\xE4\xDD @\x80' @\x8E\xB3\xF4H @\x80\ \x80\xBC\x9B @\x80\x81\xE3\xAE~"§\xA7\xA7\xD3\xE3\xE3\xE3\xFB任\xBB\xD3\xCD\xCD\xCDOo\xBB\xBF\xBF\xFF\x98\xB7W\xFC\xBA\xD3\xE9͊\x8B\xCF_?\xBE5\xFC\xCA_?6\xF0O @\xE0\xFF\xB8:@\x9E\x9F\x9FO\xEF\xB3ί\xCF|{{\xFB\xD3\xDB\xFE\xFCm\xBF\xEE퉔\xCF?.o_L\xBE\x8E~|K\xF9U\xBE޿z\x85\xF8\xDB\xE7\x97ח\xF5\xEB\x9E\xCF\xDFO\xA7\x97o؇\x8F\xE1\x9D\xF8U^\xBF\x9E.|\x9B\xFCUW\xF3\xFB&@\x80\x9F!\xE0\xEF\x80|\x86\xBA\x99 @\x80F\xC8\xE8\xE1\xADM\x80 @\xE03\xC8g\xA8\x9BI\x80 @`T@\x80\x8C\xDE\xDA @\x80>C@\x80|\x86\xBA\x99 @\x80F\xAE\xFE1\xBC\xE7\xDF^\xCC?\xFC\xE2'm\xFD\xC3\xC0\x9E @\x80\xC0'\ \xC5ﳈ\x9Cb3\xF8\xFE\xA3\xC8׸\xA3- @\xE0\xDF%p\xD5\xF3\x80?\xFF->\xFC\xCB\xFE\xDF\xF5\xC9\xE1wC\x80\xC0\xAB\x80\xE7\xF1i@\x80\xF0w@\xE5\xF4` @\x80|$ @>\xD2\xF1> @\x80 \x87rz0 @\x80> \xE9x @\x80\x87
+\x90C9= @\x80	\x90\x8Ft\xBC\x8F @\x80\x81Cȡ\x9C\x8C @\x80\x81\x8F\xAEz"§\xA7\xA7\xD3\xE3\xE3\xE3\xFB\xE3\xDC\xDDݝnnn~z\xDB\xFD\xFD\xFD\xFB\xC7x\x85 @\x80\xB8*@\x9E\x9F\x9FO\xEF\xBF\xF6\xFC\xFA\xC4\\xB7\xB7\xB7?\xBDM\x80\xBCy\x85 @\x80\x81\xBF\xF0L\xE8\x81\xE2Mx\xF0L\xE8\xEF^!@\x80G\xF8; G(z @\x80\xAE W1\xF9  @\x80\x8E G(z @\x80\xAE W1\xF9  @\x80\x8E G(z @\x80\xAE W1\xF9  @\x80\x8E\xB8\xEAy@ο\x9D\x8F\x98u\xF91\xDE~ܥ @\x80\xBE\xAC\xC0\xE5\xE7\xF9\xB2\xAB[\x8C @\x80\x81Z\xC0\xFF\x82U\x8B\x9BG\x80 @`X@\x80\xDF\xEA @\x80jR\x8B\x9BG\x80 @`X@\x80\xDF\xEA @\x80jR\x8B\x9BG\x80 @`X@\x80\xDF\xEA @\x80jR\x8B\x9BG\x80 @`X@\x80\xDF\xEA @\x80jR\x8B\x9BG\x80 @`X@\x80\xDF\xEA @\x80jR\x8B\x9BG\x80 @`X@\x80\xDF\xEA @\x80jR\x8B\x9BG\x80 @`X@\x80\xDF\xEA @\x80jR\x8B\x9BG\x80 @`X@\x80\xDF\xEA @\x80jR\x8B\x9BG\x80 @`X@\x80\xDF\xEA @\x80jR\x8B\x9BG\x80 @`X@\x80\xDF\xEA @\x80jR\x8B\x9BG\x80 @`X@\x80\xDF\xEA @\x80jR\x8B\x9BG\x80 @`X@\x80\xDF\xEA @\x80jR\x8B\x9BG\x80 @`X@\x80\xDF\xEA @\x80jR\x8B\x9BG\x80 @`X@\x80\xDF\xEA @\x80jR\x8B\x9BG\x80 @`X@\x80\xDF\xEA @\x80jR\x8B\x9BG\x80 @`X@\x80\xDF\xEA @\x80jR\x8B\x9BG\x80 @`X@\x80\xDF\xEA @\x80jR\x8B\x9BG\x80 @`X@\x80\xDF\xEA @\x80jR\x8B\x9BG\x80 @`X@\x80\xDF\xEA @\x80jR\x8B\x9BG\x80 @`X@\x80\xDF\xEA @\x80jR\x8B\x9BG\x80 @`X@\x80\xDF\xEA @\x80jR\x8B\x9BG\x80 @`X@\x80\xDF\xEA @\x80jR\x8B\x9BG\x80 @`X@\x80\xDF\xEA @\x80jR\x8B\x9BG\x80 @`X@\x80\xDF\xEA @\x80jR\x8B\x9BG\x80 @`X@\x80\xDF\xEA @\x80jR\x8B\x9BG\x80 @`X@\x80\xDF\xEA @\x80jR\x8B\x9BG\x80 @`X@\x80\xDF\xEA @\x80jR\x8B\x9BG\x80 @`X@\x80\xDF\xEA @\x80jR\x8B\x9BG\x80 @`X@\x80\xDF\xEA @\x80jR\x8B\x9BG\x80 @`X@\x80\xDF\xEA @\x80jR\x8B\x9BG\x80 @`X@\x80\xDF\xEA @\x80jR\x8B\x9BG\x80 @`X@\x80\xDF\xEA @\x80jR\x8B\x9BG\x80 @`X@\x80\xDF\xEA @\x80jR\x8B\x9BG\x80 @`X@\x80\xDF\xEA @\x80jR\x8B\x9BG\x80 @`X@\x80\xDF\xEA @\x80jR\x8B\x9BG\x80 @`X@\x80\xDF\xEA @\x80jR\x8B\x9BG\x80 @`X@\x80\xDF\xEA @\x80jR\x8B\x9BG\x80 @`X@\x80\xDF\xEA @\x80jR\x8B\x9BG\x80 @`X@\x80\xDF\xEA @\x80jR\x8B\x9BG\x80 @`X@\x80\xDF\xEA @\x80jR\x8B\x9BG\x80 @`X@\x80\xDF\xEA @\x80jR\x8B\x9BG\x80 @`X@\x80\xDF\xEA @\x80jR\x8B\x9BG\x80 @`X@\x80\xDF\xEA @\x80jR\x8B\x9BG\x80 @`X@\x80\xDF\xEA @\x80jR\x8B\x9BG\x80 @`X@\x80\xDF\xEA @\x80jR\x8B\x9BG\x80 @`X@\x80\xDF\xEA @\x80jR\x8B\x9BG\x80 @`X@\x80\xDF\xEA @\x80jR\x8B\x9BG\x80 @`X@\x80\xDF\xEA @\x80jR\x8B\x9BG\x80 @`X@\x80\xDF\xEA @\x80jR\x8B\x9BG\x80 @`X@\x80\xDF\xEA @\x80jR\x8B\x9BG\x80 @`X@\x80\xDF\xEA @\x80jR\x8B\x9BG\x80 @`X@\x80\xDF\xEA @\x80jR\x8B\x9BG\x80 @`X@\x80\xDF\xEA @\x80jR\x8B\x9BG\x80 @`X@\x80\xDF\xEA @\x80jR\x8B\x9BG\x80 @`X@\x80\xDF\xEA @\x80jR\x8B\x9BG\x80 @`X@\x80\xDF\xEA @\x80jR\x8B\x9BG\x80 @`X@\x80\xDF\xEA @\x80jR\x8B\x9BG\x80 @`X@\x80\xDF\xEA @\x80jR\x8B\x9BG\x80 @`X@\x80\xDF\xEA @\x80jR\x8B\x9BG\x80 @`X@\x80\xDF\xEA @\x80jR\x8B\x9BG\x80 @`X@\x80\xDF\xEA @\x80jR\x8B\x9BG\x80 @`X@\x80\xDF\xEA @\x80jR\x8B\x9BG\x80 @`X@\x80\xDF\xEA @\x80jR\x8B\x9BG\x80 @`X@\x80\xDF\xEA @\x80jR\x8B\x9BG\x80 @`X@\x80\xDF\xEA @\x80jR\x8B\x9BG\x80 @`X@\x80\xDF\xEA @\x80jR\x8B\x9BG\x80 @`X@\x80\xDF\xEA @\x80jR\x8B\x9BG\x80 @`X@\x80\xDF\xEA @\x80jR\x8B\x9BG\x80 @`X@\x80\xDF\xEA @\x80jR\x8B\x9BG\x80 @`X@\x80\xDF\xEA @\x80jR\x8B\x9BG\x80 @`X@\x80\xDF\xEA @\x80jR\x8B\x9BG\x80 @`X@\x80\xDF\xEA @\x80jR\x8B\x9BG\x80 @`X@\x80\xDF\xEA @\x80jR\x8B\x9BG\x80 @`X@\x80\xDF\xEA @\x80jR\x8B\x9BG\x80 @`X@\x80\xDF\xEA @\x80jR\x8B\x9BG\x80 @`X@\x80\xDF\xEA @\x80jR\x8B\x9BG\x80 @`X@\x80\xDF\xEA @\x80jR\x8B\x9BG\x80 @`X@\x80\xDF\xEA @\x80jR\x8B\x9BG\x80 @`X@\x80\xDF\xEA @\x80jR\x8B\x9BG\x80 @`X@\x80\xDF\xEA @\x80jR\x8B\x9BG\x80 @`X@\x80\xDF\xEA @\x80jR\x8B\x9BG\x80 @`X@\x80\xDF\xEA @\x80jR\x8B\x9BG\x80 @`X@\x80\xDF\xEA @\x80jR\x8B\x9BG\x80 @`X@\x80\xDF\xEA @\x80jR\x8B\x9BG\x80 @`X@\x80\xDF\xEA @\x80jR\x8B\x9BG\x80 @`X@\x80\xDF\xEA @\x80jR\x8B\x9BG\x80 @`X@\x80\xDF\xEA @\x80jR\x8B\x9BG\x80 @`X@\x80\xDF\xEA @\x80jR\x8B\x9BG\x80 @`X@\x80\xDF\xEA @\x80jR\x8B\x9BG\x80 @`X@\x80\xDF\xEA @\x80jR\x8B\x9BG\x80 @`X@\x80\xDF\xEA @\x80jR\x8B\x9BG\x80 @`X@\x80\xDF\xEA @\x80jR\x8B\x9BG\x80 @`X@\x80\xDF\xEA @\x80jR\x8B\x9BG\x80 @`X@\x80\xDF\xEA @\x80jR\x8B\x9BG\x80 @`X@\x80\xDF\xEA @\x80jR\x8B\x9BG\x80 @`X@\x80\xDF\xEA @\x80jR\x8B\x9BG\x80 @`X@\x80\xDF\xEA @\x80jR\x8B\x9BG\x80 @`X@\x80\xDF\xEA @\x80jR\x8B\x9BG\x80 @`X@\x80\xDF\xEA @\x80jR\x8B\x9BG\x80 @`X@\x80\xDF\xEA @\x80jR\x8B\x9BG\x80 @`X@\x80\xDF\xEA @\x80jR\x8B\x9BG\x80 @`X@\x80\xDF\xEA @\x80jR\x8B\x9BG\x80 @`X@\x80\xDF\xEA @\x80jR\x8B\x9BG\x80 @`X@\x80\xDF\xEA @\x80jR\x8B\x9BG\x80 @`X@\x80\xDF\xEA @\x80jR\x8B\x9BG\x80 @`X@\x80\xDF\xEA @\x80jR\x8B\x9BG\x80 @`X@\x80\xDF\xEA @\x80jR\x8B\x9BG\x80 @`X@\x80\xDF\xEA @\x80jR\x8B\x9BG\x80 @`X@\x80\xDF\xEA @\x80jR\x8B\x9BG\x80 @`X@\x80\xDF\xEA @\x80jR\x8B\x9BG\x80 @`X@\x80\xDF\xEA @\x80jR\x8B\x9BG\x80 @`X@\x80\xDF\xEA @\x80jR\x8B\x9BG\x80 @`X@\x80\xDF\xEA @\x80jR\x8B\x9BG\x80 @`X@\x80\xDF\xEA @\x80jR\x8B\x9BG\x80 @`X@\x80\xDF\xEA @\x80jR\x8B\x9BG\x80 @`X@\x80\xDF\xEA @\x80jR\x8B\x9BG\x80 @`X@\x80\xDF\xEA @\x80jR\x8B\x9BG\x80 @`X@\x80\xDF\xEA @\x80jR\x8B\x9BG\x80 @`X@\x80\xDF\xEA @\x80jR\x8B\x9BG\x80 @`X@\x80\xDF\xEA @\x80jR\x8B\x9BG\x80 @`X@\x80\xDF\xEA @\x80jR\x8B\x9BG\x80 @`X@\x80\xDF\xEA @\x80jR\x8B\x9BG\x80 @`X@\x80\xDF\xEA @\x80jR\x8B\x9BG\x80 @`X@\x80\xDF\xEA @\x80jR\x8B\x9BG\x80 @`X@\x80\xDF\xEA @\x80jR\x8B\x9BG\x80 @`X@\x80\xDF\xEA @\x80jR\x8B\x9BG\x80 @`X@\x80\xDF\xEA @\x80jR\x8B\x9BG\x80 @`X@\x80\xDF\xEA @\x80jR\x8B\x9BG\x80 @`X@\x80\xDF\xEA @\x80jR\x8B\x9BG\x80 @`X@\x80\xDF\xEA @\x80jR\x8B\x9BG\x80 @`X@\x80\xDF\xEA @\x80jR\x8B\x9BG\x80 @`X@\x80\xDF\xEA @\x80jR\x8B\x9BG\x80 @`X@\x80\xDF\xEA @\x80jR\x8B\x9BG\x80 @`X@\x80\xDF\xEA @\x80jR\x8B\x9BG\x80 @`X@\x80\xDF\xEA @\x80jR\x8B\x9BG\x80 @`X@\x80\xDF\xEA @\x80jR\x8B\x9BG\x80 @`X@\x80\xDF\xEA @\x80jR\x8B\x9BG\x80 @`X@\x80\xDF\xEA @\x80jR\x8B\x9BG\x80 @`X@\x80\xDF\xEA @\x80jR\x8B\x9BG\x80 @`X@\x80\xDF\xEA @\x80jR\x8B\x9BG\x80 @`X@\x80\xDF\xEA @\x80jR\x8B\x9BG\x80 @`X@\x80\xDF\xEA @\x80jR\x8B\x9BG\x80 @`X@\x80\xDF\xEA @\x80jR\x8B\x9BG\x80 @`X@\x80\xDF\xEA @\x80jR\x8B\x9BG\x80 @`X@\x80\xDF\xEA @\x80jR\x8B\x9BG\x80 @`X@\x80\xDF\xEA @\x80jR\x8B\x9BG\x80 @`X@\x80\xDF\xEA @\x80jR\x8B\x9BG\x80 @`X@\x80\xDF\xEA @\x80jR\x8B\x9BG\x80 @`X@\x80\xDF\xEA @\x80jR\x8B\x9BG\x80 @`X@\x80\xDF\xEA @\x80jR\x8B\x9BG\x80 @`X@\x80\xDF\xEA @\x80jR\x8B\x9BG\x80 @`X@\x80\xDF\xEA @\x80jR\x8B\x9BG\x80 @`X@\x80\xDF\xEA @\x80jR\x8B\x9BG\x80 @`X@\x80\xDF\xEA @\x80jR\x8B\x9BG\x80 @`X@\x80\xDF\xEA @\x80jR\x8B\x9BG\x80 @`X@\x80\xDF\xEA @\x80jR\x8B\x9BG\x80 @`X@\x80\xDF\xEA @\x80jR\x8B\x9BG\x80 @`X@\x80\xDF\xEA @\x80jR\x8B\x9BG\x80 @`X@\x80\xDF\xEA @\x80jR\x8B\x9BG\x80 @`X@\x80\xDF\xEA @\x80jR\x8B\x9BG\x80 @`X@\x80\xDF\xEA @\x80jR\x8B\x9BG\x80 @`X@\x80\xDF\xEA @\x80jR\x8B\x9BG\x80 @`X@\x80\xDF\xEA @\x80jR\x8B\x9BG\x80 @`X@\x80\xDF\xEA @\x80jR\x8B\x9BG\x80 @`X@\x80\xDF\xEA @\x80jR\x8B\x9BG\x80 @`X@\x80\xDF\xEA @\x80jR\x8B\x9BG\x80 @`X@\x80\xDF\xEA @\x80jR\x8B\x9BG\x80 @`X@\x80\xDF\xEA @\x80jR\x8B\x9BG\x80 @`X@\x80\xDF\xEA @\x80jR\x8B\x9BG\x80 @`X@\x80\xDF\xEA @\x80jR\x8B\x9BG\x80 @`X@\x80\xDF\xEA @\x80jR\x8B\x9BG\x80 @`X@\x80\xDF\xEA @\x80jR\x8B\x9BG\x80 @`X@\x80\xDF\xEA @\x80jR\x8B\x9BG\x80 @`X@\x80\xDF\xEA @\x80jR\x8B\x9BG\x80 @`X@\x80\xDF\xEA @\x80jR\x8B\x9BG\x80 @`X@\x80\xDF\xEA @\x80jR\x8B\x9BG\x80 @`X@\x80\xDF\xEA @\x80jR\x8B\x9BG\x80 @`X@\x80\xDF\xEA @\x80jR\x8B\x9BG\x80 @`X@\x80\xDF\xEA @\x80jR\x8B\x9BG\x80 @`X@\x80\xDF\xEA @\x80jR\x8B\x9BG\x80 @`X@\x80\xDF\xEA @\x80jR\x8B\x9BG\x80 @`X@\x80\xDF\xEA @\x80jR\x8B\x9BG\x80 @`X@\x80\xDF\xEA @\x80jR\x8B\x9BG\x80 @`X@\x80\xDF\xEA @\x80jR\x8B\x9BG\x80 @`X@\x80\xDF\xEA @\x80jR\x8B\x9BG\x80 @`X@\x80\xDF\xEA @\x80jR\x8B\x9BG\x80 @`X@\x80\xDF\xEA @\x80jR\x8B\x9BG\x80 @`X@\x80\xDF\xEA @\x80jR\x8B\x9BG\x80 @`X@\x80\xDF\xEA @\x80jR\x8B\x9BG\x80 @`X@\x80\xDF\xEA @\x80jR\x8B\x9BG\x80 @`X@\x80\xDF\xEA @\x80jR\x8B\x9BG\x80 @`X@\x80\xDF\xEA @\x80jR\x8B\x9BG\x80 @`X@\x80\xDF\xEA @\x80jR\x8B\x9BG\x80 @`X@\x80\xDF\xEA @\x80jR\x8B\x9BG\x80 @`X@\x80\xDF\xEA @\x80jR\x8B\x9BG\x80 @`X@\x80\xDF\xEA @\x80jR\x8B\x9BG\x80 @`X@\x80\xDF\xEA @\x80jR\x8B\x9BG\x80 @`X@\x80\xDF\xEA @\x80jR\x8B\x9BG\x80 @`X@\x80\xDF\xEA @\x80jR\x8B\x9BG\x80 @`X@\x80\xDF\xEA @\x80jR\x8B\x9BG\x80 @`X@\x80\xDF\xEA @\x80jR\x8B\x9BG\x80 @`X@\x80\xDF\xEA @\x80jR\x8B\x9BG\x80 @`X@\x80\xDF\xEA @\x80jR\x8B\x9BG\x80 @`X@\x80\xDF\xEA @\x80jR\x8B\x9BG\x80 @`X@\x80\xDF\xEA @\x80jR\x8B\x9BG\x80 @`X@\x80\xDF\xEA @\x80jK\x87	U	bIDATR\x8B\x9BG\x80 @`X@\x80\xDF\xEA @\x80jR\x8B\x9BG\x80 @`X@\x80\xDF\xEA @\x80jR\x8B\x9BG\x80 @`X@\x80\xDF\xEA @\x80jR\x8B\x9BG\x80 @`X@\x80\xDF\xEA @\x80jR\x8B\x9BG\x80 @`X@\x80\xDF\xEA @\x80jR\x8B\x9BG\x80 @`X@\x80\xDF\xEA @\x80jR\x8B\x9BG\x80 @`X@\x80\xDF\xEA @\x80jR\x8B\x9BG\x80 @`X@\x80\xDF\xEA @\x80jR\x8B\x9BG\x80 @`X@\x80\xDF\xEA @\x80jR\x8B\x9BG\x80 @`X@\x80\xDF\xEA @\x80jR\x8B\x9BG\x80 @`X@\x80\xDF\xEA @\x80jR\x8B\x9BG\x80 @`X@\x80\xDF\xEA @\x80jR\x8B\x9BG\x80 @`X@\x80\xDF\xEA @\x80jR\x8B\x9BG\x80 @`X@\x80\xDF\xEA @\x80jR\x8B\x9BG\x80 @`X@\x80\xDF\xEA @\x80jR\x8B\x9BG\x80 @`X@\x80\xDF\xEA @\x80jR\x8B\x9BG\x80 @`X@\x80\xDF\xEA @\x80jR\x8B\x9BG\x80 @`X@\x80\xDF\xEA @\x80jR\x8B\x9BG\x80 @`X@\x80\xDF\xEA @\x80jR\x8B\x9BG\x80 @`X@\x80\xDF\xEA @\x80jR\x8B\x9BG\x80 @`X@\x80\xDF\xEA @\x80jR\x8B\x9BG\x80 @`X@\x80\xDF\xEA @\x80jR\x8B\x9BG\x80 @`X@\x80\xDF\xEA @\x80jR\x8B\x9BG\x80 @`X@\x80\xDF\xEA @\x80jR\x8B\x9BG\x80 @`X@\x80\xDF\xEA @\x80jR\x8B\x9BG\x80 @`X@\x80\xDF\xEA @\x80jR\x8B\x9BG\x80 @`X@\x80\xDF\xEA @\x80jR\x8B\x9BG\x80 @`X@\x80\xDF\xEA @\x80jR\x8B\x9BG\x80 @`X@\x80\xDF\xEA @\x80jR\x8B\x9BG\x80 @`X@\x80\xDF\xEA @\x80jR\x8B\x9BG\x80 @`X@\x80\xDF\xEA @\x80jR\x8B\x9BG\x80 @`X@\x80\xDF\xEA @\x80jR\x8B\x9BG\x80 @`X@\x80\xDF\xEA @\x80jR\x8B\x9BG\x80 @`X@\x80\xDF\xEA @\x80jR\x8B\x9BG\x80 @`X@\x80\xDF\xEA @\x80jR\x8B\x9BG\x80 @`X@\x80\xDF\xEA @\x80jR\x8B\x9BG\x80 @`X@\x80\xDF\xEA @\x80jR\x8B\x9BG\x80 @`X@\x80\xDF\xEA @\x80jR\x8B\x9BG\x80 @`X@\x80\xDF\xEA @\x80jR\x8B\x9BG\x80 @`X@\x80\xDF\xEA @\x80jR\x8B\x9BG\x80 @`X@\x80\xDF\xEA @\x80jR\x8B\x9BG\x80 @`X@\x80\xDF\xEA @\x80jR\x8B\x9BG\x80 @`X@\x80\xDF\xEA @\x80jR\x8B\x9BG\x80 @`X@\x80\xDF\xEA @\x80jR\x8B\x9BG\x80 @`X@\x80\xDF\xEA @\x80jR\x8B\x9BG\x80 @`X@\x80\xDF\xEA @\x80jR\x8B\x9BG\x80 @`X@\x80\xDF\xEA @\x80jR\x8B\x9BG\x80 @`X@\x80\xDF\xEA @\x80jR\x8B\x9BG\x80 @`X@\x80\xDF\xEA @\x80jR\x8B\x9BG\x80 @`X@\x80\xDF\xEA @\x80jR\x8B\x9BG\x80 @`X@\x80\xDF\xEA @\x80jR\x8B\x9BG\x80 @`X@\x80\xDF\xEA @\x80jR\x8B\x9BG\x80 @`X@\x80\xDF\xEA @\x80jR\x8B\x9BG\x80 @`X@\x80\xDF\xEA @\x80jR\x8B\x9BG\x80 @`X@\x80\xDF\xEA @\x80jR\x8B\x9BG\x80 @`X@\x80\xDF\xEA @\x80jR\x8B\x9BG\x80 @`X@\x80\xDF\xEA @\x80jR\x8B\x9BG\x80 @`X@\x80\xDF\xEA @\x80jR\x8B\x9BG\x80 @`X@\x80\xDF\xEA @\x80jR\x8B\x9BG\x80 @`X@\x80\xDF\xEA @\x80jR\x8B\x9BG\x80 @`X@\x80\xDF\xEA @\x80jR\x8B\x9BG\x80 @`X@\x80\xDF\xEA @\x80jR\x8B\x9BG\x80 @`X@\x80\xDF\xEA @\x80jR\x8B\x9BG\x80 @`X@\x80\xDF\xEA @\x80jR\x8B\x9BG\x80 @`X@\x80\xDF\xEA @\x80jR\x8B\x9BG\x80 @`X@\x80\xDF\xEA @\x80jR\x8B\x9BG\x80 @`X@\x80\xDF\xEA @\x80jR\x8B\x9BG\x80 @`X@\x80\xDF\xEA @\x80jR\x8B\x9BG\x80 @`X@\x80\xDF\xEA @\x80jR\x8B\x9BG\x80 @`X@\x80\xDF\xEA @\x80jR\x8B\x9BG\x80 @`X\xE0\xD6K\xCDS-\xBEBIEND\xAEB`\x82
\ No newline at end of file

Added: trunk/LayoutTests/platform/mac/fast/inline/inline-wrap-with-parent-padding-expected.txt (0 => 88175)


--- trunk/LayoutTests/platform/mac/fast/inline/inline-wrap-with-parent-padding-expected.txt	                        (rev 0)
+++ trunk/LayoutTests/platform/mac/fast/inline/inline-wrap-with-parent-padding-expected.txt	2011-06-06 18:32:54 UTC (rev 88175)
@@ -0,0 +1,79 @@
+layer at (0,0) size 800x600
+  RenderView at (0,0) size 800x600
+layer at (0,0) size 800x54
+  RenderBlock {HTML} at (0,0) size 800x54
+    RenderBody {BODY} at (8,8) size 784x38
+      RenderBlock {DIV} at (0,0) size 322x38 [border: (1px solid #FF0000)]
+        RenderInline {SPAN} at (0,0) size 319x38 [border: (1px solid #008000)]
+          RenderText {#text} at (2,1) size 14x18
+            text run at (2,1) width 14: "-- "
+          RenderInline {SPAN} at (0,0) size 319x36
+            RenderText {#text} at (16,1) size 14x18
+              text run at (16,1) width 14: "-- "
+            RenderInline {SPAN} at (0,0) size 319x36
+              RenderText {#text} at (30,1) size 14x18
+                text run at (30,1) width 14: "-- "
+              RenderInline {SPAN} at (0,0) size 319x36
+                RenderText {#text} at (44,1) size 14x18
+                  text run at (44,1) width 14: "-- "
+                RenderInline {SPAN} at (0,0) size 319x36
+                  RenderText {#text} at (58,1) size 14x18
+                    text run at (58,1) width 14: "-- "
+                  RenderInline {SPAN} at (0,0) size 319x36
+                    RenderText {#text} at (72,1) size 14x18
+                      text run at (72,1) width 14: "-- "
+                    RenderInline {SPAN} at (0,0) size 319x36
+                      RenderText {#text} at (86,1) size 14x18
+                        text run at (86,1) width 14: "-- "
+                      RenderInline {SPAN} at (0,0) size 319x36
+                        RenderText {#text} at (100,1) size 14x18
+                          text run at (100,1) width 14: "-- "
+                        RenderInline {SPAN} at (0,0) size 319x36
+                          RenderText {#text} at (114,1) size 14x18
+                            text run at (114,1) width 14: "-- "
+                          RenderInline {SPAN} at (0,0) size 319x36
+                            RenderText {#text} at (128,1) size 14x18
+                              text run at (128,1) width 14: "-- "
+                            RenderInline {SPAN} at (0,0) size 319x36
+                              RenderText {#text} at (142,1) size 14x18
+                                text run at (142,1) width 14: "-- "
+                              RenderInline {SPAN} at (0,0) size 319x36
+                                RenderText {#text} at (156,1) size 14x18
+                                  text run at (156,1) width 14: "-- "
+                                RenderInline {SPAN} at (0,0) size 319x36
+                                  RenderText {#text} at (170,1) size 14x18
+                                    text run at (170,1) width 14: "-- "
+                                  RenderInline {SPAN} at (0,0) size 319x36
+                                    RenderText {#text} at (184,1) size 14x18
+                                      text run at (184,1) width 14: "-- "
+                                    RenderInline {SPAN} at (0,0) size 319x36
+                                      RenderText {#text} at (198,1) size 14x18
+                                        text run at (198,1) width 14: "-- "
+                                      RenderInline {SPAN} at (0,0) size 319x36
+                                        RenderText {#text} at (212,1) size 14x18
+                                          text run at (212,1) width 14: "-- "
+                                        RenderInline {SPAN} at (0,0) size 319x36
+                                          RenderText {#text} at (226,1) size 14x18
+                                            text run at (226,1) width 14: "-- "
+                                          RenderInline {SPAN} at (0,0) size 319x36
+                                            RenderText {#text} at (240,1) size 14x18
+                                              text run at (240,1) width 14: "-- "
+                                            RenderInline {SPAN} at (0,0) size 319x36
+                                              RenderText {#text} at (254,1) size 14x18
+                                                text run at (254,1) width 14: "-- "
+                                              RenderInline {SPAN} at (0,0) size 319x36
+                                                RenderText {#text} at (268,1) size 14x18
+                                                  text run at (268,1) width 14: "-- "
+                                                RenderInline {SPAN} at (0,0) size 319x36
+                                                  RenderText {#text} at (282,1) size 14x18
+                                                    text run at (282,1) width 14: "-- "
+                                                  RenderInline {SPAN} at (0,0) size 319x36
+                                                    RenderText {#text} at (296,1) size 14x18
+                                                      text run at (296,1) width 14: "-- "
+                                                    RenderInline {SPAN} at (0,0) size 319x36
+                                                      RenderText {#text} at (310,1) size 10x18
+                                                        text run at (310,1) width 10: "--"
+                                                      RenderInline {SPAN} at (0,0) size 10x18
+                                                        RenderText {#text} at (1,19) size 10x18
+                                                          text run at (1,19) width 10: "--"
+        RenderText {#text} at (0,0) size 0x0

Modified: trunk/Source/WebCore/ChangeLog (88174 => 88175)


--- trunk/Source/WebCore/ChangeLog	2011-06-06 18:03:05 UTC (rev 88174)
+++ trunk/Source/WebCore/ChangeLog	2011-06-06 18:32:54 UTC (rev 88175)
@@ -1,3 +1,19 @@
+2011-06-06  Yael Aharon  <[email protected]>
+
+        Reviewed by Eric Seidel.
+
+        Inline elements are wrapped prematurely with non-zero border, margin or padding.
+        https://bugs.webkit.org/show_bug.cgi?id=60459
+
+        When an inline element has a right border/margin/padding and it has more than one descendant with no siblings,
+        the width of the right border/margin/padding should be included in line breaking calculation only once,
+        and not for each descendant.
+
+        Test: fast/inline/inline-wrap-with-parent-padding.html
+
+        * rendering/RenderBlockLineLayout.cpp:
+        (WebCore::RenderBlock::LineBreaker::nextLineBreak):
+
 2011-06-06  Shishir Agrawal  <[email protected]>
 
         Reviewed by Tony Gentilcore.

Modified: trunk/Source/WebCore/rendering/RenderBlockLineLayout.cpp (88174 => 88175)


--- trunk/Source/WebCore/rendering/RenderBlockLineLayout.cpp	2011-06-06 18:03:05 UTC (rev 88174)
+++ trunk/Source/WebCore/rendering/RenderBlockLineLayout.cpp	2011-06-06 18:32:54 UTC (rev 88175)
@@ -1843,6 +1843,7 @@
     ASSERT(resolver.position().root() == m_block);
 
     bool appliedStartWidth = resolver.position().m_pos > 0;
+    bool includeEndWidth = true;
     LineMidpointState& lineMidpointState = resolver.midpointState();
 
     LineWidth width(m_block, lineInfo.isFirstLine());
@@ -1887,6 +1888,8 @@
     EWhiteSpace lastWS = currWS;
     while (current.m_obj) {
         RenderObject* next = bidiNext(m_block, current.m_obj);
+        if (next && next->parent() && !next->parent()->isDescendantOf(current.m_obj->parent()))
+            includeEndWidth = true;
 
         currWS = current.m_obj->isReplaced() ? current.m_obj->parent()->style()->whiteSpace() : current.m_obj->style()->whiteSpace();
         lastWS = last->isReplaced() ? last->parent()->style()->whiteSpace() : last->style()->whiteSpace();
@@ -2264,7 +2267,8 @@
 
             // IMPORTANT: current.m_pos is > length here!
             float additionalTmpW = ignoringSpaces ? 0 : textWidth(t, lastSpace, current.m_pos - lastSpace, f, width.currentWidth(), isFixedPitch, collapseWhiteSpace) + lastSpaceWordSpacing;
-            width.addUncommittedWidth(additionalTmpW + inlineLogicalWidth(current.m_obj, !appliedStartWidth, true));
+            width.addUncommittedWidth(additionalTmpW + inlineLogicalWidth(current.m_obj, !appliedStartWidth, includeEndWidth));
+            includeEndWidth = false;
 
             if (!width.fitsOnLine()) {
                 if (canHyphenate)

_______________________________________________
webkit-changes mailing list
[email protected]
http://lists.webkit.org/mailman/listinfo.cgi/webkit-changes