RESOLVED FIXED 200612
Factor complex line layout path out from RenderBlockFlow 
https://bugs.webkit.org/show_bug.cgi?id=200612
Summary Factor complex line layout path out from RenderBlockFlow
Antti Koivisto
Reported 2019-08-10 08:46:39 PDT
Move the code in RenderBlockLineLayout.cpp into a class of its own.
Attachments
patch (346.87 KB, patch)
2019-08-10 09:12 PDT, Antti Koivisto 		no flags
DetailsFormatted DiffDiff
patch (351.72 KB, patch)
2019-08-10 10:20 PDT, Antti Koivisto 		zalan: review+
DetailsFormatted DiffDiff
patch (352.19 KB, patch)
2019-08-11 00:37 PDT, Antti Koivisto 		no flags
DetailsFormatted DiffDiff
Show Obsolete (2) View All Add attachment proposed patch, testcase, etc.
Antti Koivisto
Comment 1 2019-08-10 09:12:44 PDT
Created attachment 376012 [details] patch
EWS Watchlist
Comment 2 2019-08-10 09:14:01 PDT
Attachment 376012 [details] did not pass style-queue: ERROR: Source/WebCore/rendering/ComplexLineLayout.cpp:219: Should have only a single space after a punctuation in a comment. [whitespace/comments] [5] ERROR: Source/WebCore/rendering/ComplexLineLayout.cpp:221: Should have only a single space after a punctuation in a comment. [whitespace/comments] [5] ERROR: Source/WebCore/rendering/ComplexLineLayout.cpp:227: Should have only a single space after a punctuation in a comment. [whitespace/comments] [5] ERROR: Source/WebCore/rendering/ComplexLineLayout.cpp:336: Should have only a single space after a punctuation in a comment. [whitespace/comments] [5] ERROR: Source/WebCore/rendering/ComplexLineLayout.cpp:346: Should have only a single space after a punctuation in a comment. [whitespace/comments] [5] ERROR: Source/WebCore/rendering/ComplexLineLayout.cpp:878: Should have only a single space after a punctuation in a comment. [whitespace/comments] [5] ERROR: Source/WebCore/rendering/ComplexLineLayout.cpp:878: Weird number of spaces at line-start. Are you using a 4-space indent? [whitespace/indent] [3] ERROR: Source/WebCore/rendering/ComplexLineLayout.cpp:879: Weird number of spaces at line-start. Are you using a 4-space indent? [whitespace/indent] [3] ERROR: Source/WebCore/rendering/ComplexLineLayout.cpp:999: When wrapping a line, only indent 4 spaces. [whitespace/indent] [3] ERROR: Source/WebCore/rendering/ComplexLineLayout.cpp:1009: Should have only a single space after a punctuation in a comment. [whitespace/comments] [5] ERROR: Source/WebCore/rendering/ComplexLineLayout.cpp:1111: Please replace ASSERT_WITH_SECURITY_IMPLICATION() with RELEASE_ASSERT_WITH_SECURITY_IMPLICATION(). [security/assertion] [5] ERROR: Source/WebCore/rendering/ComplexLineLayout.cpp:1293: Should have only a single space after a punctuation in a comment. [whitespace/comments] [5] ERROR: Source/WebCore/rendering/ComplexLineLayout.cpp:1302: Should have only a single space after a punctuation in a comment. [whitespace/comments] [5] ERROR: Source/WebCore/rendering/ComplexLineLayout.cpp:1571: Please replace ASSERT_WITH_SECURITY_IMPLICATION() with RELEASE_ASSERT_WITH_SECURITY_IMPLICATION(). [security/assertion] [5] ERROR: Source/WebCore/rendering/ComplexLineLayout.cpp:1575: Please replace ASSERT_WITH_SECURITY_IMPLICATION() with RELEASE_ASSERT_WITH_SECURITY_IMPLICATION(). [security/assertion] [5] ERROR: Source/WebCore/rendering/ComplexLineLayout.cpp:1675: Should have only a single space after a punctuation in a comment. [whitespace/comments] [5] ERROR: Source/WebCore/rendering/ComplexLineLayout.cpp:1684: Weird number of spaces at line-start. Are you using a 4-space indent? [whitespace/indent] [3] ERROR: Source/WebCore/rendering/ComplexLineLayout.cpp:1738: Weird number of spaces at line-start. Are you using a 4-space indent? [whitespace/indent] [3] ERROR: Source/WebCore/rendering/ComplexLineLayout.cpp:1763: Should have only a single space after a punctuation in a comment. [whitespace/comments] [5] ERROR: Source/WebCore/rendering/ComplexLineLayout.cpp:1772: Please replace ASSERT_WITH_SECURITY_IMPLICATION() with RELEASE_ASSERT_WITH_SECURITY_IMPLICATION(). [security/assertion] [5] ERROR: Source/WebCore/rendering/ComplexLineLayout.cpp:1824: Should have only a single space after a punctuation in a comment. [whitespace/comments] [5] ERROR: Source/WebCore/rendering/ComplexLineLayout.cpp:1903: Please replace ASSERT_WITH_SECURITY_IMPLICATION() with RELEASE_ASSERT_WITH_SECURITY_IMPLICATION(). [security/assertion] [5] ERROR: Source/WebCore/rendering/ComplexLineLayout.cpp:2139: Should have only a single space after a punctuation in a comment. [whitespace/comments] [5] ERROR: Source/WebCore/rendering/ComplexLineLayout.cpp:2151: Should have only a single space after a punctuation in a comment. [whitespace/comments] [5] ERROR: Source/WebCore/rendering/ComplexLineLayout.cpp:2152: Should have only a single space after a punctuation in a comment. [whitespace/comments] [5] Total errors found: 25 in 17 files If any of these errors are false positives, please file a bug against check-webkit-style.
Antti Koivisto
Comment 3 2019-08-10 10:20:47 PDT
Created attachment 376014 [details] patch
EWS Watchlist
Comment 4 2019-08-10 10:22:11 PDT
Attachment 376014 [details] did not pass style-queue: ERROR: Source/WebCore/rendering/ComplexLineLayout.cpp:1110: Please replace ASSERT_WITH_SECURITY_IMPLICATION() with RELEASE_ASSERT_WITH_SECURITY_IMPLICATION(). [security/assertion] [5] ERROR: Source/WebCore/rendering/ComplexLineLayout.cpp:1570: Please replace ASSERT_WITH_SECURITY_IMPLICATION() with RELEASE_ASSERT_WITH_SECURITY_IMPLICATION(). [security/assertion] [5] ERROR: Source/WebCore/rendering/ComplexLineLayout.cpp:1574: Please replace ASSERT_WITH_SECURITY_IMPLICATION() with RELEASE_ASSERT_WITH_SECURITY_IMPLICATION(). [security/assertion] [5] ERROR: Source/WebCore/rendering/ComplexLineLayout.cpp:1771: Please replace ASSERT_WITH_SECURITY_IMPLICATION() with RELEASE_ASSERT_WITH_SECURITY_IMPLICATION(). [security/assertion] [5] ERROR: Source/WebCore/rendering/ComplexLineLayout.cpp:1902: Please replace ASSERT_WITH_SECURITY_IMPLICATION() with RELEASE_ASSERT_WITH_SECURITY_IMPLICATION(). [security/assertion] [5] Total errors found: 5 in 17 files If any of these errors are false positives, please file a bug against check-webkit-style.
Antti Koivisto
Comment 5 2019-08-11 00:37:35 PDT
Created attachment 376032 [details] patch
EWS Watchlist
Comment 6 2019-08-11 03:08:14 PDT
Attachment 376032 [details] did not pass style-queue: ERROR: Source/WebCore/rendering/ComplexLineLayout.cpp:1110: Please replace ASSERT_WITH_SECURITY_IMPLICATION() with RELEASE_ASSERT_WITH_SECURITY_IMPLICATION(). [security/assertion] [5] ERROR: Source/WebCore/rendering/ComplexLineLayout.cpp:1570: Please replace ASSERT_WITH_SECURITY_IMPLICATION() with RELEASE_ASSERT_WITH_SECURITY_IMPLICATION(). [security/assertion] [5] ERROR: Source/WebCore/rendering/ComplexLineLayout.cpp:1574: Please replace ASSERT_WITH_SECURITY_IMPLICATION() with RELEASE_ASSERT_WITH_SECURITY_IMPLICATION(). [security/assertion] [5] ERROR: Source/WebCore/rendering/ComplexLineLayout.cpp:1771: Please replace ASSERT_WITH_SECURITY_IMPLICATION() with RELEASE_ASSERT_WITH_SECURITY_IMPLICATION(). [security/assertion] [5] ERROR: Source/WebCore/rendering/ComplexLineLayout.cpp:1902: Please replace ASSERT_WITH_SECURITY_IMPLICATION() with RELEASE_ASSERT_WITH_SECURITY_IMPLICATION(). [security/assertion] [5] Total errors found: 5 in 18 files If any of these errors are false positives, please file a bug against check-webkit-style.
WebKit Commit Bot
Comment 7 2019-08-11 04:02:08 PDT
Comment on attachment 376032 [details] patch Clearing flags on attachment: 376032 Committed r248517: <https://trac.webkit.org/changeset/248517>
WebKit Commit Bot
Comment 8 2019-08-11 04:02:09 PDT
All reviewed patches have been landed. Closing bug.
Radar WebKit Bug Importer
Comment 9 2019-08-11 04:03:22 PDT
<rdar://problem/54178193>
Note You need to log in before you can comment on or make changes to this bug.