Patching mechanism in CFE doesn't allow to patch members with members of a different kind

[chloestefantsova]

Currently both factories and redirecting factories are represented as Procedures in Kernel. The latter has special bodies that encode the redirection information, that is, the target constructor and the type arguments. There's also a specialized subclass of Member to represent redirecting factories, namely RedirectingFactoryConstructor.

Now, some platform-specific patches to core libraries patch non-redirecting factories with redirecting factories. It does work currently, as both are represented as Procedures, but it doesn't work when we try to change the representation of redirecting factories to the specialized one.

The patching mechanism needs to be update to work in that case.

[mit-mit]

Punting to 2.1 per comment in #29989 (comment) that this is not critical for stable

[kmillikin]

This is unrelated to the Dart 2.1 release.

[vsmenon]

Note, this is blocking another 2.3 issue.

[dgrove]

Specifically, this blocks #33041

[a-siva]

I think we can punt this to 2.4 we realize the front end team is busy and we have lived with #33041 for almost a year now.