Bug Asset delete - big performance overhead

I have noticed that every first asset deletion in my project (which is quite big) takes around 10s, but after that it runs smoothly. I have profiled it and it says that it is because of ProjectWindowUtil.AnyTargetMaterialHasChildren method. It made me suspicious, because the asset that I have deleted wasn't a material.

Take a look at the code in repository, lines 995-1015:
https://github.com/Unity-Technologi...ditor/Mono/ProjectWindow/ProjectWindowUtil.cs

Fist it checks whether any deleted asset is a material and stores it to "containsMaterial" flag.
But after that, it makes a bitwise 'and' operation '&' while it is obvious that it should be a logical 'and' '&&'.

containsMaterial &= AnyTargetMaterialHasChildren(paths);

Changing it to:

containsMaterial = containsMaterial && AnyTargetMaterialHasChildren(paths);

Would save our team 10s every time we delete the first asset after opening editor, which would be another small step to make the editor responsive.

 

You could make a bug report for this with this post attatched. Make sure to upload a project with the performance issue not fixed