Bug 188846

Summary: [LFC][Floating] Move files to a dedicated directory.
Product: WebKit Reporter: alan <zalan>
Component: Layout and RenderingAssignee: alan <zalan>
Status: RESOLVED FIXED    
Severity: Normal CC: bfulgham, commit-queue, koivisto, simon.fraser, webkit-bug-importer, zalan
Priority: P2 Keywords: InRadar
Version: WebKit Nightly Build   
Hardware: Unspecified   
OS: Unspecified   
Attachments:
Description Flags
Patch
none
Patch
none
Patch
none
Patch none

alan
Reported 2018-08-22 07:42:51 PDT
->floatformatting
Attachments
Patch (74.56 KB, patch)
2018-08-22 07:46 PDT, alan 		no flags
DetailsFormatted DiffDiff
Patch (75.53 KB, patch)
2018-08-22 07:54 PDT, alan 		no flags
DetailsFormatted DiffDiff
Patch (74.08 KB, patch)
2018-08-22 09:07 PDT, alan 		no flags
DetailsFormatted DiffDiff
Patch (74.07 KB, patch)
2018-08-22 09:30 PDT, alan 		no flags
DetailsFormatted DiffDiff
Show Obsolete (3) View All Add attachment proposed patch, testcase, etc.
alan
Comment 1 2018-08-22 07:46:19 PDT
Created attachment 347790 [details] Patch
alan
Comment 2 2018-08-22 07:54:49 PDT
Created attachment 347791 [details] Patch
Simon Fraser (smfr)
Comment 3 2018-08-22 08:27:01 PDT
Comment on attachment 347791 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=347791&action=review > Source/WebCore/ChangeLog:8 > + layout/Float* -> layout/floatformatting floatformatting is a bit of a mouthful. Maybe layout/floats? I see you have blockformatting now, so maybe that should be layout/block too.
alan
Comment 4 2018-08-22 09:07:48 PDT
Created attachment 347801 [details] Patch
alan
Comment 5 2018-08-22 09:30:26 PDT
Created attachment 347806 [details] Patch
WebKit Commit Bot
Comment 6 2018-08-22 10:10:41 PDT
Comment on attachment 347806 [details] Patch Clearing flags on attachment: 347806 Committed r235178: <https://trac.webkit.org/changeset/235178>
WebKit Commit Bot
Comment 7 2018-08-22 10:10:42 PDT
All reviewed patches have been landed. Closing bug.
Radar WebKit Bug Importer
Comment 8 2018-08-22 10:13:07 PDT
<rdar://problem/43609708>
Note You need to log in before you can comment on or make changes to this bug.