Files
StellarX-kaifa/开发记录/Fix/Fix-BUG-20261009-0002-同父Canvas上层完整恢复优先级修复.md

3.4 KiB
Raw Permalink Blame History

Fix-BUG-20261009-0002

适用场景:记录某个 BUG 的修复方案、影响评估、验证结果与落地信息。

关联信息

  • Fix ID: Fix-BUG-20261009-0002
  • 关联 BUG ID: BUG-20261009-0002
  • 修复目标: 下层持久重绘命中上层兄弟时,优先完整恢复上层而不是只提交其脏子树
  • 状态:已完成,自动与 KEY5 人工复验通过
  • 负责人: Codex
  • 分支 / 版本: master,实现提交 29df994

根因分析

  • 根因: Canvas::requestRepaint() 把“上层自身有脏子树”放在“下层持久 coverage 命中上层”之前,导致完整 overlay 恢复被局部提交短路。
  • 触发条件: 下层与上层相交,且一次焦点事务同时让下层失焦、上层获焦。
  • 为什么之前没发现: 既有 overlay 测试主要覆盖只有下层变脏的情况,没有覆盖上下层同批变脏。
  • 关键证据: 额外事件触发完整重绘后画面恢复,说明 z-order 和内容本身正确,错误集中在局部提交优先级。

修复方案

  • 修复思路: 每个上层兄弟在选择提交方式前,先判断本轮累计的下层持久 coverage 是否与其持久/完整 coverage 相交。
  • 关键改动:
    • persistentHit 为真时,无条件作废上层旧快照并完整重画。
    • 只有不命中下层持久内容时,才允许继续提交上层自己的局部脏子树。
    • Tooltip 等临时 coverage 不进入持久相交条件,继续保持既有临时浮层契约。
    • 扩展 focus UI,构造同父上下层 Canvas,并在下层失焦、上层获焦后精确断言交叠点颜色。
  • 涉及文件 / 类 / 函数:
    • src/StellarX/Canvas.cpp
    • tests/UiFixture.cpp
    • scripts/run-ui-tests.ps1
  • 影响的 API / 行为: 无公开 API 变化;调整托管局部重绘内部优先级。
  • 关键约束 / 不变量: 完整 coverage 负责“是否被写到”,持久 coverage 负责“是否需要刷新背景快照”;两者不能重新合并。
  • 回滚点 / 开关: 回退 Canvas 分支优先级与同批双脏像素回归。

影响评估

  • 影响范围: Canvas 内相交兄弟、嵌套 Canvas、焦点框与持久内容的同批重绘。
  • 兼容性影响:无
  • 行为变化:有,错误覆盖改为按 z-order 完整恢复。
  • 性能影响:仅在下层持久写屏区域命中上层时,把上层局部提交升级为完整控件重画。
  • 回归风险:
    • 若把 Tooltip 临时 coverage 错当持久内容,可能重新引入 Tooltip 被擦除;本修复继续使用独立 persistent coverage,并执行完整 tooltip UI。
    • 完整上层重画会刷新其背景快照,因此必须先让下层本轮像素稳定落屏。

验证结果

  • 扩展 focus UI 连续 3/3 通过;同父下层/上层焦点切换后,交叠点精确保持上层背景色 RGB(200,160,120)。
  • 完整 UI 1/1 通过,tooltip 到期显示与移出清除均未回归。
  • Debug x64、x64 ASan、Release x64、Debug Win32 均为 67/67;KEY1–KEY6 全部编译/链接成功。
  • 最终 fixture 自动 UI 证据:TestResults/ui/20261009-223506-755-18932,九场景 9/9,通过 fixture SHA-256 F439C0636CD6B979A47C4594BC5AB3728F6342F49602FE43078435011A877793。
  • 人工复验: 用户重新执行 KEY5 D 区,D-1 → D-2 未再发生下层重绘覆盖,暂未发现问题。

落地信息

  • Commit: 29df994
  • PR: 无
  • 发布版本: 未发布
  • 证据:
    • TestResults/ui/20261009-220625-941-15096
    • TestResults/ui/20261009-220940-941-31488