This moves setting baseShader.hex_version and baseShader.functionLength to
shader_get_registers_used(), where it's more appropriate.

The find_builtin_prop internal routine in dispex.c implements a binary
search with the requirement that a class properties array be in
ascending order, so this fix allows String's properties array to
satisfy this requirement and allows the tests for String.sub() to
pass.

As pointed out by Dan Kegel.

This is consistent with other uses of struct color_fixup_desc.

This fixes a regression introduced by commit
89139b75.

crypt32: Allow setting CERT_KEY_PROV_HANDLE_PROP_ID if CERT_KEY_CONTEXT_PROP_ID has not previously been set.