| Summary: | Add TraceScope to capture layout-ey activities only | ||||||||||||
|---|---|---|---|---|---|---|---|---|---|---|---|---|---|
| Product: | WebKit | Reporter: | zalan <zalan> | ||||||||||
| Component: | Layout and Rendering | Assignee: | zalan <zalan> | ||||||||||
| Status: | RESOLVED FIXED | ||||||||||||
| Severity: | Normal | CC: | benjamin, bfulgham, cdumez, cmarcelo, ews-watchlist, simon.fraser, webkit-bug-importer, zalan | ||||||||||
| Priority: | P2 | Keywords: | InRadar | ||||||||||
| Version: | WebKit Nightly Build | ||||||||||||
| Hardware: | Unspecified | ||||||||||||
| OS: | Unspecified | ||||||||||||
| Attachments: |
|
||||||||||||
|
Description
zalan
2023-07-27 14:21:16 PDT
Created attachment 467137 [details]
Patch
Comment on attachment 467137 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=467137&action=review > Source/WebCore/page/LocalFrameViewLayoutContext.cpp:246 > + TraceScope tracingScope(LayoutStart, LayoutEnd); This no longer includes InViewSizeAdjust or InPostLayout, which don't have their own trace point coverage. This will make it harder to see how much tine they are taking for forced layouts. (In reply to Simon Fraser (smfr) from comment #2) > Comment on attachment 467137 [details] > Patch > > View in context: > https://bugs.webkit.org/attachment.cgi?id=467137&action=review > > > Source/WebCore/page/LocalFrameViewLayoutContext.cpp:246 > > + TraceScope tracingScope(LayoutStart, LayoutEnd); > > This no longer includes InViewSizeAdjust or InPostLayout, which don't have > their own trace point coverage. This will make it harder to see how much > tine they are taking for forced layouts. sure but they are not layout either. do you want me to add new trace scopes? (In reply to zalan from comment #3) > (In reply to Simon Fraser (smfr) from comment #2) > > Comment on attachment 467137 [details] > > Patch > > > > View in context: > > https://bugs.webkit.org/attachment.cgi?id=467137&action=review > > > > > Source/WebCore/page/LocalFrameViewLayoutContext.cpp:246 > > > + TraceScope tracingScope(LayoutStart, LayoutEnd); > > > > This no longer includes InViewSizeAdjust or InPostLayout, which don't have > > their own trace point coverage. This will make it harder to see how much > > tine they are taking for forced layouts. > > sure but they are not layout either. do you want me to add new trace scopes? also don't we just re-enter LocalFrameViewLayoutContext::performLayout? Created attachment 467138 [details]
Patch
Created attachment 467139 [details]
Patch
Created attachment 467140 [details]
Patch
Committed 266372@main (703501c4541f): <https://commits.webkit.org/266372@main> All reviewed patches have been landed. Closing bug and clearing flags on attachment 467140 [details]. |