RESOLVED FIXED 238079
[IFC][Integration] InlineIterator::Line::enclosingTop is really an enclosingTopAdjustedForPrecedingLine 
https://bugs.webkit.org/show_bug.cgi?id=238079
Summary [IFC][Integration] InlineIterator::Line::enclosingTop is really an enclosingT...
zalan
Reported 2022-03-18 09:24:38 PDT
ssia
Attachments
Patch (7.66 KB, patch)
2022-03-18 09:27 PDT, zalan 		no flags
DetailsFormatted DiffDiff
View All Add attachment proposed patch, testcase, etc.
zalan
Comment 1 2022-03-18 09:27:08 PDT
Created attachment 455099 [details] Patch
Antti Koivisto
Comment 2 2022-03-18 09:57:41 PDT
Comment on attachment 455099 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=455099&action=review > Source/WebCore/layout/integration/InlineIteratorLine.h:98 > - LayoutUnit enclosingTop() const; > LayoutUnit enclosingBottom() const; enclosingBottom looks lonely now.
zalan
Comment 3 2022-03-18 10:02:17 PDT
(In reply to Antti Koivisto from comment #2) > Comment on attachment 455099 [details] > Patch > > View in context: > https://bugs.webkit.org/attachment.cgi?id=455099&action=review > > > Source/WebCore/layout/integration/InlineIteratorLine.h:98 > > - LayoutUnit enclosingTop() const; > > LayoutUnit enclosingBottom() const; > > enclosingBottom looks lonely now. not for long!
EWS
Comment 4 2022-03-18 14:24:12 PDT
Found 1 new test failure: imported/w3c/web-platform-tests/workers/interfaces/WorkerUtils/importScripts/report-error-setTimeout-redirect-to-cross-origin.sub.any.worker.html
EWS
Comment 5 2022-03-18 17:42:04 PDT
Committed r291512 (248618@main): <https://commits.webkit.org/248618@main> All reviewed patches have been landed. Closing bug and clearing flags on attachment 455099 [details].
Radar WebKit Bug Importer
Comment 6 2022-03-18 17:43:14 PDT
<rdar://problem/90512456>
Note You need to log in before you can comment on or make changes to this bug.