Remove .is checks in UniformUtils

[LeviPesin]

Related issue: #24199 (comment)

Description

Remove .is checks in the UniformUtils, instead use "check by method" (duck typing).

[LeviPesin]

Also, Array.isArray( property ) can be also changed to property.slice !== undefined for more readability... Should I do that?

So it is something like this:

if ( property && property.clone !== undefined ) {

    clonedProperty = property.clone();

} else if ( property && property.slice !== undefined ) {

    clonedProperty = property.slice();

} else {

    clonedProperty = property;

}

And if I remember correctly, similar code is actually used in other places of the Three.js codebase... (I have found only https://github.com/mrdoob/three.js/blob/dev/src/core/Uniform.js#L18).

[donmccurdy]

It's nice that this change is shorter than the original, so I'm OK with this PR, but let's not remove .is properties generally please. See #24199 (comment)

Array.isArray( property ) can be also changed to property.slice !== undefined for more readability... Should I do that?

Personally I find Array.isArray( property ) easier to read.

[mrdoob]

This seems better: #24199 (comment)