[8] Request for review: 8005414: Removing fix for JDK-6500133

Sean Mullan sean.mullan at oracle.com
Mon Jan 14 14:18:47 PST 2013


On 01/14/2013 05:00 PM, Bernd Eckenfels wrote:
> Am 14.01.2013, 22:52 Uhr, schrieb Sean Mullan <sean.mullan at oracle.com>:
>
>> Looks good to me.
>
> The patch removes all unit tests for this class it seems. There should be
> one parseable URL and if you intent to reject illegal content you can use
> the test vectors to test for the expected exception.

Good point. Jason, can you keep the test but readjust to test for an 
expected exception?

Thanks,
Sean


More information about the security-dev mailing list