Skip to content

fix: 修复 DForeignWindow event方法返回值的问题#365

Merged
deepin-bot[bot] merged 1 commit intolinuxdeepin:masterfrom
18202781743:master
Feb 4, 2026
Merged

fix: 修复 DForeignWindow event方法返回值的问题#365
deepin-bot[bot] merged 1 commit intolinuxdeepin:masterfrom
18202781743:master

Conversation

@18202781743
Copy link
Contributor

将 DForeignWindow::event 方法的返回值从固定返回 false 改为返回基类
QWindow::event(e) 的结果。这样确保事件能够正确地在继承链中传递和处理,避
免事件处理被意外截断。
特别是deleteLater() 投递的
QEvent::DeferredDelete事件,被截断后造成了内存不断增长的问题.

Log: 修复窗口事件处理问题

Influence:

  1. 测试窗口各种事件(如鼠标、键盘、焦点等)是否正常响应
  2. 验证窗口状态变化是否正确处理
  3. 测试窗口与其他组件的交互是否正常
  4. 验证事件传递链的完整性

Bug: https://pms.uniontech.com//bug-view-342765.html
Change-Id: I6b9f82b6fdec9e23f7e64dccad007779be24e166

将 DForeignWindow::event 方法的返回值从固定返回 false 改为返回基类
QWindow::event(e) 的结果。这样确保事件能够正确地在继承链中传递和处理,避
免事件处理被意外截断。
特别是deleteLater() 投递的
QEvent::DeferredDelete事件,被截断后造成了内存不断增长的问题.

Log: 修复窗口事件处理问题

Influence:
1. 测试窗口各种事件(如鼠标、键盘、焦点等)是否正常响应
2. 验证窗口状态变化是否正确处理
3. 测试窗口与其他组件的交互是否正常
4. 验证事件传递链的完整性

Bug: https://pms.uniontech.com//bug-view-342765.html
Change-Id: I6b9f82b6fdec9e23f7e64dccad007779be24e166
@18202781743 18202781743 requested review from BLumia and mhduiy February 4, 2026 03:05
@deepin-ci-robot
Copy link
Contributor

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: 18202781743, BLumia

The full list of commands accepted by this bot can be found here.

Details Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@18202781743
Copy link
Contributor Author

/forcemerge

@deepin-bot
Copy link
Contributor

deepin-bot bot commented Feb 4, 2026

This pr force merged! (status: blocked)

@deepin-bot deepin-bot bot merged commit 9b11d77 into linuxdeepin:master Feb 4, 2026
19 of 22 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants