[6.x] Fix CP tab crash when expanding a Bard set after a copy#14951
[6.x] Fix CP tab crash when expanding a Bard set after a copy#14951o1y wants to merge 3 commits into
Conversation
Defer the set reveal by one animation frame so the nested ProseMirror editor doesn't lay out during the trusted-click turn (Chrome ≥148 CFI crash). Covers both reveal paths: expandSet() and expandAll().
|
Whoa. I'm converting this PR to a draft, as I've realized that quite a few fields are affected by this bug and proper testing is needed here. Maybe the team can take it from here. Unfortunately, we're also hitting this crash a few times a day and losing content.. I was hoping it'd be an easy fix 😅 Other reveal-on-click surfaces (collapsible sections, tabs, revealer) share the same pattern and likely need the same guard as well (not verified though).. |
|
Well then let Claude decide! 🥲 |
|
@jasonvarga what was the outcome of that? We're now affected by this bug so anything we can do to help :) |
|
I've opened #14967 blindly. I couldn't reproduce the bug myself. Would one or either of you be able to test that PR to see if it fixes the issue? We can continue the conversation there. |

Fixes #14946
The chromium render crash ("Aw, Snap") fires when all of these line up in the same click
display:noneto visible).When all three happen inside the one click, Chrome probably trips over its own editing/clipboard code and kills the tab.
In this commit we defer the reveal by one animation frame so the Bard editor is shown just after the click. That moves the work off the click, and the crash can't happen.