feat: add xdg-activation-v1 support to dock compositor

[18202781743]

Implement the xdg-activation-v1 protocol for the dock compositor,
enabling applications to activate their windows through the dock's
taskbar integration. This allows proper window focus management when
clicking dock icons.

The patch adds:

1. XdgActivationManager compositor extension implementing
   xdg_activation_v1 global
2. XdgActivationTokenV1 implementation for token lifecycle management
3. Integration with outer compositor via ds::XdgActivation for token
   delegation
4. QML registration for easy inclusion in DockCompositor

When a plugin client commits an activation token, the manager requests
a real token from the outer compositor and forwards it back, ensuring
proper multi-layer Wayland compositor activation handling.

Log: Added xdg-activation-v1 protocol support for dock activation
functionality

Influence:

1. Test clicking dock icons focuses applications correctly
2. Verify activation tokens are properly forwarded between compositor
   layers
3. Test multiple rapid activation requests
4. Verify proper cleanup when clients disconnect
5. Test activation tokens with different app IDs

feat: 为任务栏合成器添加 xdg-activation-v1 支持

在任务栏合成器中实现 xdg-activation-v1 协议，使应用程序能通过任务栏集成
激活其窗口。这允许在点击任务栏图标时进行正确的窗口焦点管理。

修改内容：

1. 添加 XdgActivationManager 合成器扩展，实现 xdg_activation_v1 全局对象
2. 实现 XdgActivationTokenV1 用于令牌生命周期管理
3. 通过 ds::XdgActivation 与外部合成器集成进行令牌代理
4. 在 DockCompositor 中注册 QML 组件

当插件客户端提交激活令牌时，管理器从外部合成器请求真实令牌并转发回去，确
保多层 Wayland 合成器激活机制的正确处理。

Log: 新增 xdg-activation-v1 协议支持任务栏激活功能

Influence:

1. 测试点击任务栏图标能否正确聚焦应用程序
2. 验证激活令牌在合成器层之间正确转发
3. 测试多次快速激活请求
4. 验证客户端断开连接时的清理逻辑
5. 测试不同应用 ID 的激活令牌

[sourcery-ai]

Sorry @18202781743, you have reached your weekly rate limit of 500000 diff characters.

Please try again later or upgrade to continue using Sourcery

[deepin-bot]

TAG Bot

TAG: 2.0.43
EXISTED: yes
DISTRIBUTION: unstable

[deepin-ci-robot]

[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:

• 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

[deepin-ci-robot]

deepin pr auto review

代码审查报告

总体评价

这段代码实现了一个 XDG 激活管理器(XdgActivationManager)，用于处理 Wayland 协议中的激活令牌功能。代码结构清晰，实现了基本的激活令牌管理功能。不过有一些可以改进的地方。

具体改进建议

1. 代码质量

1. 错误处理不足：

   • 在 XdgActivationManager::initialize() 中，对 wl_resource_create 的返回值没有检查，可能导致空指针解引用
   • 建议：

   QWaylandResource tokenResource(wl_resource_create(resource->client(), &xdg_activation_token_v1_interface, wl_resource_get_version(resource->handle), id));
   if (!tokenResource.resource()) {
       qCWarning(xdgActivationMgr) << "Failed to create activation token resource";
       return;
   }

2. 资源管理：

   • XdgActivationTokenV1 在析构时调用 deleteLater()，但这种方式可能导致资源释放时机不确定
   • 建议：考虑使用智能指针或明确的生命周期管理机制

3. 代码重复：

   • 头文件中有两处 // XdgActivationTokenV1 的注释，这是重复的
   • 建议：删除重复的注释

2. 性能优化

1. 内存使用：

   • m_outerActivation 在构造函数中创建，但没有看到在合适的地方释放
   • 建议：在 XdgActivationManager 析构函数中添加 delete m_outerActivation;

2. 信号连接：

   • connect(m_outerActivation, &ds::XdgActivation::tokenReady, this, &XdgActivationManager::onTokenReady); 的连接没有指定连接类型
   • 建议：显式指定连接类型，如 Qt::DirectConnection 或 Qt::QueuedConnection

3. 代码安全

1. 输入验证：

   • xdg_activation_token_v1_set_app_id 中直接使用传入的 app_id，没有进行有效性验证
   • 建议：添加空字符串检查或长度限制

2. 线程安全：

   • m_pendingToken 的访问没有同步保护
   • 建议：使用 QMutex 或 QMutexLocker 保护共享数据

3. 资源释放：

   • Wayland 资源的释放没有完整的错误处理机制
   • 建议：在关键资源操作处添加错误检查和适当的清理逻辑

4. 其他建议

1. 日志记录：

   • 日志记录不够全面，建议在关键操作点添加更多调试信息
   • 例如：在 requestOuterToken 中添加日志记录

2. 代码注释：

   • 部分公共方法的文档注释不完整
   • 建议：为所有公共接口添加详细的文档注释，说明参数、返回值和可能的异常情况

3. 常量使用：

   • 魔法数字如 120 (pluginScale) 应该定义为命名常量
   • 建议：添加 static constexpr 常量定义

总结

代码整体实现合理，但在错误处理、资源管理和线程安全方面还有改进空间。建议按照上述建议进行优化，以提高代码的健壮性和可维护性。特别是要加强错误处理和资源管理，这对于系统级组件来说非常重要。