[v3.8.6] Optimize code size again.

[dumganhar]

Re: #18056

Changelog

---

Continuous Integration

This pull request:

• needs automatic test cases check.

  Manual trigger with @cocos-robot run test cases afterward.

• does not change any runtime related code or build configuration

  If any reviewer thinks the CI checks are needed, please uncheck this option, then close and reopen the issue.

---

Compatibility Check

This pull request:

• changes public API, and have ensured backward compatibility with deprecated features.
• affects platform compatibility, e.g. system version, browser version, platform sdk version, platform toolchain, language version, hardware compatibility etc.
• affects file structure of the build package or build configuration which requires user project upgrade.
• introduces breaking changes, please list all changes, affected features and the scope of violation.

[dumganhar]

The static method Pass.getTypeFromHandle could not be optimized by terser but getTypeFromHandle could.

[github-actions]

👍 Package size ⤵ -1477 bytes, old: 5241693, new: 5240216

Interface Check Report

! WARNING this pull request has changed these public interfaces:

@@ -1137,32 +1137,10 @@
          */
         ensureMeshData(): void;
         destroy(): boolean;
         /**
-         * Calculate UV for sliced
          * @deprecated since v3.5.0, this is an engine private interface that will be removed in the future.
          */
-        _calculateSlicedUV(): void;
-        /**
-         * Calculate UV
-         * @deprecated since v3.5.0, this is an engine private interface that will be removed in the future.
-         */
-        _calculateUV(): void;
-        /**
-         * @deprecated since v3.5.0, this is an engine private interface that will be removed in the future.
-         */
-        _setDynamicAtlasFrame(frame: any): void;
-        /**
-         * @deprecated since v3.5.0, this is an engine private interface that will be removed in the future.
-         */
-        _resetDynamicAtlasFrame(): void;
-        /**
-         * @deprecated since v3.5.0, this is an engine private interface that will be removed in the future.
-         */
-        _checkPackable(): void;
-        /**
-         * @deprecated since v3.5.0, this is an engine private interface that will be removed in the future.
-         */
         _serialize(ctxForExporting: any): any;
         /**
          * @deprecated since v3.5.0, this is an engine private interface that will be removed in the future.
          */
@@ -11191,9 +11169,9 @@
                 /**
                  * Editor specific gizmo camera logic
                  * @internal
                  */
-                syncCameraEditor(camera: any): void;
+                syncCameraEditor(camera: Camera): void;
                 /**
                  * @en Update the camera's builtin matrixes
                  * @zh 更新相机的视图、投影等矩阵
                  * @param forceUpdate If force update, then dirty flag will be ignored

[minggo]

It seems these constants are not used.

[dumganhar]

I just moved it from bellow lines. It's better to put constants at the beginning.
And they are used in this file, see

if (WECHAT || WECHAT_MINI_PROGRAM || BYTEDANCE) {
                    this.__isWebIOS14OrIPadOS14Env = (sys.os === OS.IOS || sys.os === OS.OSX) && GameGlobal?.isIOSHighPerformanceMode
            && /(OS 1((4\.[0-9])|(5\.[0-3])))|(Version\/1((4\.[0-9])|(5\.[0-3])))/.test(window.navigator.userAgent);
                } else {
                    this.__isWebIOS14OrIPadOS14Env = (sys.os === OS.IOS || sys.os === OS.OSX) && systemInfo.isBrowser
            && /(OS 14)|(Version\/14)/.test(window.navigator.userAgent);
                }