Guthries excellent recommendation

Signed-off-by: mrieggeramzn <mriegger@amazon.com>

Gems/Atom/Feature/Common/Code/Source/Decals/DecalTextureArrayFeatureProcessor.cpp

@@ -303,7 +303,9 @@ namespace AZ
 
             if (material.IsValid())
             {
-                AZ_Assert(m_decalData.GetData(handle.GetIndex()).m_textureArrayIndex == DecalData::UnusedIndex, "Setting Material on a decal more than once is not currently supported.");
+                AZ_Assert(
+                    m_decalData.GetData(handle.GetIndex()).m_textureArrayIndex == DecalData::UnusedIndex || GetMaterialUsedByDecal(handle) == material,
+                    "Setting Material on a decal more than once is not currently supported.");
 
                 const auto iter = m_materialToTextureArrayLookupTable.find(material);
                 if (iter != m_materialToTextureArrayLookupTable.end())

Gems/AtomLyIntegration/CommonFeatures/Code/Source/Decals/EditorDecalComponent.cpp

@@ -15,11 +15,6 @@ namespace AZ
 {
     namespace Render
     {
-        static AZ::Data::Asset<AZ::RPI::MaterialAsset> QueueLoad(const AZ::Data::AssetId id)
-        {
-            return AZ::Data::AssetManager::Instance().GetAsset<AZ::RPI::MaterialAsset>(id, AZ::Data::AssetLoadBehavior::QueueLoad);
-        }
-
         EditorDecalComponent::EditorDecalComponent(const DecalComponentConfig& config)
             : BaseClass(config)
         {
@@ -92,26 +87,16 @@ namespace AZ
             AzFramework::EntityDebugDisplayEventBus::Handler::BusConnect(GetEntityId());
             AzToolsFramework::EditorComponentSelectionRequestsBus::Handler::BusConnect(GetEntityId());
             AzFramework::BoundsRequestBus::Handler::BusConnect(GetEntityId());
-            CacheMaterial();
         }
 
         void EditorDecalComponent::Deactivate()
         {
-            m_cachedMaterial = {};
             AzFramework::BoundsRequestBus::Handler::BusDisconnect();
             AzToolsFramework::EditorComponentSelectionRequestsBus::Handler::BusDisconnect();
             AzFramework::EntityDebugDisplayEventBus::Handler::BusDisconnect();
             BaseClass::Deactivate();
         }
 
-        void EditorDecalComponent::CacheMaterial()
-        {
-            DecalComponentConfig decalComponentConfig;
-            GetConfiguration(decalComponentConfig);
-            const auto& materialAsset = decalComponentConfig.m_materialAsset;
-            m_cachedMaterial = QueueLoad(materialAsset.GetId());
-        }
-
         AZ::Transform EditorDecalComponent::GetWorldTransform() const
         {
             AZ::Transform transform = AZ::Transform::CreateIdentity();
@@ -207,8 +192,7 @@ namespace AZ
 
         u32 EditorDecalComponent::OnConfigurationChanged()
         {
-            BaseClass::OnConfigurationChanged();
-            CacheMaterial();
+            m_controller.ConfigurationChanged();
             return Edit::PropertyRefreshLevels::AttributesAndValues;
         }
 

Gems/AtomLyIntegration/CommonFeatures/Code/Source/Decals/EditorDecalComponent.h

@@ -64,12 +64,6 @@ namespace AZ
 
             //! EditorRenderComponentAdapter overrides ...
             u32 OnConfigurationChanged() override;
-
-            // Hold onto the material for the lifespan of the EditorDecalComponent to smooth out performance. This is so we can avoid
-            // duplicate loads as the DecalTextureArrayFeatureProcessor will unload the materials after texture packing.
-            void CacheMaterial();
-
-            AZ::Data::Asset<Data::AssetData> m_cachedMaterial;
         };
     } // namespace Render
 } // namespace AZ