[webkit-changes] [93721] trunk

[commit-queue]

Title: [93721] trunk

Revision

93721

Author

commit-qu...@webkit.org

Date

2011-08-24 12:53:32 -0700 (Wed, 24 Aug 2011)

Log Message

[CSSRegions] RenderRegion is not used if there's another renderer after it
https://bugs.webkit.org/show_bug.cgi?id=66844

Source/WebCore:

There was no call to attachRegion in RenderObjectChildList::insertChildNode, so
the region was not added to the RenderFlowThread.

Patch by Alexandru Chiculita <ach...@adobe.com> on 2011-08-24
Reviewed by David Hyatt.

Test: fast/regions/content-flowed-into-regions-dynamically-inserted.html

* rendering/RenderObjectChildList.cpp:
(WebCore::RenderObjectChildList::insertChildNode):

LayoutTests:

Patch by Alexandru Chiculita <ach...@adobe.com> on 2011-08-24
Reviewed by David Hyatt.

* fast/regions/content-flowed-into-regions-dynamically-inserted-expected.png: Added.
* fast/regions/content-flowed-into-regions-dynamically-inserted-expected.txt: Added.
* fast/regions/content-flowed-into-regions-dynamically-inserted.html: Added.

Modified Paths

• trunk/LayoutTests/ChangeLog
• trunk/Source/WebCore/ChangeLog
• trunk/Source/WebCore/rendering/RenderObjectChildList.cpp

Added Paths

• trunk/LayoutTests/fast/regions/content-flowed-into-regions-dynamically-inserted-expected.png
• trunk/LayoutTests/fast/regions/content-flowed-into-regions-dynamically-inserted-expected.txt
• trunk/LayoutTests/fast/regions/content-flowed-into-regions-dynamically-inserted.html

Diff

Modified: trunk/LayoutTests/ChangeLog (93720 => 93721)

--- trunk/LayoutTests/ChangeLog	2011-08-24 19:34:03 UTC (rev 93720)
+++ trunk/LayoutTests/ChangeLog	2011-08-24 19:53:32 UTC (rev 93721)
@@ -1,3 +1,14 @@
+2011-08-24  Alexandru Chiculita  <ach...@adobe.com>
+
+        [CSSRegions] RenderRegion is not used if there's another renderer after it
+        https://bugs.webkit.org/show_bug.cgi?id=66844
+
+        Reviewed by David Hyatt.
+
+        * fast/regions/content-flowed-into-regions-dynamically-inserted-expected.png: Added.
+        * fast/regions/content-flowed-into-regions-dynamically-inserted-expected.txt: Added.
+        * fast/regions/content-flowed-into-regions-dynamically-inserted.html: Added.
+
 2011-08-23  Peter Kasting  <pkast...@google.com>
 
         [chromium] Organize SLOW section of expectations.

Added: trunk/LayoutTests/fast/regions/content-flowed-into-regions-dynamically-inserted-expected.png (0 => 93721)

--- trunk/LayoutTests/fast/regions/content-flowed-into-regions-dynamically-inserted-expected.png	                        (rev 0)
+++ trunk/LayoutTests/fast/regions/content-flowed-into-regions-dynamically-inserted-expected.png	2011-08-24 19:53:32 UTC (rev 93721)
@@ -0,0 +1,82 @@
+\x89PNG
+
+
+IHDR X\x9Av\x82p)tEXtchecksumb9fb3e768a870229ca4adababd247878\x9F\xD7ޚ\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 \xBC
v\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\xDDA\x8A$\xD4EQS\x9C\xA5\x93\xF9\x93\xF9\xE3,E\xC1\xE6\xCB-\xE7\xB2\xB2\xFB\xAE'\xA2\xD9pCA}\xFD\xF8\xF3\xE7? @\x80
\xF8\x80\xC0\xAF\xF8A\x80
 @\x80\x81\xBF\x88_ @\x80
>& @>F\xED\x83 @\x80
 ~ @\x80
\xF8\x98\x80\xF9\xB5"@\x80
 @@\x80\xF8 @\x80
 @\xE0c\xE4c\xD4>\x88
 @\x80\x81߾#\xF8\xFA\xFA\xFA\xEE\xC4;\x813
\xFF,\xCD\xA5! @\x80\xC0O)\xF0m\x80\xFC\xF5\xAD\xFF\xF7S~\xF7\xFF\xA5\xFE\xF9s\xFCW\xFE\xEDſ\xE3\xF0\xC7\xFF\xFF=\xC5\xCF\xF8_|\xFD\xFE3~+߉
 @\xE0R\xC0_\xC1\xBAԴE\x80
 @\x80\xC0S@\x80<y< @\x80
 p) @.5m @\x80
 \xF0 O\x8F @\x80
\
+\x90KM[ @\x80
<ȓ\xC7#
 @\x80
\x97\xE4R\xD3
 @\x80
O

\xF2\xE4\xF1H\x80
 @\x80\xC0\xA5\x80\xB9ԴE\x80
 @\x80\xC0S@\x80<y< @\x80
 p) @.5m @\x80
 \xF0 O\x8F @\x80
\
+\x90KM[ @\x80
<ȓ\xC7#
 @\x80
\x97\xE4R\xD3
 @\x80
O

\xF2\xE4\xF1H\x80
 @\x80\xC0\xA5\x80\xB9ԴE\x80
 @\x80\xC0S@\x80<y< @\x80
 p) @.5m @\x80
 \xF0 O\x8F @\x80
\
+\x90KM[ @\x80
<ȓ\xC7#
 @\x80
\x97\xE4R\xD3
 @\x80
O

\xF2\xE4\xF1H\x80
 @\x80\xC0\xA5\x80\xB9ԴE\x80
 @\x80\xC0S@\x80<y< @\x80
 p) @.5m @\x80
 \xF0 O\x8F @\x80
\
+\x90KM[ @\x80
<ȓ\xC7#
 @\x80
\x97\xE4R\xD3
 @\x80
O

\xF2\xE4\xF1H\x80
 @\x80\xC0\xA5\x80\xB9ԴE\x80
 @\x80\xC0S@\x80<y< @\x80
 p) @.5m @\x80
 \xF0 O\x8F @\x80
\
+\x90KM[ @\x80
<ȓ\xC7#
 @\x80
\x97\xE4R\xD3
 @\x80
O

\xF2\xE4\xF1H\x80
 @\x80\xC0\xA5\x80\xB9ԴE\x80
 @\x80\xC0S@\x80<y< @\x80
 p) @.5m @\x80
 \xF0 O\x8F @\x80
\
+\x90KM[ @\x80
<ȓ\xC7#
 @\x80
\x97\xE4R\xD3
 @\x80
O

\xF2\xE4\xF1H\x80
 @\x80\xC0\xA5\x80\xB9ԴE\x80
 @\x80\xC0S\xE0\xEBǟ?\xAF\x8B\xAF\xAF\xAF׳7\xA7\xDF\xFC:\x9E~\x961 @\x80\x81\xCF\xFC\xF6\xDDG\xFA\xC2 @\x80
 P\xFC\xAC*\xE5\x8E
 @\x80\x81Y@\x80̄ @\x80
\xA8\xA4J\xB9#@\x80
 @` 3\xA1
 @\x80
\xAA\x80\xA9R\xEE @\x80
\x98\xC8Lh\x80
 @\x80\x81* @\xAA\x94; @\x80
f

2 @\x80
 @\xA0
+\x90*\xE5\x8E
 @\x80\x81Y@\x80̄ @\x80
\xA8\xA4J\xB9#@\x80
 @` 3\xA1
 @\x80
\xAA\x80\xA9R\xEE @\x80
\x98\xC8Lh\x80
 @\x80\x81* @\xAA\x94; @\x80
f

2 @\x80
 @\xA0
+\x90*\xE5\x8E
 @\x80\x81Y@\x80̄ @\x80
\xA8\xA4J\xB9#@\x80
 @` 3\xA1
 @\x80
\xAA\x80\xA9R\xEE @\x80
\x98\xC8Lh\x80
 @\x80\x81* @\xAA\x94; @\x80
f

2 @\x80
 @\xA0
+\x90*\xE5\x8E
 @\x80\x81Y@\x80̄ @\x80
\xA8\xA4J\xB9#@\x80
 @` 3\xA1
 @\x80
\xAA\x80\xA9R\xEE @\x80
\x98\xC8Lh\x80
 @\x80\x81* @\xAA\x94; @\x80
f

2 @\x80
 @\xA0
+\x90*\xE5\x8E
 @\x80\x81Y@\x80̄ @\x80
\xA8\xA4J\xB9#@\x80
 @` 3\xA1
 @\x80
\xAA\x80\xA9R\xEE @\x80
\x98\xC8Lh\x80
 @\x80\x81* @\xAA\x94; @\x80
f

2 @\x80
 @\xA0
+\x90*\xE5\x8E
 @\x80\x81Y@\x80̄ @\x80
\xA8\xA4J\xB9#@\x80
 @` 3\xA1
 @\x80
\xAA\x80\xA9R\xEE @\x80
\x98\xC8Lh\x80
 @\x80\x81* @\xAA\x94; @\x80
f

2 @\x80
 @\xA0
+\x90*\xE5\x8E
 @\x80\x81Y@\x80̄ @\x80
\xA8\xA4J\xB9#@\x80
 @` 3\xA1
 @\x80
\xAA\x80\xA9R\xEE @\x80
\x98\xC8Lh\x80
 @\x80\x81* @\xAA\x94; @\x80
f

2 @\x80
 @\xA0
+\x90*\xE5\x8E
 @\x80\x81Y@\x80̄ @\x80
\xA8\xA4J\xB9#@\x80
 @` 3\xA1
 @\x80
\xAA\x80\xA9R\xEE @\x80
\x98\xC8Lh\x80
 @\x80\x81* @\xAA\x94; @\x80
f

2 @\x80
 @\xA0
+\x90*\xE5\x8E
 @\x80\x81Y@\x80̄ @\x80
\xA8\xA4J\xB9#@\x80
 @` 3\xA1
 @\x80
\xAA\x80\xA9R\xEE @\x80
\x98\xC8Lh\x80
 @\x80\x81* @\xAA\x94; @\x80
f

2 @\x80
 @\xA0
+\x90*\xE5\x8E
 @\x80\x81Y@\x80̄ @\x80
\xA8\xA4J\xB9#@\x80
 @` 3\xA1
 @\x80
\xAA\x80\xA9R\xEE @\x80
\x98\xC8Lh\x80
 @\x80\x81* @\xAA\x94; @\x80
f

2 @\x80
 @\xA0
+\x90*\xE5\x8E
 @\x80\x81Y@\x80̄ @\x80
\xA8\xA4J\xB9#@\x80
 @` 3\xA1
 @\x80
\xAA\x80\xA9R\xEE @\x80
\x98\xC8Lh\x80
 @\x80\x81* @\xAA\x94; @\x80
f

2 @\x80
 @\xA0
+\x90*\xE5\x8E
 @\x80\x81Y@\x80̄ @\x80
\xA8\xA4J\xB9#@\x80
 @` 3\xA1
 @\x80
\xAA\x80\xA9R\xEE @\x80
\x98\xC8Lh\x80
 @\x80\x81* @\xAA\x94; @\x80
f

2 @\x80
 @\xA0
+\x90*\xE5\x8E
 @\x80\x81Y@\x80̄ @\x80
\xA8\xA4J\xB9#@\x80
 @` 3\xA1
 @\x80
\xAA\x80\xA9R\xEE @\x80
\x98\xC8Lh\x80
 @\x80\x81* @\xAA\x94; @\x80
f

2 @\x80
 @\xA0
+\x90*\xE5\x8E
 @\x80\x81Y@\x80̄ @\x80
\xA8\xA4J\xB9#@\x80
 @` 3\xA1
 @\x80
\xAA\x80\xA9R\xEE @\x80
\x98\xC8Lh\x80
 @\x80\x81* @\xAA\x94; @\x80
f

2 @\x80
 @\xA0
+\x90*\xE5\x8E
 @\x80\x81Y@\x80̄ @\x80
\xA8\xA4J\xB9#@\x80
 @` 3\xA1
 @\x80
\xAA\x80\xA9R\xEE @\x80
\x98\xC8Lh\x80
 @\x80\x81* @\xAA\x94; @\x80
f

2 @\x80
 @\xA0
+\x90*\xE5\x8E
 @\x80\x81Y@\x80̄ @\x80
\xA8\xA4J\xB9#@\x80
 @` 3\xA1
 @\x80
\xAA\x80\xA9R\xEE @\x80
\x98\xC8Lh\x80
 @\x80\x81* @\xAA\x94; @\x80
f

2 @\x80
 @\xA0
+\x90*\xE5\x8E
 @\x80\x81Y@\x80̄ @\x80
\xA8\xA4J\xB9#@\x80
 @` 3\xA1
 @\x80
\xAA\x80\xA9R\xEE @\x80
\x98\xC8Lh\x80
 @\x80\x81* @\xAA\x94; @\x80
f

2 @\x80
 @\xA0
+\x90*\xE5\x8E
 @\x80\x81Y@\x80̄ @\x80
\xA8\xA4J\xB9#@\x80
 @` 3\xA1
 @\x80
\xAA\x80\xA9R\xEE @\x80
\x98\xC8Lh\x80
 @\x80\x81* @\xAA\x94; @\x80
f

2 @\x80
 @\xA0
+\x90*\xE5\x8E
 @\x80\x81Y@\x80̄ @\x80
\xA8\xA4J\xB9#@\x80
 @` 3\xA1
 @\x80
\xAA\x80\xA9R\xEE @\x80
\x98\xC8Lh\x80
 @\x80\x81* @\xAA\x94; @\x80
f

2 @\x80
 @\xA0
+\x90*\xE5\x8E
 @\x80\x81Y@\x80̄ @\x80
\xA8\xA4J\xB9#@\x80
 @` 3\xA1
 @\x80
\xAA\x80\xA9R\xEE @\x80
\x98\xC8Lh\x80
 @\x80\x81* @\xAA\x94; @\x80
f

2 @\x80
 @\xA0
+\x90*\xE5\x8E
 @\x80\x81Y@\x80̄ @\x80
\xA8\xA4J\xB9#@\x80
 @` 3\xA1
 @\x80
\xAA\x80\xA9R\xEE @\x80
\x98\xC8Lh\x80
 @\x80\x81* @\xAA\x94; @\x80
f

2 @\x80
 @\xA0
+\x90*\xE5\x8E
 @\x80\x81Y@\x80̄ @\x80
\xA8\xA4J\xB9#@\x80
 @` 3\xA1
 @\x80
\xAA\x80\xA9R\xEE @\x80
\x98\xC8Lh\x80
 @\x80\x81* @\xAA\x94; @\x80
f

2 @\x80
 @\xA0
+\x90*\xE5\x8E
 @\x80\x81Y@\x80̄ @\x80
\xA8\xA4J\xB9#@\x80
 @` 3\xA1
 @\x80
\xAA\x80\xA9R\xEE @\x80
\x98\xC8Lh\x80
 @\x80\x81* @\xAA\x94; @\x80
f

2 @\x80
 @\xA0
+\x90*\xE5\x8E
 @\x80\x81Y@\x80̄ @\x80
\xA8\xA4J\xB9#@\x80
 @` 3\xA1
 @\x80
\xAA\x80\xA9R\xEE @\x80
\x98\xC8Lh\x80
 @\x80\x81* @\xAA\x94; @\x80
f

2 @\x80
 @\xA0
+\x90*\xE5\x8E
 @\x80\x81Y@\x80̄ @\x80
\xA8\xA4J\xB9#@\x80
 @` 3\xA1
 @\x80
\xAA\x80\xA9R\xEE @\x80
\x98\xC8Lh\x80
 @\x80\x81* @\xAA\x94; @\x80
f

2 @\x80
 @\xA0
+\x90*\xE5\x8E
 @\x80\x81Y@\x80̄ @\x80
\xA8\xA4J\xB9#@\x80
 @` 3\xA1
 @\x80
\xAA\x80\xA9R\xEE @\x80
\x98\xC8Lh\x80
 @\x80\x81* @\xAA\x94; @\x80
f

2 @\x80
 @\xA0
+\x90*\xE5\x8E
 @\x80\x81Y@\x80̄ @\x80
\xA8\xA4J\xB9#@\x80
 @` 3\xA1
 @\x80
\xAA\x80\xA9R\xEE @\x80
\x98\xC8Lh\x80
 @\x80\x81* @\xAA\x94; @\x80
f

2 @\x80
 @\xA0
+\x90*\xE5\x8E
 @\x80\x81Y@\x80̄ @\x80
\xA8\xA4J\xB9#@\x80
 @` 3\xA1
 @\x80
\xAA\x80\xA9R\xEE @\x80
\x98\xC8Lh\x80
 @\x80\x81* @\xAA\x94; @\x80
f

2 @\x80
 @\xA0
+\x90*\xE5\x8E
 @\x80\x81Y@\x80̄ @\x80
\xA8\xA4J\xB9#@\x80
 @` 3\xA1
 @\x80
\xAA\x80\xA9R\xEE @\x80
\x98\xC8Lh\x80
 @\x80\x81* @\xAA\x94; @\x80
f

2 @\x80
 @\xA0
+\x90*\xE5\x8E
 @\x80\x81Y@\x80̄ @\x80
\xA8\xA4J\xB9#@\x80
 @` 3\xA1
 @\x80
\xAA\x80\xA9R\xEE @\x80
\x98\xC8Lh\x80
 @\x80\x81* @\xAA\x94; @\x80
f

2 @\x80
 @\xA0
+\x90*\xE5\x8E
 @\x80\x81Y@\x80̄ @\x80
\xA8\xA4J\xB9#@\x80
 @` 3\xA1
 @\x80
\xAA\x80\xA9R\xEE @\x80
\x98\xC8Lh\x80
 @\x80\x81* @\xAA\x94; @\x80
f

2 @\x80
 @\xA0
+\x90*\xE5\x8E
 @\x80\x81Y@\x80̄ @\x80
\xA8\xA4J\xB9#@\x80
 @` 3\xA1
 @\x80
\xAA\x80\xA9R\xEE @\x80
\x98\xC8Lh\x80
 @\x80\x81* @\xAA\x94; @\x80
f

2 @\x80
 @\xA0
+\x90*\xE5\x8E
 @\x80\x81Y@\x80̄ @\x80
\xA8\xA4J\xB9#@\x80
 @` 3\xA1
 @\x80
\xAA\x80\xA9R\xEE @\x80
\x98\xC8Lh\x80
 @\x80\x81* @\xAA\x94; @\x80
f

2 @\x80
 @\xA0
+\x90*\xE5\x8E
 @\x80\x81Y@\x80̄ @\x80
\xA8\xA4J\xB9#@\x80
 @` 3\xA1
 @\x80
\xAA\x80\xA9R\xEE @\x80
\x98\xC8Lh\x80
 @\x80\x81* @\xAA\x94; @\x80
f

2 @\x80
 @\xA0
+\x90*\xE5\x8E
 @\x80\x81Y@\x80̄ @\x80
\xA8\xA4J\xB9#@\x80
 @` 3\xA1
 @\x80
\xAA\x80\xA9R\xEE @\x80
\x98\xC8Lh\x80
 @\x80\x81* @\xAA\x94; @\x80
f

2 @\x80
 @\xA0
+\x90*\xE5\x8E
 @\x80\x81Y@\x80̄ @\x80
\xA8\xA4J\xB9#@\x80
 @` 3\xA1
 @\x80
\xAA\x80\xA9R\xEE @\x80
\x98\xC8Lh\x80
 @\x80\x81* @\xAA\x94; @\x80
f

2 @\x80
 @\xA0
+\x90*\xE5\x8E
 @\x80\x81Y@\x80̄ @\x80
\xA8\xA4J\xB9#@\x80
 @` 3\xA1
 @\x80
\xAA\x80\xA9R\xEE @\x80
\x98\xC8Lh\x80
 @\x80\x81* @\xAA\x94; @\x80
f

2 @\x80
 @\xA0
+\x90*\xE5\x8E
 @\x80\x81Y@\x80̄ @\x80
\xA8\xA4J\xB9#@\x80
 @` 3\xA1
 @\x80
\xAA\x80\xA9R\xEE @\x80
\x98\xC8Lh\x80
 @\x80\x81* @\xAA\x94; @\x80
f

2 @\x80
 @\xA0
+\x90*\xE5\x8E
 @\x80\x81Y@\x80̄ @\x80
\xA8\xA4J\xB9#@\x80
 @` 3\xA1
 @\x80
\xAA\x80\xA9R\xEE @\x80
\x98\xC8Lh\x80
 @\x80\x81* @\xAA\x94; @\x80
f

2 @\x80
 @\xA0
+\x90*\xE5\x8E
 @\x80\x81Y@\x80̄ @\x80
\xA8\xA4J\xB9#@\x80
 @` 3\xA1
 @\x80
\xAA\x80\xA9R\xEE @\x80
\x98\xC8Lh\x80
 @\x80\x81* @\xAA\x94; @\x80
f

2 @\x80
 @\xA0
+\x90*\xE5\x8E
 @\x80\x81Y@\x80̄ @\x80
\xA8\xA4J\xB9#@\x80
 @` 3\xA1
 @\x80
\xAA\x80\xA9R\xEE @\x80
\x98\xC8Lh\x80
 @\x80\x81* @\xAA\x94; @\x80
f

2 @\x80
 @\xA0
+\x90*\xE5\x8E
 @\x80\x81Y@\x80̄ @\x80
\xA8\xA4J\xB9#@\x80
 @` 3\xA1
 @\x80
\xAA\x80\xA9R\xEE @\x80
\x98\xC8Lh\x80
 @\x80\x81* @\xAA\x94; @\x80
f

2 @\x80
 @\xA0
+\x90*\xE5\x8E
 @\x80\x81Y@\x80̄ @\x80
\xA8\xA4J\xB9#@\x80
 @` 3\xA1
 @\x80
\xAA\x80\xA9R\xEE @\x80
\x98\xC8Lh\x80
 @\x80\x81* @\xAA\x94; @\x80
f

2 @\x80
 @\xA0
+\x90*\xE5\x8E
 @\x80\x81Y@\x80̄ @\x80
\xA8\xA4J\xB9#@\x80
 @` 3\xA1
 @\x80
\xAA\x80\xA9R\xEE @\x80
\x98\xC8Lh\x80
 @\x80\x81* @\xAA\x94; @\x80
f

2 @\x80
 @\xA0
+\x90*\xE5\x8E
 @\x80\x81Y@\x80̄ @\x80
\xA8\xA4J\xB9#@\x80
 @` 3\xA1
 @\x80
\xAA\x80\xA9R\xEE @\x80
\x98\xC8Lh\x80
 @\x80\x81* @\xAA\x94; @\x80
f

2 @\x80
 @\xA0
+\x90*\xE5\x8E
 @\x80\x81Y@\x80̄ @\x80
\xA8\xA4J\xB9#@\x80
 @` 3\xA1
 @\x80
\xAA\x80\xA9R\xEE @\x80
\x98\xC8Lh\x80
 @\x80\x81* @\xAA\x94; @\x80
f

2 @\x80
 @\xA0
+\x90*\xE5\x8E
 @\x80\x81Y@\x80̄ @\x80
\xA8\xA4J\xB9#@\x80
 @` 3\xA1
 @\x80
\xAA\x80\xA9R\xEE @\x80
\x98\xC8Lh\x80
 @\x80\x81* @\xAA\x94; @\x80
f

2 @\x80
 @\xA0
+\x90*\xE5\x8E
 @\x80\x81Y@\x80̄ @\x80
\xA8\xA4J\xB9#@\x80
 @` 3\xA1
 @\x80
\xAA\x80\xA9R\xEE @\x80
\x98\xC8Lh\x80
 @\x80\x81* @\xAA\x94; @\x80
f

2 @\x80
 @\xA0
+\x90*\xE5\x8E
 @\x80\x81Y@\x80̄ @\x80
\xA8\xA4J\xB9#@\x80
 @` 3\xA1
 @\x80
\xAA\x80\xA9R\xEE @\x80
\x98\xC8Lh\x80
 @\x80\x81* @\xAA\x94; @\x80
f

2 @\x80
 @\xA0
+\x90*\xE5\x8E
 @\x80\x81Y@\x80̄ @\x80
\xA8\xA4J\xB9#@\x80
 @` 3\xA1
 @\x80
\xAA\x80\xA9R\xEE @\x80
\x98\xC8Lh\x80
 @\x80\x81* @\xAA\x94; @\x80
f

2 @\x80
 @\xA0
+\x90*\xE5\x8E
 @\x80\x81Y@\x80̄ @\x80
\xA8\xA4J\xB9#@\x80\xAEM;	JIDAT
 @` 3\xA1
 @\x80
\xAA\x80\xA9R\xEE @\x80
\x98\xC8Lh\x80
 @\x80\x81* @\xAA\x94; @\x80
f

2 @\x80
 @\xA0
+\x90*\xE5\x8E
 @\x80\x81Y@\x80̄ @\x80
\xA8\xA4J\xB9#@\x80
 @` 3\xA1
 @\x80
\xAA\x80\xA9R\xEE @\x80
\x98\xC8Lh\x80
 @\x80\x81* @\xAA\x94; @\x80
f

2 @\x80
 @\xA0
+\x90*\xE5\x8E
 @\x80\x81Y@\x80̄ @\x80
\xA8\xA4J\xB9#@\x80
 @` 3\xA1
 @\x80
\xAA\x80\xA9R\xEE @\x80
\x98\xC8Lh\x80
 @\x80\x81* @\xAA\x94; @\x80
f

2 @\x80
 @\xA0
+\x90*\xE5\x8E
 @\x80\x81Y@\x80̄ @\x80
\xA8\xA4J\xB9#@\x80
 @` 3\xA1
 @\x80
\xAA\x80\xA9R\xEE @\x80
\x98\xC8Lh\x80
 @\x80\x81* @\xAA\x94; @\x80
f

2 @\x80
 @\xA0
+\x90*\xE5\x8E
 @\x80\x81Y@\x80̄ @\x80
\xA8\xA4J\xB9#@\x80
 @` 3\xA1
 @\x80
\xAA\x80\xA9R\xEE @\x80
\x98\xC8Lh\x80
 @\x80\x81* @\xAA\x94; @\x80
f

2 @\x80
 @\xA0
+\x90*\xE5\x8E
 @\x80\x81Y@\x80̄ @\x80
\xA8\xA4J\xB9#@\x80
 @` 3\xA1
 @\x80
\xAA\x80\xA9R\xEE @\x80
\x98\xC8Lh\x80
 @\x80\x81* @\xAA\x94; @\x80
f

2 @\x80
 @\xA0
+\x90*\xE5\x8E
 @\x80\x81Y@\x80̄ @\x80
\xA8\xA4J\xB9#@\x80
 @` 3\xA1
 @\x80
\xAA\x80\xA9R\xEE @\x80
\x98\xC8Lh\x80
 @\x80\x81* @\xAA\x94; @\x80
f

2 @\x80
 @\xA0
+\x90*\xE5\x8E
 @\x80\x81Y@\x80̄ @\x80
\xA8\xA4J\xB9#@\x80
 @` 3\xA1
 @\x80
\xAA\x80\xA9R\xEE @\x80
\x98\xC8Lh\x80
 @\x80\x81* @\xAA\x94; @\x80
f

2 @\x80
 @\xA0
+\x90*\xE5\x8E
 @\x80\x81Y@\x80̄ @\x80
\xA8\xA4J\xB9#@\x80
 @` 3\xA1
 @\x80
\xAA\x80\xA9R\xEE @\x80
\x98\xC8Lh\x80
 @\x80\x81* @\xAA\x94; @\x80
f

2 @\x80
 @\xA0
+\x90*\xE5\x8E
 @\x80\x81Y@\x80̄ @\x80
\xA8\xA4J\xB9#@\x80
 @` 3\xA1
 @\x80
\xAA\x80\xA9R\xEE @\x80
\x98\xC8Lh\x80
 @\x80\x81* @\xAA\x94; @\x80
f

2 @\x80
 @\xA0
+\x90*\xE5\x8E
 @\x80\x81Y@\x80̄ @\x80
\xA8\xA4J\xB9#@\x80
 @` 3\xA1
 @\x80
\xAA\x80\xA9R\xEE @\x80
\x98\xC8Lh\x80
 @\x80\x81* @\xAA\x94; @\x80
f

2 @\x80
 @\xA0
+\x90*\xE5\x8E
 @\x80\x81Y@\x80̄ @\x80
\xA8\xA4J\xB9#@\x80
 @` 3\xA1
 @\x80
\xAA\x80\xA9R\xEE @\x80
\x98\xC8Lh\x80
 @\x80\x81* @\xAA\x94; @\x80
f

2 @\x80
 @\xA0
+\x90*\xE5\x8E
 @\x80\x81Y@\x80̄ @\x80
\xA8\xA4J\xB9#@\x80
 @` 3\xA1
 @\x80
\xAA\x80\xA9R\xEE @\x80
\x98\xC8Lh\x80
 @\x80\x81* @\xAA\x94; @\x80
f

2 @\x80
 @\xA0
+\x90*\xE5\x8E
 @\x80\x81Y@\x80̄ @\x80
\xA8\xA4J\xB9#@\x80
 @` 3\xA1
 @\x80
\xAA\x80\xA9R\xEE @\x80
\x98\xC8Lh\x80
 @\x80\x81* @\xAA\x94; @\x80
f

2 @\x80
 @\xA0
+\x90*\xE5\x8E
 @\x80\x81Y@\x80̄ @\x80
\xA8\xA4J\xB9#@\x80
 @` 3\xA1
 @\x80
\xAA\x80\xA9R\xEE @\x80
\x98\xC8Lh\x80
 @\x80\x81* @\xAA\x94; @\x80
f

2 @\x80
 @\xA0
+\x90*\xE5\x8E
 @\x80\x81Y@\x80̄ @\x80
\xA8\xA4J\xB9#@\x80
 @` 3\xA1
 @\x80
\xAA\x80\xA9R\xEE @\x80
\x98\xC8Lh\x80
 @\x80\x81* @\xAA\x94; @\x80
f

2 @\x80
 @\xA0
+\x90*\xE5\x8E
 @\x80\x81Y@\x80̄ @\x80
\xA8\xA4J\xB9#@\x80
 @` 3\xA1
 @\x80
\xAA\x80\xA9R\xEE @\x80
\x98\xC8Lh\x80
 @\x80\x81* @\xAA\x94; @\x80
f

2 @\x80
 @\xA0
+\x90*\xE5\x8E
 @\x80\x81Y@\x80̄ @\x80
\xA8\xA4J\xB9#@\x80
 @`\xF8\xF4F\xAA\xBF\xF80AIEND\xAEB`\x82
\ No newline at end of file

Added: trunk/LayoutTests/fast/regions/content-flowed-into-regions-dynamically-inserted-expected.txt (0 => 93721)

--- trunk/LayoutTests/fast/regions/content-flowed-into-regions-dynamically-inserted-expected.txt	                        (rev 0)
+++ trunk/LayoutTests/fast/regions/content-flowed-into-regions-dynamically-inserted-expected.txt	2011-08-24 19:53:32 UTC (rev 93721)
@@ -0,0 +1,20 @@
+layer at (0,0) size 800x600
+  RenderView at (0,0) size 800x600
+layer at (0,0) size 800x60
+  RenderBlock {HTML} at (0,0) size 800x60
+    RenderBody {BODY} at (8,8) size 784x0
+      RenderRegion {DIV} at (0,0) size 52x52 [border: (1px solid #000000)]
+      RenderRegion {DIV} at (52,0) size 52x52 [border: (1px solid #000000)]
+      RenderRegion {DIV} at (104,0) size 52x52 [border: (1px solid #000000)]
+Flow Threads
+  Thread with flow-name 'article'
+    layer at (0,0) size 50x150
+      RenderFlowThread at (0,0) size 50x150
+        RenderBlock {DIV} at (0,0) size 50x150
+          RenderBlock {DIV} at (0,0) size 50x50 [bgcolor=#008000]
+          RenderBlock {DIV} at (0,50) size 50x50 [bgcolor=#0000FF]
+          RenderBlock {DIV} at (0,100) size 50x50 [bgcolor=#FFA500]
+  Regions for flow 'article'
+    RenderRegion {DIV} with index 0
+    RenderRegion {DIV} #region with index 0
+    RenderRegion {DIV} with index 0

Added: trunk/LayoutTests/fast/regions/content-flowed-into-regions-dynamically-inserted.html (0 => 93721)

--- trunk/LayoutTests/fast/regions/content-flowed-into-regions-dynamically-inserted.html	                        (rev 0)
+++ trunk/LayoutTests/fast/regions/content-flowed-into-regions-dynamically-inserted.html	2011-08-24 19:53:32 UTC (rev 93721)
@@ -0,0 +1,29 @@
+<!doctype html>
+<html>
+    <!--
+    Test that the content is properly flowed into dynamically added regions.
+    On success you should see that the colored boxes are displayed inside the 3 rectangles with black border.
+    The green box should be displayed in left region.
+    The blue box should be displayed in middle region.
+    The orange box should be displayed in right region.
+    -->
+    <head>
+        <link rel="stylesheet" href=""
+    </head>
+    <body>
+        <div class="flowArticle">
+            <div class="greenBox"></div>
+            <div class="blueBox"></div>
+            <div class="orangeBox"></div>
+        </div>
+        <div class="regionArticle borderBox"></div>
+        <div id="region"></div>
+        <div class="regionArticle borderBox"></div>
+        <script>
+            var region = document.getElementById("region");
+            // The next line will create a new RenderRegion and insert it
+            // into the flow thread.
+            region.className = "regionArticle borderBox";
+        </script>
+    </body>
+</html>

Modified: trunk/Source/WebCore/ChangeLog (93720 => 93721)

--- trunk/Source/WebCore/ChangeLog	2011-08-24 19:34:03 UTC (rev 93720)
+++ trunk/Source/WebCore/ChangeLog	2011-08-24 19:53:32 UTC (rev 93721)
@@ -1,3 +1,18 @@
+2011-08-24  Alexandru Chiculita  <ach...@adobe.com>
+
+        [CSSRegions] RenderRegion is not used if there's another renderer after it
+        https://bugs.webkit.org/show_bug.cgi?id=66844
+
+        There was no call to attachRegion in RenderObjectChildList::insertChildNode, so
+        the region was not added to the RenderFlowThread.
+
+        Reviewed by David Hyatt.
+
+        Test: fast/regions/content-flowed-into-regions-dynamically-inserted.html
+
+        * rendering/RenderObjectChildList.cpp:
+        (WebCore::RenderObjectChildList::insertChildNode):
+
 2011-08-24  Sam Weinig  <s...@webkit.org>
 
         JSHTMLImageElement (and associated Node) is abandoned when image load is canceled

Modified: trunk/Source/WebCore/rendering/RenderObjectChildList.cpp (93720 => 93721)

--- trunk/Source/WebCore/rendering/RenderObjectChildList.cpp	2011-08-24 19:34:03 UTC (rev 93720)
+++ trunk/Source/WebCore/rendering/RenderObjectChildList.cpp	2011-08-24 19:53:32 UTC (rev 93721)
@@ -259,7 +259,10 @@
 
         if (!child->isFloating() && owner->childrenInline())
             owner->dirtyLinesFromChangedChild(child);
-        
+
+        if (child->isRenderRegion())
+            toRenderRegion(child)->attachRegion();
+
         if (RenderFlowThread* containerFlowThread = renderFlowThreadContainer(owner))
             containerFlowThread->addFlowChild(child, beforeChild);
     }

_______________________________________________
webkit-changes mailing list
webkit-changes@lists.webkit.org
http://lists.webkit.org/mailman/listinfo.cgi/webkit-changes