com.unity.postprocessing@3.2.1

## [3.2.1] - 2022-01-12

### Fixed
- Fixed missing XR warnings for XR non-friendly effects when using XR plugins (case 1328062)
- Fixed rendering artifacts when depth buffer is used after screen space reflections pass on iOS and M1 (case 1341052)
- Fixed compilation error when built-in VR module is disabled (case 1389160)
- Fixed bug where alpha could not be used with HDR on mobile (case 1387848). HDR texture format is now set from Graphics Tier Settings when using Unity 2019.3 or newer.
- Fixed incorrect Screen-space Reflections rendering on OpenGL platforms (case 1368370)

Author: Unity Technologies

CHANGELOG.md

@@ -4,6 +4,15 @@ All notable changes to this package will be documented in this file.
 The format is based on [Keep a Changelog](http://keepachangelog.com/en/1.0.0/)
 and this project adheres to [Semantic Versioning](http://semver.org/spec/v2.0.0.html).
 
+## [3.2.1] - 2022-01-12
+
+### Fixed
+- Fixed missing XR warnings for XR non-friendly effects when using XR plugins (case 1328062)
+- Fixed rendering artifacts when depth buffer is used after screen space reflections pass on iOS and M1 (case 1341052)
+- Fixed compilation error when built-in VR module is disabled (case 1389160)
+- Fixed bug where alpha could not be used with HDR on mobile (case 1387848). HDR texture format is now set from Graphics Tier Settings when using Unity 2019.3 or newer.
+- Fixed incorrect Screen-space Reflections rendering on OpenGL platforms (case 1368370)
+
 ## [3.2.0] - 2021-11-15
 
 ### Fixed

PostProcessing/Editor/Effects/BloomEditor.cs

@@ -54,7 +54,7 @@ public override void OnInspectorGUI()
             PropertyField(m_DirtTexture);
             PropertyField(m_DirtIntensity);
 
-            if (RuntimeUtilities.isVREnabled)
+            if (EditorUtilities.isVREnabled)
             {
                 if ((m_DirtIntensity.overrideState.boolValue && m_DirtIntensity.value.floatValue > 0f)
                     || (m_DirtTexture.overrideState.boolValue && m_DirtTexture.value.objectReferenceValue != null))

PostProcessing/Editor/Effects/LensDistortionEditor.cs

@@ -7,7 +7,7 @@ internal sealed class LensDistortionEditor : DefaultPostProcessEffectEditor
     {
         public override void OnInspectorGUI()
         {
-            if (RuntimeUtilities.isVREnabled)
+            if (EditorUtilities.isVREnabled)
                 EditorGUILayout.HelpBox("Lens Distortion is available only for non-stereo cameras.", MessageType.Warning);
 
             base.OnInspectorGUI();

PostProcessing/Editor/Effects/MotionBlurEditor.cs

@@ -7,7 +7,7 @@ internal sealed class MotionBlurEditor : DefaultPostProcessEffectEditor
     {
         public override void OnInspectorGUI()
         {
-            if (RuntimeUtilities.isVREnabled)
+            if (EditorUtilities.isVREnabled)
                 EditorGUILayout.HelpBox("Motion Blur is available only for non-stereo cameras.", MessageType.Warning);
 
             base.OnInspectorGUI();

PostProcessing/Editor/Unity.Postprocessing.Editor.asmdef

@@ -1,9 +1,11 @@
 {
     "name": "Unity.Postprocessing.Editor",
+    "rootNamespace": "",
     "references": [
-        "Unity.Postprocessing.Runtime"
+        "Unity.Postprocessing.Runtime",
+        "Unity.XR.Management",
+        "Unity.XR.Management.Editor"
     ],
-    "optionalUnityReferences": [],
     "includePlatforms": [
         "Editor"
     ],
@@ -12,5 +14,23 @@
     "overrideReferences": false,
     "precompiledReferences": [],
     "autoReferenced": true,
-    "defineConstraints": []
-}
+    "defineConstraints": [],
+    "versionDefines": [
+        {
+            "name": "com.unity.xr.management",
+            "expression": "4.0.1",
+            "define": "XR_MANAGEMENT_4_0_1_OR_NEWER"
+        },
+        {
+            "name": "com.unity.modules.vr",
+            "expression": "1.0.0",
+            "define": "ENABLE_VR_MODULE"
+        },
+        {
+            "name": "com.unity.modules.xr",
+            "expression": "1.0.0",
+            "define": "ENABLE_XR_MODULE"
+        }
+    ],
+    "noEngineReferences": false
+}

PostProcessing/Editor/Utils/EditorUtilities.cs

@@ -5,6 +5,10 @@
 using UnityEngine.Assertions;
 using UnityEngine.Rendering.PostProcessing;
 
+#if XR_MANAGEMENT_4_0_1_OR_NEWER
+using UnityEditor.XR.Management;
+#endif
+
 namespace UnityEditor.Rendering.PostProcessing
 {
     /// <summary>
@@ -360,5 +364,28 @@ static bool CanPaste(PostProcessEffectSettings target)
             return s_ClipboardContent != null
                 && s_ClipboardContent.GetType() == target.GetType();
         }
+
+        internal static bool isVREnabled
+        {
+            get
+            {
+#if ENABLE_VR_MODULE && ENABLE_VR
+#if ENABLE_XR_MODULE && XR_MANAGEMENT_4_0_1_OR_NEWER
+                // If XR manager extension is available, we can query it to know if any XR extension is currently active
+                var buildTargetSettings = XRGeneralSettingsPerBuildTarget.XRGeneralSettingsForBuildTarget(BuildTargetGroup.Standalone);
+                return (buildTargetSettings != null && buildTargetSettings.AssignedSettings != null &&
+                        buildTargetSettings.AssignedSettings.activeLoaders.Count > 0);
+#elif !UNITY_2020_1_OR_NEWER
+                // This will only work with 2019.3 and older since it rely on the old VR module
+                return UnityEditorInternal.VR.VREditor.GetVREnabledOnTargetGroup(BuildPipeline.GetBuildTargetGroup(EditorUserBuildSettings.activeBuildTarget));
+#else
+                // If we reach this code-path, it means we can't really detect if VR/XR is active in the Editor, so return false
+                return false;
+#endif
+#else
+                return false;
+#endif
+            }
+        }
     }
 }

PostProcessing/Runtime/Effects/ScreenSpaceReflections.cs

@@ -260,7 +260,6 @@ public override void Render(PostProcessRenderContext context)
 
             sheet.properties.SetMatrix(ShaderIDs.ViewMatrix, context.camera.worldToCameraMatrix);
             sheet.properties.SetMatrix(ShaderIDs.InverseViewMatrix, context.camera.worldToCameraMatrix.inverse);
-            sheet.properties.SetMatrix(ShaderIDs.InverseProjectionMatrix, projectionMatrix.inverse);
             sheet.properties.SetMatrix(ShaderIDs.ScreenSpaceProjectionMatrix, screenSpaceProjectionMatrix);
             sheet.properties.SetVector(ShaderIDs.Params, new Vector4((float)settings.vignette.value, settings.distanceFade.value, settings.maximumMarchDistance.value, lodCount));
             sheet.properties.SetVector(ShaderIDs.Params2, new Vector4((float)context.width / (float)context.height, (float)size / (float)noiseTex.width, settings.thickness.value, settings.maximumIterationCount.value));
@@ -329,7 +328,7 @@ public override void Render(PostProcessRenderContext context)
                 cmd.ReleaseTemporaryRT(m_MipIDs[i]);
 
             sheet.properties.SetTexture(ShaderIDs.Resolve, m_Resolve);
-            cmd.BlitFullscreenTriangle(context.source, context.destination, sheet, (int)Pass.Composite);
+            cmd.BlitFullscreenTriangle(context.source, context.destination, sheet, (int)Pass.Composite, preserveDepth: true);
             cmd.EndSample("Screen-space Reflections");
         }
 

PostProcessing/Runtime/PostProcessDebugLayer.cs

@@ -328,8 +328,10 @@ internal void RenderSpecialOverlays(PostProcessRenderContext context)
                 var sheet = context.propertySheets.Get(context.resources.shaders.debugOverlays);
                 sheet.ClearKeywords();
 
+#if !UNITY_2022_1_OR_NEWER
                 if (context.camera.actualRenderingPath == RenderingPath.DeferredLighting)
                     sheet.EnableKeyword("SOURCE_GBUFFER");
+#endif
 
                 PushDebugOverlay(context.command, BuiltinRenderTextureType.None, sheet, 1);
             }

PostProcessing/Runtime/PostProcessLayer.cs

@@ -713,6 +713,7 @@ void OnPostRender()
                 {
                     // The camera must be reset on precull and post render to avoid issues with alpha when toggling TAA.
                     m_Camera.ResetProjectionMatrix();
+#if (ENABLE_VR_MODULE && ENABLE_VR)
                     if (m_CurrentContext.stereoActive)
                     {
                         if (RuntimeUtilities.isSinglePassStereoEnabled || m_Camera.stereoActiveEye == Camera.MonoOrStereoscopicEye.Right)
@@ -723,6 +724,7 @@ void OnPostRender()
                                 m_Camera.projectionMatrix = m_Camera.GetStereoProjectionMatrix(Camera.StereoscopicEye.Left);
                         }
                     }
+#endif
                 }
             }
         }

PostProcessing/Runtime/Utils/RuntimeUtilities.cs

@@ -865,16 +865,16 @@ public static RenderTextureFormat defaultHDRRenderTextureFormat
         {
             get
             {
-#if UNITY_ANDROID || UNITY_IPHONE || UNITY_TVOS || UNITY_SWITCH || UNITY_EDITOR
+#if !UNITY_2019_3_OR_NEWER && (UNITY_ANDROID || UNITY_IPHONE || UNITY_TVOS || UNITY_EDITOR)
                 RenderTextureFormat format = RenderTextureFormat.RGB111110Float;
 #if UNITY_EDITOR
                 var target = EditorUserBuildSettings.activeBuildTarget;
-                if (target != BuildTarget.Android && target != BuildTarget.iOS && target != BuildTarget.tvOS && target != BuildTarget.Switch)
+                if (target != BuildTarget.Android && target != BuildTarget.iOS && target != BuildTarget.tvOS)
                     return RenderTextureFormat.DefaultHDR;
 #endif // UNITY_EDITOR
                 if (format.IsSupported())
                     return format;
-#endif // UNITY_ANDROID || UNITY_IPHONE || UNITY_TVOS || UNITY_SWITCH || UNITY_EDITOR
+#endif // #if !UNITY_2019_3_OR_NEWER && (UNITY_ANDROID || UNITY_IPHONE || UNITY_TVOS || UNITY_EDITOR)
                 return RenderTextureFormat.DefaultHDR;
             }
         }