[rfc][icedtea-web] mov repeated trusAll/none/shouldPromt/isHeadless to one palce
Jiri Vanek
jvanek at redhat.com
Wed Jun 17 09:45:10 UTC 2015
On 06/16/2015 05:32 PM, Jiri Vanek wrote:
> Unluckily the test needed a lot of work, because now the message is always posted to queue.
>
>
> Otherwise the patch is pretty straightworward.
>
> Aslo I have encountered some issues with reprodcuers:
>
> FAILED: testCaching(AbsolutePathsAndQueryStrings) expected:<...ipHttpPathParams.jar[]> but
> was:<...ipHttpPathParams.jar[.i_abcd]>
>
> - query patch is guilty
>
> Those three are very suspicious.
> FAILED: ReplaceSecurityManagerLunch1(ReplaceSecurityManagerTest) ReplaceSecurityManagerLunch1 should
> not be terminated, but was
> FAILED: ApplicationJNLPLocalTest(CodeBaseManifestEntrySignedNotMatching) should contain `*** APPLET
> FINISHED ***`
> FAILED: AppletJNLPRLocalTest(CodeBaseManifestEntrySignedNotMatching) should contain `*** APPLET
> FINISHED ***`
>
> those three seems to be ths patch related....hmmm
> FAILED: testJavawsJNLP - opera(PackGZipTest) stdout should contain init, but it didnt.
> FAILED: testJavawsJNLP - midori(PackGZipTest) stdout should contain init, but it didnt.
> FAILED: testJavawsJNLP - epiphany(PackGZipTest) stdout should contain init, but it didnt.
>
>
> I needed to temrinate testun so I have not run them all. But seem sthat soemthing already went wrong
> in alst few pushes. (will run them today overnight)
>
> J.
>
>
>
Looks like I overlooked
private void checkTrustWithUser() throws LaunchException {
if (JNLPRuntime.isTrustNone()) {
if (!securityDelegate.getRunInSandbox()) {
setRunInSandbox();
}
return;
}
if (JNLPRuntime.isTrustAll() || securityDelegate.getRunInSandbox()) {
return;
}
if (getSigningState() == SigningState.FULL && jcv.isFullySigned() &&
!jcv.getAlreadyTrustPublisher()) {
jcv.checkTrustWithUser(securityDelegate, file);
}
}
=>
private void checkTrustWithUser() throws LaunchException {
if (getSigningState() == SigningState.FULL && jcv.isFullySigned() &&
!jcv.getAlreadyTrustPublisher()) {
jcv.checkTrustWithUser(securityDelegate, file);
}
}
ok?
More information about the distro-pkg-dev
mailing list