cherry-pick commits of v20

[18202781743]

• fix: 修复DTK搜索框图标颜色不跟随主题变化
• fix: 没有匹配最佳尺寸的图标

[18202781743]

/topic dtk-cpv20

[deepin-ci-robot]

Add topic: dtk-cpv20 successed.

[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

我来对这段代码进行审查：

1. 代码逻辑审查：

• 新增的 compositedPixmap 函数很好地封装了图标合成逻辑，提高了代码复用性
• perfectEntryForSize 函数的添加优化了图标尺寸匹配逻辑，先尝试精确匹配再回退到普通匹配
• 在 paint 和 virtual_hook 中都正确使用了新的合成逻辑

2. 代码质量改进建议：

• compositedPixmap 函数中的 pm 参数建议使用 const 引用：const QPixmap &pm，因为函数内部不会修改原始 pixmap
• perfectEntryForSize 函数中的 scale 参数默认值 1.0 应该显式写出：qreal scale = 1.0
• 建议在 compositedPixmap 函数开始处添加空指针检查：if (!entry) return pm;

3. 性能优化建议：

• perfectEntryForSize 函数可以缓存结果，避免重复计算
• compositedPixmap 中的 QPainter pa(&pm) 创建可能带来性能开销，考虑复用 painter 参数（如果存在）

4. 安全性改进：

• static_cast<ImageEntry *>(entry) 的强制类型转换存在风险，建议添加类型检查：

auto* imageEntry = dynamic_cast<ImageEntry*>(entry);
if (!imageEntry) return pm;

5. 代码规范建议：

• 函数命名遵循驼峰命名法，但 compositedPixmap 建议改为 compositePixmap 更符合 Qt 命名风格
• 建议为新增函数添加详细的文档注释，说明参数含义和返回值

6. 其他建议：

• 在 paint 函数中，背景图绘制逻辑可以考虑也封装成独立函数，提高代码可维护性
• 建议添加单元测试用例，特别是针对新增的 perfectEntryForSize 和 compositedPixmap 函数

总体来说，这次改动提高了代码的复用性和可维护性，但还有上述几点可以进一步改进。