RFR: 8349750: [TestBug] NodeInitializationStressTest: remaining Nodes

[Andy Goryachev]

On Tue, 25 Feb 2025 17:26:30 GMT, Kevin Rushforth <kcr at openjdk.org> wrote:

>> ## Added Missing Controls
>> 
>> ButtonBar
>> ProgressIndicator
>> Separator
>> Slider,
>> 
>> ## Added Node Classes
>> 
>> AnchorPane
>> AmbientLight
>> Arc
>> BorderPane
>> Box
>> Circle
>> CubicCurve
>> Cylinder
>> DialogPane
>> DirectionalLight
>> Ellipse
>> FlowPane
>> GridPane
>> Group
>> HBox
>> ImageView
>> Line
>> MediaView
>> MeshView
>> Pane
>> ParallelCamera
>> Path
>> PerspectiveCamera
>> PointLight
>> Polygon
>> Polyline
>> QuadCurve
>> Rectangle
>> Region
>> Sphere
>> StackPane
>> SVGPath
>> TilePane
>> VBox
>> 
>> 
>> ## Miscellaneous
>> 
>> - minor improvements
>> - remove tests that execute show() in non-fx threads per [JDK-8350048](https://bugs.openjdk.org/browse/JDK-8350048)
>> 
>> 
>> 
>> ## Not Included Due to Threading Limitations
>> 
>> HTMLEditor
>> MenuBar
>> SwingNode
>> WebView
>> 
>> ## Note to the Reviewers
>> 
>> To avoid merge conflicts, the preferred order of integrations:
>> 
>> #1697 
>> #1713 
>> #1717
>
> tests/system/src/test/java/test/robot/javafx/scene/NodeInitializationStressTest.java line 737:
> 
>> 735:             c.setFitHeight(nextDouble(200));
>> 736:             c.setFitWidth(nextDouble(200));
>> 737:             //c.setMediaPlayer(new MediaPlayer(new Media("no-data-url-support")));
> 
> Would it be useful to load a media file? If so, you could use the default media file that Ensemble uses.

I don't think we want to download media from an URL as part of this test, and data URLs are not supported by Media (and it will be too long anyway).

I think we can classify this under rubric "one can only create a JavaFX object in a background thread, but not utilize it".

-------------

PR Review Comment: https://git.openjdk.org/jfx/pull/1713#discussion_r1970413210