已合并
feat(skparagraph): dump shaped glyph IDs in GetDumpInfo #2366
feat(skparagraph): dump shaped glyph IDs in GetDumpInfo #2366
已合并
liufei225创建于 4 天前
liufei225
4 天前

一、内容说明(相关的Issue)

#1848 feat(skparagraph): dump shaped glyph IDs in GetDumpInfo

二、建议测试周期和提测地址

建议测试完成时间:xxxx.xx.xx
投产上线时间:xxxx.xx.xx
提测地址:CI环境
测试账号:无

三、变更内容

  • 3.1 关联PR列表

本PR:feat/dump-glyph-ids → master

  • 3.2 数据库和部署说明

    1. 常规更新
    2. 重启unicorn
    3. 重启sidekiq
    4. 迁移任务:无
    5. rake脚本:无
  • 3.4 其他技术优化内容(做了什么,变更了什么)

    • 在 ParagraphImpl::GetDumpInfo 中按每个 Run 追加输出 shaping 结果的完整 glyph ID 序列(SkGlyphID),与该 Run 的文本范围一一对应;该 glyph ID 即 Run::copyTo 原样 memcpy 进绘制用 SkTextBlob 的同一批字形,可用于与 RS 侧实际渲染字形对账。
    • 顺带精简 GetDumpInfo(合并相邻流式输出、折叠 typeface if/else、为 block.fStyle 取别名),打印内容保持不变。
    • 代码位于 ENABLE_TEXT_ENHANCE 宏保护范围内。
  • 3.5 废弃通知(什么字段、方法弃用?)

  • 3.6 后向不兼容变更(是否有无法向后兼容的变更?)

四、研发自测点(自测哪些?冒烟用例全部自测?)

自测测试结论:对比修改前后 GetDumpInfo 输出,字段、顺序、分隔符逐项一致,仅新增 glyphs:[...] 段;输出无回归。

五、测试关注点(需要提醒QA重点关注的、可能会忽略的地方)

检查点:仅影响开启 ENABLE_TEXT_ENHANCE 时的段落 dump 文本,不影响文本布局与渲染结果。

需求名称 是否影响xx公共模块 是否需要xx功能 需求升级是否依赖其他子产品
dump shaped glyph IDs 不需要 不需要

接口测试:无新增/变更接口

性能测试:dump 频繁调用时需关注字符串拼接开销(段落较长时 glyph 序列较大)

并发测试:无

其他:无

likedislike
Pull Request已成功合入, 合并人@openharmony_ci
(感谢 liufei225 的贡献)
openharmony_ciopenharmony_ci成员
4 天前 添加了label:waiting_on_author
openharmony_ci
openharmony_ci成员
4 天前 评论:

感谢提交 Pull Requests!如果您提交的PR已经开发完毕,请评论 "start build" 触发门禁,更多交互操作,请访问OpenHarmony社区支持命令清单。如果需要调整订阅PR、Issue的变更状态,请访问订阅链接


Thanks for submitting the pull request. If your Pull Request has already been developed, you can leave a "start build" comment to trigger the gated system. For more commands, please visit OpenHarmony Command List. If you need to change the subscription of a Pull Request or Issue, please visit the link.

likedislike
openharmony_ciopenharmony_ci成员
4 天前 添加了label:dco检查成功
Lliufei225
4 天前 修改了pull request 的描述
liufei225
4 天前 评论:

start build

likedislike
openharmony_ci
openharmony_ci成员
4 天前 评论:

首次触发
门禁构建开始,包含静态检查、代码编译和测试【dayu200_tdd编译, ohos-host_mini_tdd编译, dayu600_7885测试, dayu200编译, mac-sdk编译, part_compile编译, ohos-sdk编译, dayu200测试, master_inner_build编译, x86_64_virt编译, dayu600_7885编译】,预计在60分钟内完成,门禁结果会同步发送到注册邮箱。您可以通过如下链接跟踪门禁进展:http://dcp.openharmony.cn/workbench/cicd/detail/6a73fe0764650f998b0e8eda/runlist

likedislike
openharmony_ciopenharmony_ci成员
4 天前 添加了label:编译成功
openharmony_ciopenharmony_ci成员
4 天前 添加了label:静态检查成功
openharmony_ciopenharmony_ci成员
4 天前 添加了label:冒烟测试成功
openharmony_ciopenharmony_ci成员
4 天前 通过测试
openharmony_ci
openharmony_ci成员
4 天前 评论:

代码门禁通过
您可以通过如下链接查看门禁报告:http://dcp.openharmony.cn/workbench/cicd/detail/6a73fe0764650f998b0e8eda/runlist

静态检查:

# check type result report
1 codeCheck pass >>>

编译测试:
# Device build result test result package
1 ohos-sdk success NA >>>
2 dayu200 success success >>>
3 dayu200_tdd success NA >>>
4 part_compile success(IGNORE) NA >>>
5 master_inner_build failed(IGNORE)(联合门禁失败) NA >>>
6 mac-sdk success NA >>>
7 ohos-host_mini_tdd success NA >>>
8 dayu600_7885 success success >>>
9 x86_64_virt success NA >>>

likedislike
openharmony_ciopenharmony_ci成员
4 天前 删除了label:waiting_on_author
openharmony_ciopenharmony_ci成员
4 天前 添加了label:waiting_for_review
openharmony_ci
openharmony_ci成员
4 天前 评论:

您好,Committer @liumingxiang1 @mk_gc @aslklw @b200200 @kroswang @wyykak @gmiao522 ,请分配检视人员检视该PR,可以通过命令"assign [@someone_id]"分配检视人员,也可以直接评论"assign"分配给自己进行检视。


Hello, Committer @liumingxiang1 @mk_gc @aslklw @b200200 @kroswang @wyykak @gmiao522 . Please assign someone to review the PR. You can assign a reviewer by using the command "assign [@someone_id]", or you can comment "assign" to review the PR by yourself.

likedislike
Kejie Peng
Kejie Peng4 天前进行代码检视1
m133/modules/skparagraph/src/ParagraphImpl.cpp
@@ -3864,0 +3830,4 @@
3830+ << (typeface ? typeface->GetFamilyName() : "null") << ",id:"
3831+ << (typeface ? typeface->GetUniqueID() : 0) << ",";
3832+ paragraphInfo << "glyphs:[";
3833+ for (SkGlyphID g : run.glyphs()) {
Kejie Peng
Kejie Peng4 天前评论:

新增TDD防护

likedislike
GMiao522GMiao522成员
4 天前 通过审查
openharmony_ciopenharmony_ci成员
4 天前 关闭了关联的issue
openharmony_ciopenharmony_ci成员
4 天前 合入了pull request,合并节点 SHA:63f7b48032c0a54aeeb5444b3fd0bfe05cd230a0
openharmony_ciopenharmony_ci成员
4 天前 删除了label:waiting_for_review
openharmony_ciopenharmony_ci成员
4 天前 添加了label:merged