Bug 121843

Summary: [CSS Regions] Propagating margins fails when there's a <p> inside the flowed <div>
Product: WebKit Reporter: Mihai Maerean <mmaerean>
Component: WebCore Misc.Assignee: Nobody <webkit-unassigned>
Status: RESOLVED INVALID    
Severity: Normal CC: WebkitBugTracker
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: Unspecified   
OS: Unspecified   
Bug Depends on:    
Bug Blocks: 57312    
Attachments:
Description Flags
patch to delete the expected png(s) and to add the -expected.html file
none
test with regions
none
html without regions none

Mihai Maerean
Reported 2013-09-24 09:22:41 PDT
Created attachment 212474 [details] patch to delete the expected png(s) and to add the -expected.html file When there's a <p> inside <div style="flow-into:flow">, the margins of the paragraph should collapse with the parent div but they don't. The LayoutTests/fast/regions/flow-content-basic.html test can't be turned into a ref test. The current expected png results are not correct. This bug blocks bug #120457.
Attachments
patch to delete the expected png(s) and to add the -expected.html file (75.95 KB, patch)
2013-09-24 09:22 PDT, Mihai Maerean 		no flags
DetailsFormatted DiffDiff
test with regions (1.27 KB, text/html)
2013-09-25 02:02 PDT, Mihai Maerean 		no flags
Details
html without regions (1.04 KB, text/html)
2013-09-25 02:03 PDT, Mihai Maerean 		no flags
Details
View All Add attachment proposed patch, testcase, etc.
Mihai Maerean
Comment 1 2013-09-25 02:02:37 PDT
Created attachment 212542 [details] test with regions
Mihai Maerean
Comment 2 2013-09-25 02:03:50 PDT
Created attachment 212543 [details] html without regions
Mihai Maerean
Comment 3 2013-10-01 04:51:25 PDT
This bug is invalid because: 1. In CSS, the adjoining margins of two or more boxes (which might or might not be siblings) can combine to form a single margin. Two margins are adjoining if and only if both belong to in-flow block-level boxes that participate in the same block formatting context ( http://www.w3.org/TR/CSS2/box.html#collapsing-margins ) 2. CSS Regions establish a new block formatting Context. ( http://dev.w3.org/csswg/css-regions/ )
Note You need to log in before you can comment on or make changes to this bug.