JDK-8199357 : Remove references to applets and Java Web Start from FX
  • Type: Bug
  • Component: javafx
  • Sub-Component: other
  • Affected Version: openjfx11
  • Priority: P3
  • Status: Resolved
  • Resolution: Fixed
  • Submitted: 2018-03-08
  • Updated: 2021-08-27
  • Resolved: 2018-04-13
The Version table provides details related to the release that this issue/RFE will be addressed.

Unresolved : Release in which this issue/RFE will be addressed.
Resolved: Release in which this issue/RFE has been resolved.
Fixed : Release in which this issue/RFE has been fixed. The release containing this fix may be available for download as an Early Access Release or a General Availability Release.

To download the current JDK release, click here.
Other
openjfx11Fixed
Related Reports
Blocks :  
JDK-8201538 - Remove implementation support for applets from JavaFX
Duplicate :  
JDK-8089972 - Application.getHostServices() javadoc is incomplete
Description
With the deprecation and impending removal of Java Plugin (for Applet support) and Java Web Start we should remove references to applets and Java Web Start from FX. This includes:

- Removing code that (reflectively) calls into the deploy stack
- Removing any apps or tests that use Java Web Start or Applets
- Removing any documentation that refers to running a JavaFX application via Web Start or in a browser.
Comments
Changeset: 345376202d2b Author: kcr Date: 2018-04-13 15:43 -0700 URL: http://hg.openjdk.java.net/openjfx/jfx-dev/rt/rev/345376202d2b 8199357: Remove references to applets and Java Web Start from FX Reviewed-by: prr, aghaisas
13-04-2018
OK. Thanks for confirming. +1 for the changes in this webrev.
13-04-2018
Thanks for the confirmation. I applied your patch & did search for "applet" and "browser" Using below commands threw up some entries in - javafx.graphics, javafx.web & jdk.packager modules. grep -inr "applet" * grep -inr "browser" * In my opinion, most of these entries cannot be eliminated. Can you please have a second look after running the above commands?
13-04-2018
Thanks for the additional checking. I ran a "grep -i" for "applet", "browser", "plugin", "deploy", "jnlp", "javaws", "webstart", and "web.start" just prior to sending the webrev. I didn't see any others that I thought should be removed as part of this bug. I do want to file a follow-on issue to remove the implementation support for applets in glass and the quantum toolkit, but that would be a "P4" cleanup task (and not targeted to 11). I filed JDK-8201538 to track the additional cleanup.
13-04-2018
[~aghaisas] Thanks for checking. No this does not need a CSR. The changes in the API docs are a reaction to removing applet and JNLP support from the Oracle JDK. Nothing changes from a functional point of view.
12-04-2018
This has modifications (mostly removals) in documentation - do we need a CSR for this changeset?
12-04-2018
+1 from me with the caveat I did not go grepping the source for anything you may have missed. -phil.
11-04-2018
I noticed one thing in LauncherImpl that I forgot to remove. The following variable is now unused: private static Method notifyMethod = null; I plan to delete this unused variable before pushing with the following delta-diff: diff --git a/modules/javafx.graphics/src/main/java/com/sun/javafx/application/LauncherImpl.java b/modules/javafx.graphics/src/main/java/com/sun/javafx/application/LauncherImpl.java --- a/modules/javafx.graphics/src/main/java/com/sun/javafx/application/LauncherImpl.java +++ b/modules/javafx.graphics/src/main/java/com/sun/javafx/application/LauncherImpl.java @@ -950,8 +950,6 @@ }); } - private static Method notifyMethod = null; - public static void notifyPreloader(Application app, final PreloaderNotification info) { if (launchCalled.get()) { // Standalone launcher mode
11-04-2018
Webrev: http://cr.openjdk.java.net/~kcr/8199357/webrev.00/ Most of the changes are docs changes, but reflective calls to deploy were removed from HostServicesDelegate and LauncherImpl.
10-04-2018