chore: Remove the decorator of a part of the program

justforlxz commented 5 months ago

Desktop and Launchpad do not yet support LayerShell

Log:

deepin-ci-robot commented 5 months ago

deepin pr auto review

关键摘要:

在onSurfaceModeChanged函数中，增加了对特定客户端名称的直接处理，而不是依赖于XdgDecorationManager.Client模式。这可能违反了原有的逻辑，需要确认这是否是预期的行为。
重复代码：在onSurfaceModeChanged和Component.onCompleted中，对特定客户端名称的处理逻辑是相同的，可以考虑提取成一个单独的函数以减少重复。
性能问题：每次onSurfaceModeChanged或Component.onCompleted调用时，都会执行Helper.clientName的调用，这可能会导致不必要的性能开销。如果这些名称在应用程序中不会频繁变化，可以考虑缓存结果。

是否建议立即修改:

是，需要确认对特定客户端名称的处理逻辑是否正确，并考虑是否有更好的方式来处理这类情况。
是，重复代码的提取可以立即进行，以避免不必要的性能开销。
是，如果客户端名称在应用程序中不会频繁变化，缓存Helper.clientName的结果可以提高性能。

deepin-ci-robot commented 5 months ago

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: Groveer, justforlxz

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

Needs approval from an approver in each of these files: - **[debian/deepin/OWNERS](https://github.com/linuxdeepin/treeland/blob/master/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

linuxdeepin / treeland

chore: Remove the decorator of a part of the program #335

deepin pr auto review