Skip to content

deps: upgrade trace_engine to 0.0.56 #16565

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Merged
connorjclark merged 2 commits into from
Jun 26, 2025
Merged

deps: upgrade trace_engine to 0.0.56 #16565

connorjclark merged 2 commits into from
Jun 26, 2025

Conversation

@connorjclark
Copy link
Collaborator

@connorjclark connorjclark commented Jun 26, 2025 •
edited
Loading

https://github.com/ChromeDevTools/devtools-frontend/commits/main/front_end/models/trace

git log 74f60061a4564c53ac50920a5bea2bba52522b09...447cfed9f0643430ed5b1cde5d28d47bd275bb4e -- front_end/models/trace/

commit 447cfed9f0643430ed5b1cde5d28d47bd275bb4e
Author: Connor Clark <[email protected]>
Date:   Thu Jun 26 11:57:19 2025 -0700

    [RPP] Rename "INP by phase" to "INP breakdown"
    
    Fixed: 427766228
    Change-Id: I4b29343576fd57c964b7e7535fec5e53a117a298
    Reviewed-on: https://chromium-review.googlesource.com/c/devtools/devtools-frontend/+/6681887
    Reviewed-by: Paul Irish <[email protected]>
    Auto-Submit: Connor Clark <[email protected]>
    Commit-Queue: Connor Clark <[email protected]>

commit 13b84d38a7efaf087a73e5cd716b6943ffe56241
Author: Connor Clark <[email protected]>
Date:   Thu Jun 26 11:35:12 2025 -0700

    [RPP] Make DOM size insight informative
    
    Fixed: 425710510
    Change-Id: Ic8e628bfa4431f409885b9d09ef465c12cc919fb
    Reviewed-on: https://chromium-review.googlesource.com/c/devtools/devtools-frontend/+/6681807
    Commit-Queue: Paul Irish <[email protected]>
    Auto-Submit: Connor Clark <[email protected]>
    Reviewed-by: Paul Irish <[email protected]>
    Commit-Queue: Connor Clark <[email protected]>

commit 822b143deba1dadb8af173dceb2b620ee7a98c44
Author: Connor Clark <[email protected]>
Date:   Wed Jun 25 13:25:27 2025 -0700

    [RPP] Rename "LCP by phase" to "LCP breakdown"
    
    Bug: 427766228
    Change-Id: Ia4517cc2476d1e0027c70b4b574a2734888f0053
    Reviewed-on: https://chromium-review.googlesource.com/c/devtools/devtools-frontend/+/6674556
    Auto-Submit: Connor Clark <[email protected]>
    Reviewed-by: Jack Franklin <[email protected]>
    Commit-Queue: Jack Franklin <[email protected]>

commit b323a109134d218fcdad8fa4f23e558a80114e92
Author: Connor Clark <[email protected]>
Date:   Wed Jun 25 15:06:48 2025 -0700

    [RPP] Show top layout/style recalcs in DOM size insight
    
    Bug: 425710510
    Screenshot: https://i.imgur.com/gzojWXU.png
    Change-Id: Ib00c32c72b031e9a3c641ee49553ecc08d6eb7e5
    Reviewed-on: https://chromium-review.googlesource.com/c/devtools/devtools-frontend/+/6676083
    Auto-Submit: Connor Clark <[email protected]>
    Commit-Queue: Connor Clark <[email protected]>
    Commit-Queue: Paul Irish <[email protected]>
    Reviewed-by: Paul Irish <[email protected]>

commit ca4734014ee3ae48050fde2ea4a56ecbcf72e0ee
Author: Connor Clark <[email protected]>
Date:   Wed Jun 25 13:55:01 2025 -0700

    [RPP] Use "Web font" instead of "Font request" for CLS culprits
    
    Fixed: 427283051
    Change-Id: If8e38f02e6dcafee4cac413a9b4b2fd1357a1d9b
    Reviewed-on: https://chromium-review.googlesource.com/c/devtools/devtools-frontend/+/6674557
    Auto-Submit: Connor Clark <[email protected]>
    Commit-Queue: Paul Irish <[email protected]>
    Reviewed-by: Paul Irish <[email protected]>

commit 6fc694727e2cfccf74abfa735186c6e8e78c2975
Author: Connor Clark <[email protected]>
Date:   Wed Jun 25 13:44:26 2025 -0700

    Defer to X-TCPMs for request timings in Lightrider
    
    Fixed: 3086587
    Change-Id: I42458cfc309c2707155e7eedc0f5bc9004ca519d
    Reviewed-on: https://chromium-review.googlesource.com/c/devtools/devtools-frontend/+/6676082
    Auto-Submit: Connor Clark <[email protected]>
    Commit-Queue: Paul Irish <[email protected]>
    Reviewed-by: Paul Irish <[email protected]>

commit 5642fd62b6f81f9195a9e26c760220116962f0dc
Author: Connor Clark <[email protected]>
Date:   Wed Jun 25 12:51:54 2025 -0700

    Defer to X-TotalFetchedSize for transfer size in Lightrider
    
    Fixed: 427460845, 427460205
    Change-Id: I87bfddc9e4e58c202da982e2cde49937fd6d5e6f
    Reviewed-on: https://chromium-review.googlesource.com/c/devtools/devtools-frontend/+/6673836
    Commit-Queue: Paul Irish <[email protected]>
    Reviewed-by: Paul Irish <[email protected]>
    Auto-Submit: Connor Clark <[email protected]>

commit 44cc0789659c6bebac72771baee2367bc712d955
Author: Paul Irish <[email protected]>
Date:   Wed Jun 25 12:23:58 2025 -0700

    RPP: Rewrite LCP phase calculation
    
    Refactoring our LCP phase calculation to do it just once, allowing us to
    delete plenty of redundant phase building. The LCP insight's `phases`
    object now provides, for each phase, the Micro bounds plus the
    associated UIString.
    
    - s/loadTime/loadDuration/ as we renamed the user-facing name
    - Added some ascii art to clarify these phases.
    - The phase values in the UI are unchanged by this CL
    
    Bug: 372432365
    Change-Id: Ibaa3abd77bc563bfeae60d26694b2c6ca02df61c
    Reviewed-on: https://chromium-review.googlesource.com/c/devtools/devtools-frontend/+/6651250
    Reviewed-by: Jack Franklin <[email protected]>
    Reviewed-by: Connor Clark <[email protected]>
    Auto-Submit: Paul Irish <[email protected]>
    Commit-Queue: Paul Irish <[email protected]>

commit 931925a93515a9cfec22d3b61fae4a5fe11677e3
Author: Connor Clark <[email protected]>
Date:   Tue Jun 24 10:22:26 2025 -0700

    [RPP] Revise description for Force reflow insight
    
    Bug: 393453175
    Change-Id: Ib47a46f3582ccf15dcf3cb9e5f790f50de979712
    Reviewed-on: https://chromium-review.googlesource.com/c/devtools/devtools-frontend/+/6664481
    Reviewed-by: Paul Irish <[email protected]>
    Auto-Submit: Connor Clark <[email protected]>
    Commit-Queue: Connor Clark <[email protected]>

commit 3ab19cd9b992bf5ca4204b8922946da21ff83d49
Author: Connor Clark <[email protected]>
Date:   Tue Jun 24 16:49:58 2025 -0700

    [RPP] Correct image paint size using host DPR
    
    It turns out that image paints in Chrome never use emulated DPR, but
    instead always use the host's DPR setting. This means our responsive
    image advice was working with bad data.
    
    Fixed: 416580500
    Change-Id: I5019ad5e4906f71caadb1413e49ba3ff568bc3c0
    Reviewed-on: https://chromium-review.googlesource.com/c/devtools/devtools-frontend/+/6669776
    Commit-Queue: Connor Clark <[email protected]>
    Reviewed-by: Paul Irish <[email protected]>

commit 33fb7789afcbe4ad8092c932bdb35128ded66da4
Author: Connor Clark <[email protected]>
Date:   Tue Jun 24 16:59:01 2025 -0700

    Revert "[RPP] Exclude fixed-width viewport from Viewport insight"
    
    This reverts commit b5eef4c0f5e56fc0f5a07501a1bcad1b021d3de3.
    
    Reason for revert: the code was correct before.
    
    Bug: https://github.com/GoogleChrome/lighthouse/issues/16554
    Original change's description:
    > [RPP] Exclude fixed-width viewport from Viewport insight
    >
    > Bug: https://github.com/GoogleChrome/lighthouse/issues/16554
    > Change-Id: I8f72a595d84d1edc251f3d9e81fe6c0558b0a955
    > Reviewed-on: https://chromium-review.googlesource.com/c/devtools/devtools-frontend/+/6668717
    > Auto-Submit: Connor Clark <[email protected]>
    > Reviewed-by: Paul Irish <[email protected]>
    > Commit-Queue: Connor Clark <[email protected]>
    
    Bug: https://github.com/GoogleChrome/lighthouse/issues/16554
    No-Presubmit: true
    No-Tree-Checks: true
    No-Try: true
    Change-Id: I656feda73e66d4112f625c7fce1e79513eeb8a34
    Reviewed-on: https://chromium-review.googlesource.com/c/devtools/devtools-frontend/+/6669383
    Auto-Submit: Connor Clark <[email protected]>
    Commit-Queue: Rubber Stamper <[email protected]>
    Bot-Commit: Rubber Stamper <[email protected]>

commit a81c8a4ad7c94428a6b0b6a03b4d28182f4f6732
Author: Connor Clark <[email protected]>
Date:   Tue Jun 24 10:19:57 2025 -0700

    Add translator notes to not translate "preconnect"
    
    Bug: none
    Change-Id: I92a21e9488ec56162d0759db82b1a1df3f31aef9
    Reviewed-on: https://chromium-review.googlesource.com/c/devtools/devtools-frontend/+/6668715
    Commit-Queue: Paul Irish <[email protected]>
    Reviewed-by: Paul Irish <[email protected]>
    Auto-Submit: Connor Clark <[email protected]>

commit b5eef4c0f5e56fc0f5a07501a1bcad1b021d3de3
Author: Connor Clark <[email protected]>
Date:   Tue Jun 24 11:25:41 2025 -0700

    [RPP] Exclude fixed-width viewport from Viewport insight
    
    Bug: https://github.com/GoogleChrome/lighthouse/issues/16554
    Change-Id: I8f72a595d84d1edc251f3d9e81fe6c0558b0a955
    Reviewed-on: https://chromium-review.googlesource.com/c/devtools/devtools-frontend/+/6668717
    Auto-Submit: Connor Clark <[email protected]>
    Reviewed-by: Paul Irish <[email protected]>
    Commit-Queue: Connor Clark <[email protected]>

commit 9e8c005dc26fc79f86398f1e9a773704e4918444
Author: Nancy Li <[email protected]>
Date:   Mon Jun 23 15:59:12 2025 +0000

    [RPP] Use `byId` to find the document request
    
    Bug: none
    Change-Id: I8c87ea3971009e34e2b73998b2631271c3a0e570
    Reviewed-on: https://chromium-review.googlesource.com/c/devtools/devtools-frontend/+/6662398
    Reviewed-by: Connor Clark <[email protected]>
    Commit-Queue: Nancy Li <[email protected]>

@connorjclark connorjclark requested a review from a team as a code owner June 26, 2025 22:05
@connorjclark connorjclark requested review from paulirish and removed request for a team June 26, 2025 22:05
@connorjclark connorjclark merged commit 4cecba5 into main Jun 26, 2025
28 checks passed
@connorjclark connorjclark deleted the te-56 branch June 26, 2025 23:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@paulirish paulirish Awaiting requested review from paulirish paulirish is a code owner automatically assigned from GoogleChrome/lighthouse-hackers

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

1 participant

@connorjclark