fix: lock window just show background widget

[yixinshark]

as title

Log: as title
Pms: bug-307701

[deepin-ci-robot]

deepin pr auto review

代码审查意见：

1. 锁屏和唤醒逻辑：

   • 在LockFrame和LockWorker中，关于锁屏和唤醒的逻辑处理较为复杂，建议将这部分逻辑抽象成独立的函数或类，以提高代码的可读性和可维护性。
   • 在LockFrame中，QTimer::singleShot的延时处理可能会导致界面响应不及时，建议评估是否需要更精确的延时机制。

2. beforeInvokeAction函数返回值：

   • 在WarningContent类中，beforeInvokeAction函数的返回值类型从void改为bool，但函数体中的一些return语句没有遵循新的返回类型，可能会导致编译错误或逻辑错误。建议统一返回类型，并确保所有可能的执行路径都有返回值。

3. 重复代码：

   • 在WarningContent类中，doAccecpShutdownInhibit函数被注释掉了，但beforeInvokeAction函数中仍然有调用doAccecpShutdownInhibit的代码。如果doAccecpShutdownInhibit函数不再需要，应该移除相关调用；如果需要，应该取消注释并确保函数逻辑正确。

4. setContent函数的调用：

   • 在FullscreenBackground类中，setContent函数在设置新内容之前，没有检查新内容是否与当前内容相同，这可能会导致不必要的界面更新。建议添加检查逻辑，避免重复设置相同的内容。

5. 代码注释：

   • 代码中缺少足够的注释，特别是对于复杂的逻辑和关键步骤，应该添加详细的注释来解释代码的意图和实现细节。

6. 潜在的内存泄漏：

   • 在FullscreenBackground类中，m_content指针在设置新内容时，没有检查旧内容是否已经被正确删除或隐藏，这可能导致内存泄漏。建议在设置新内容之前，确保旧内容被正确处理。

7. 连接信号和槽的清理：

   • 在WarningContent类中，信号和槽的连接应该在适当的地方进行清理，以避免内存泄漏。建议在类的析构函数中添加清理逻辑，或者在不需要连接时断开连接。

8. 错误处理：

   • 在WarningContent类中，beforeInvokeAction函数没有处理可能出现的错误情况，例如listInhibitors函数调用失败。建议添加错误处理逻辑，确保程序的健壮性。

以上是针对代码审查的初步意见，具体的改进措施需要根据项目的具体需求和代码上下文来决定。

[deepin-ci-robot]

[APPROVALNOTIFIER] This PR is NOT APPROVED

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

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

Details

Needs approval from an approver in each of these files:

• debian/deepin/OWNERS

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