fix: thumbnail sidebar lag for XPS documents

[LiHua000]

Batch model updates and reduce mutex contention to improve scrolling performance.

Log: as title

Bug:https://pms.uniontech.com/bug-view-343541.html

[deepin-ci-robot]

deepin pr auto review

我来对这三个文件的代码修改进行详细审查：

1. XpsDocumentAdapter.cpp 的修改：

优点：

• 减小了锁的粒度，将获取 logicalSize 的操作移到锁内，其他操作移到锁外，有利于提高并发性能
• 代码结构更清晰，使用大括号明确区分了临界区的范围

建议：

• 在获取 logicalSize 后应该立即验证其有效性，避免无效数据被使用
• 可以考虑添加对 width 和 height 参数的有效性检查

2. SideBarImageViewModel.cpp 的修改：

优点：

• 引入了批量更新机制，通过定时器合并多个更新请求，减少了频繁的UI刷新
• 使用 QSet 来管理待更新页面，避免了重复
• 对更新项进行排序，保证了更新的有序性

建议：

• 批量更新的时间间隔(100ms)可能需要根据实际性能测试结果进行调整
• 可以考虑添加最大待更新数量限制，防止内存占用过大
• 在 onBatchUpdateTimer 中添加错误处理机制

3. SideBarImageViewModel.h 的修改：

优点：

• 头文件中正确声明了新增的成员变量和槽函数
• 使用了适当的前向声明，保持了头文件的简洁性

建议：

• 可以考虑为 m_batchUpdateTimer 添加注释说明其用途
• m_pendingUpdatePages 可以考虑使用 QSet::const_iterator 来提高遍历效率

整体建议：

1. 性能优化：

• XpsDocumentAdapter 中的锁粒度优化很好，但要注意确保线程安全
• SideBarImageViewModel 的批量更新机制很好，但建议进行性能测试以确定最佳的时间间隔

2. 代码安全：

• 添加更多的参数有效性检查
• 考虑添加异常处理机制

3. 代码质量：

• 建议添加更详细的注释说明批量更新机制的工作原理
• 考虑添加单元测试来验证新的批量更新机制

4. 可维护性：

• 建议将批量更新的时间间隔定义为可配置的常量
• 考虑添加性能监控日志，以便后续优化

这些修改总体上是积极的，特别是在性能优化方面。但建议进行充分的测试，确保新的实现不会引入新的问题。

[deepin-ci-robot]

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: LiHua000, lzwind

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

[LiHua000]

/merge