Mvc: Unskip error page functional test

[kichalla]

Following commit skipped the functional test as this was blocking the mirror and is potentially a behavior change to which we need to react.

Reference files(Note: Rename the file extensions to .html):
NewErrorPage.zip
OldErrorPage.zip

cc @pranavkm @mikeharder

[kichalla]

Commit: 106b34867d7b7bba45e36ea4385e68311cad3b13

[NTaylorMullen]

08965550addf660ad1ec43d32c93987eb205192e

[pranavkm]

👍

[kichalla]

It appears this issue is resurfacing again in the mirror, probably because the corefx team merged their commits from preview1(where it regressed) to dev(where it was working when @NTaylorMullen unskipped this).

[Eilon]

@kichalla - should this issue be in preview2 then?

[kichalla]

Yes, this would be for preview2. Changed the milestone now. I re-opened this issue: https://github.com/aspnet/Diagnostics/issues/372

Commit to revert later: d916cfeaddbda9f17da1998e42af85ec8c8dc3f4

[NTaylorMullen]

@kichalla Do the right people know that this was regressed over on the coreclr team?

[kichalla]

@NTaylorMullen I created the following issue now: https://github.com/dotnet/corefx/issues/19838

[kichalla]

This has been fixed. Reverted change 3c7a2dc847b4c4fa8d0d83bc0bcb5308f2b8bca4

[NTaylorMullen]

Re-skipping for desktop applications. https://github.com/dotnet/corefx/issues/19838

[NTaylorMullen]

Working around this by adding:

<DebugType Condition="'$(TargetFrameworkIdentifier)'=='.NETFramework'">Full</DebugType>