Revert footstep trace to kismet version

[muchcharles]

There were compile issues in the change to reduce allocations in
footstep traces. Revert back to the kismet version, but use MeshOwner
as the world context object so that bIgnoreSelf can ignore it. That
way MeshOwner->Children can be used directly as the rest of the ignore
list, avoiding the allocations.

[dyanikoglu]

What kind of compile issue we're talking about? As far as I know, the code compiles fine in current state?

[muchcharles]

Someone in discord, building on Mac.

They got:

'ENABLE_DRAW_DEBUG' is not defined, evaluates to 0 [-Werror,-Wundef]

Which was fixable with adding:

#include "DrawDebugHelpers.h"

But then they still got:

In file included from /Users/username/Documents/Unreal Projects/MWORLD/Plugins/ALS-Community-main/Intermediate/Build/Mac/x86_64/UE4Editor/Development/ALSV4_CPP/Module.ALSV4_CPP.cpp:14:
0>/Users/username/Documents/Unreal Projects/MWORLD/Plugins/ALS-Community-main/Source/ALSV4_CPP/Private/Character/Animation/Notify/ALSAnimNotifyFootstep.cpp:63:101: Error  : use of undeclared identifier 'TraceColor'
                        DrawDebugLineTraceSingle(MeshComp->GetWorld(), FootLocation, TraceEnd, DrawDebugType, bHit, Hit, TraceColor, TraceHitColor, DrawTime);
                                                                                                                         ^
0>/Users/username/Documents/Unreal Projects/MWORLD/Plugins/ALS-Community-main/Source/ALSV4_CPP/Private/Character/Animation/Notify/ALSAnimNotifyFootstep.cpp:63:113: Error  : use of undeclared identifier 'TraceHitColor'
                        DrawDebugLineTraceSingle(MeshComp->GetWorld(), FootLocation, TraceEnd, DrawDebugType, bHit, Hit, TraceColor, TraceHitColor, DrawTime);
                                                                                                                                     ^
0>/Users/username/Documents/Unreal Projects/MWORLD/Plugins/ALS-Community-main/Source/ALSV4_CPP/Private/Character/Animation/Notify/ALSAnimNotifyFootstep.cpp:63:128: Error  : use of undeclared identifier 'DrawTime'
                        DrawDebugLineTraceSingle(MeshComp->GetWorld(), FootLocation, TraceEnd, DrawDebugType, bHit, Hit, TraceColor, TraceHitColor, DrawTime);

I couldn't find where those are actually defined (the only place seemed to be defined privately/impl in CollisionDebugDrawing.cpp in namespace CollisionDebugDrawing).

[muchcharles]

And alternate change is probably to just set the undefined variables to the literals FLinearColor::Red, FLiniearColor::Green, and 5.0f, which was what the kismet library is defaulting them to. A benefit of the non-kismet call is it has a specific trace tag that could be used for profiling.

When I looked into it more I realized the kismet bIgnoreSelf uses the world context object as self, and by changing that it was then ok to just pass the ignore actors list as the Children array directly.