Add crash key for dirty layout during painting. [chromium/src : main]

0 views
Skip to first unread message

Morten Stenshorne (Gerrit)

unread,
Feb 3, 2026, 9:11:32 AM (5 days ago) Feb 3
to Rune Lillesveen, Ian Kilpatrick, chromium...@chromium.org, blink-revi...@chromium.org, blink-rev...@chromium.org, blink-...@chromium.org, zol...@webkit.org
Attention needed from Ian Kilpatrick and Rune Lillesveen

New activity on the change

Open in Gerrit

Related details

Attention is currently required from:
  • Ian Kilpatrick
  • Rune Lillesveen
Submit Requirements:
  • requirement satisfiedCode-Coverage
  • requirement satisfiedCode-Owners
  • requirement is not satisfiedCode-Review
  • requirement is not satisfiedReview-Enforcement
Inspect html for hidden footers to help with email filtering. To unsubscribe visit settings. DiffyGerrit
Gerrit-MessageType: comment
Gerrit-Project: chromium/src
Gerrit-Branch: main
Gerrit-Change-Id: I0c5425f4661e079e94d51a1262de7c3190ba021b
Gerrit-Change-Number: 7531432
Gerrit-PatchSet: 1
Gerrit-Owner: Morten Stenshorne <mste...@chromium.org>
Gerrit-Reviewer: Ian Kilpatrick <ikilp...@chromium.org>
Gerrit-Reviewer: Rune Lillesveen <fut...@chromium.org>
Gerrit-Attention: Ian Kilpatrick <ikilp...@chromium.org>
Gerrit-Attention: Rune Lillesveen <fut...@chromium.org>
Gerrit-Comment-Date: Tue, 03 Feb 2026 14:11:16 +0000
Gerrit-HasComments: No
Gerrit-Has-Labels: No
satisfied_requirement
unsatisfied_requirement
open
diffy

Rune Lillesveen (Gerrit)

unread,
Feb 3, 2026, 9:15:25 AM (5 days ago) Feb 3
to Morten Stenshorne, Rune Lillesveen, Ian Kilpatrick, chromium...@chromium.org, blink-revi...@chromium.org, blink-rev...@chromium.org, blink-...@chromium.org, zol...@webkit.org
Attention needed from Ian Kilpatrick and Morten Stenshorne

Rune Lillesveen voted Code-Review+1

Code-Review+1
Open in Gerrit

Related details

Attention is currently required from:
  • Ian Kilpatrick
  • Morten Stenshorne
Submit Requirements:
    • requirement satisfiedCode-Coverage
    • requirement satisfiedCode-Owners
    • requirement satisfiedCode-Review
    • requirement satisfiedReview-Enforcement
    Inspect html for hidden footers to help with email filtering. To unsubscribe visit settings. DiffyGerrit
    Gerrit-MessageType: comment
    Gerrit-Project: chromium/src
    Gerrit-Branch: main
    Gerrit-Change-Id: I0c5425f4661e079e94d51a1262de7c3190ba021b
    Gerrit-Change-Number: 7531432
    Gerrit-PatchSet: 1
    Gerrit-Owner: Morten Stenshorne <mste...@chromium.org>
    Gerrit-Reviewer: Ian Kilpatrick <ikilp...@chromium.org>
    Gerrit-Reviewer: Rune Lillesveen <fut...@chromium.org>
    Gerrit-Attention: Ian Kilpatrick <ikilp...@chromium.org>
    Gerrit-Attention: Morten Stenshorne <mste...@chromium.org>
    Gerrit-Comment-Date: Tue, 03 Feb 2026 14:15:08 +0000
    Gerrit-HasComments: No
    Gerrit-Has-Labels: Yes
    satisfied_requirement
    open
    diffy

    Morten Stenshorne (Gerrit)

    unread,
    Feb 3, 2026, 9:19:45 AM (5 days ago) Feb 3
    to Rune Lillesveen, Ian Kilpatrick, chromium...@chromium.org, blink-revi...@chromium.org, blink-rev...@chromium.org, blink-...@chromium.org, zol...@webkit.org
    Attention needed from Ian Kilpatrick

    Morten Stenshorne voted Commit-Queue+2

    Commit-Queue+2
    Open in Gerrit

    Related details

    Attention is currently required from:
    • Ian Kilpatrick
    Submit Requirements:
    • requirement satisfiedCode-Coverage
    • requirement satisfiedCode-Owners
    • requirement satisfiedCode-Review
    • requirement satisfiedReview-Enforcement
    Inspect html for hidden footers to help with email filtering. To unsubscribe visit settings. DiffyGerrit
    Gerrit-MessageType: comment
    Gerrit-Project: chromium/src
    Gerrit-Branch: main
    Gerrit-Change-Id: I0c5425f4661e079e94d51a1262de7c3190ba021b
    Gerrit-Change-Number: 7531432
    Gerrit-PatchSet: 1
    Gerrit-Owner: Morten Stenshorne <mste...@chromium.org>
    Gerrit-Reviewer: Ian Kilpatrick <ikilp...@chromium.org>
    Gerrit-Reviewer: Morten Stenshorne <mste...@chromium.org>
    Gerrit-Reviewer: Rune Lillesveen <fut...@chromium.org>
    Gerrit-Attention: Ian Kilpatrick <ikilp...@chromium.org>
    Gerrit-Comment-Date: Tue, 03 Feb 2026 14:19:29 +0000
    Gerrit-HasComments: No
    Gerrit-Has-Labels: Yes
    satisfied_requirement
    open
    diffy

    Chromium LUCI CQ (Gerrit)

    unread,
    Feb 3, 2026, 10:20:04 AM (5 days ago) Feb 3
    to Morten Stenshorne, Rune Lillesveen, Ian Kilpatrick, chromium...@chromium.org, blink-revi...@chromium.org, blink-rev...@chromium.org, blink-...@chromium.org, zol...@webkit.org

    Chromium LUCI CQ submitted the change

    Change information

    Commit message:
    Add crash key for dirty layout during painting.

    Not sure where to begin. First of all, dump some information about the
    layout tree without crashing at places where we bail when attempting to
    paint while needing layout. Also do this in LayoutBlock::Paint() before
    NOTREACHED().
    Bug: 478682594
    Change-Id: I0c5425f4661e079e94d51a1262de7c3190ba021b
    Reviewed-by: Rune Lillesveen <fut...@chromium.org>
    Commit-Queue: Morten Stenshorne <mste...@chromium.org>
    Cr-Commit-Position: refs/heads/main@{#1578777}
    Files:
    • M third_party/blink/renderer/core/layout/layout_object.cc
    • M third_party/blink/renderer/core/layout/layout_object.h
    • M third_party/blink/renderer/core/paint/box_fragment_painter.cc
    • M third_party/blink/renderer/core/paint/paint_layer.cc
    Change size: S
    Delta: 4 files changed, 35 insertions(+), 0 deletions(-)
    Branch: refs/heads/main
    Submit Requirements:
    • requirement satisfiedCode-Review: +1 by Rune Lillesveen
    Open in Gerrit
    Inspect html for hidden footers to help with email filtering. To unsubscribe visit settings. DiffyGerrit
    Gerrit-MessageType: merged
    Gerrit-Project: chromium/src
    Gerrit-Branch: main
    Gerrit-Change-Id: I0c5425f4661e079e94d51a1262de7c3190ba021b
    Gerrit-Change-Number: 7531432
    Gerrit-PatchSet: 2
    Gerrit-Owner: Morten Stenshorne <mste...@chromium.org>
    Gerrit-Reviewer: Chromium LUCI CQ <chromiu...@luci-project-accounts.iam.gserviceaccount.com>
    open
    diffy
    satisfied_requirement

    Chrome Crash (Prod) (Gerrit)

    unread,
    Feb 6, 2026, 10:35:58 PM (2 days ago) Feb 6
    to Chromium LUCI CQ, Morten Stenshorne, Rune Lillesveen, Ian Kilpatrick, chromium...@chromium.org, blink-revi...@chromium.org, blink-rev...@chromium.org, blink-...@chromium.org, zol...@webkit.org

    Chrome Crash (Prod) has created a revert of this change

    Related details

    Attention set is empty
    Submit Requirements:
    • requirement satisfiedCode-Coverage
    • requirement satisfiedCode-Owners
    • requirement satisfiedCode-Review
    • requirement satisfiedReview-Enforcement
    Inspect html for hidden footers to help with email filtering. To unsubscribe visit settings. DiffyGerrit
    Gerrit-MessageType: revert
    satisfied_requirement
    open
    diffy
    Reply all
    Reply to author
    Forward
    0 new messages