Bug 184833 - [Simple line layout] Generate inline boxtree using simple line layout runs.
Summary: [Simple line layout] Generate inline boxtree using simple line layout runs.
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: Layout and Rendering (show other bugs)
Version: WebKit Nightly Build
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: zalan
URL:
Keywords: InRadar
Depends on:
Blocks:
 
Reported: 2018-04-20 11:53 PDT by zalan
Modified: 2018-04-23 07:48 PDT (History)
7 users (show)

See Also:


Attachments
Patch (14.54 KB, patch)
2018-04-20 11:55 PDT, zalan
no flags Details | Formatted Diff | Diff
Archive of layout-test-results from ews121 for ios-simulator-wk2 (2.22 MB, application/zip)
2018-04-20 13:30 PDT, EWS Watchlist
no flags Details
Archive of layout-test-results from ews202 for win-future (1.53 MB, application/zip)
2018-04-20 16:51 PDT, EWS Watchlist
no flags Details
Patch (15.14 KB, patch)
2018-04-22 22:20 PDT, zalan
no flags Details | Formatted Diff | Diff
Archive of layout-test-results from ews202 for win-future (1.54 MB, application/zip)
2018-04-22 23:44 PDT, EWS Watchlist
no flags Details

Note You need to log in before you can comment on or make changes to this bug.
Description zalan 2018-04-20 11:53:19 PDT
only for single renderer, no pagination etc.
Comment 1 zalan 2018-04-20 11:55:03 PDT
Created attachment 338446 [details]
Patch
Comment 2 Antti Koivisto 2018-04-20 12:00:30 PDT
Comment on attachment 338446 [details]
Patch

View in context: https://bugs.webkit.org/attachment.cgi?id=338446&action=review

> Source/WebCore/rendering/SimpleLineLayoutFunctions.cpp:278
> +bool canUseForLineBoxTree(RenderBlockFlow& flow, const Layout& layout)
> +{
> +    return !layout.isPaginated() && !flow.style().preserveNewline() && flow.firstChild() && is<RenderText>(*flow.firstChild()) && flow.firstChild() == flow.lastChild();
> +}

This would be more readable with multiple lines and early returns.
Comment 3 EWS Watchlist 2018-04-20 13:30:56 PDT
Comment on attachment 338446 [details]
Patch

Attachment 338446 [details] did not pass ios-sim-ews (ios-simulator-wk2):
Output: http://webkit-queues.webkit.org/results/7386391

New failing tests:
fast/text/mark-matches-rendering-simple-lines.html
Comment 4 EWS Watchlist 2018-04-20 13:30:57 PDT
Created attachment 338457 [details]
Archive of layout-test-results from ews121 for ios-simulator-wk2

The attached test failures were seen while running run-webkit-tests on the ios-sim-ews.
Bot: ews121  Port: ios-simulator-wk2  Platform: Mac OS X 10.13.4
Comment 5 EWS Watchlist 2018-04-20 16:51:34 PDT
Comment on attachment 338446 [details]
Patch

Attachment 338446 [details] did not pass win-ews (win):
Output: http://webkit-queues.webkit.org/results/7389282

Number of test failures exceeded the failure limit.
Comment 6 EWS Watchlist 2018-04-20 16:51:36 PDT
Created attachment 338491 [details]
Archive of layout-test-results from ews202 for win-future

The attached test failures were seen while running run-webkit-tests on the win-ews.
Bot: ews202  Port: win-future  Platform: CYGWIN_NT-6.1-2.9.0-0.318-5-3-x86_64-64bit
Comment 7 zalan 2018-04-22 22:20:37 PDT
Created attachment 338567 [details]
Patch
Comment 8 EWS Watchlist 2018-04-22 23:44:35 PDT
Comment on attachment 338567 [details]
Patch

Attachment 338567 [details] did not pass win-ews (win):
Output: http://webkit-queues.webkit.org/results/7409126

Number of test failures exceeded the failure limit.
Comment 9 EWS Watchlist 2018-04-22 23:44:37 PDT
Created attachment 338568 [details]
Archive of layout-test-results from ews202 for win-future

The attached test failures were seen while running run-webkit-tests on the win-ews.
Bot: ews202  Port: win-future  Platform: CYGWIN_NT-6.1-2.9.0-0.318-5-3-x86_64-64bit
Comment 10 WebKit Commit Bot 2018-04-23 07:47:05 PDT
Comment on attachment 338567 [details]
Patch

Clearing flags on attachment: 338567

Committed r230914: <https://trac.webkit.org/changeset/230914>
Comment 11 WebKit Commit Bot 2018-04-23 07:47:07 PDT
All reviewed patches have been landed.  Closing bug.
Comment 12 Radar WebKit Bug Importer 2018-04-23 07:48:39 PDT
<rdar://problem/39645850>