Re: RFR: 8350976: MenuBarSkin: exception initializing in a background thread [v2]

2025-04-05 Thread Lukasz Kostyra
On Wed, 5 Mar 2025 18:19:53 GMT, Andy Goryachev wrote: >> Allows MenuBar to be created in a background thread by delaying >> MenuBarSkin::rebuildUI() call until after the MenuBar becomes a part of the >> scene graph. > > Andy Goryachev has updated the pull request with a new target base due to

Re: RFR: 8350976: MenuBarSkin: exception initializing in a background thread [v2]

2025-04-05 Thread Kevin Rushforth
On Wed, 5 Mar 2025 18:19:53 GMT, Andy Goryachev wrote: >> Allows MenuBar to be created in a background thread by delaying >> MenuBarSkin::rebuildUI() call until after the MenuBar becomes a part of the >> scene graph. > > Andy Goryachev has updated the pull request with a new target base due to

Re: RFR: 8350976: MenuBarSkin: exception initializing in a background thread [v2]

2025-04-04 Thread Jayathirth D V
On Wed, 5 Mar 2025 18:19:53 GMT, Andy Goryachev wrote: >> Allows MenuBar to be created in a background thread by delaying >> MenuBarSkin::rebuildUI() call until after the MenuBar becomes a part of the >> scene graph. > > Andy Goryachev has updated the pull request with a new target base due to

Re: RFR: 8350976: MenuBarSkin: exception initializing in a background thread [v2]

2025-04-04 Thread Andy Goryachev
On Wed, 2 Apr 2025 06:06:58 GMT, Jayathirth D V wrote: >> Andy Goryachev has updated the pull request with a new target base due to a >> merge or a rebase. The incremental webrev excludes the unrelated changes >> brought in by the merge/rebase. The pull request contains six additional >> commi

Re: RFR: 8350976: MenuBarSkin: exception initializing in a background thread [v2]

2025-04-01 Thread Andy Goryachev
On Wed, 5 Mar 2025 18:19:53 GMT, Andy Goryachev wrote: >> Allows MenuBar to be created in a background thread by delaying >> MenuBarSkin::rebuildUI() call until after the MenuBar becomes a part of the >> scene graph. > > Andy Goryachev has updated the pull request with a new target base due to

Re: RFR: 8350976: MenuBarSkin: exception initializing in a background thread [v2]

2025-04-01 Thread Andy Goryachev
On Wed, 5 Mar 2025 18:19:53 GMT, Andy Goryachev wrote: >> Allows MenuBar to be created in a background thread by delaying >> MenuBarSkin::rebuildUI() call until after the MenuBar becomes a part of the >> scene graph. > > Andy Goryachev has updated the pull request with a new target base due to

Re: RFR: 8350976: MenuBarSkin: exception initializing in a background thread [v2]

2025-04-01 Thread Kevin Rushforth
On Tue, 1 Apr 2025 17:51:15 GMT, Andy Goryachev wrote: > The only way this functionality is suppressed is when > glass.disableThreadChecks system property is set to true. There is no easy way to set this, so I'm almost certain that isn't the case here. Jay can confirm, but I think there is a m

Re: RFR: 8350976: MenuBarSkin: exception initializing in a background thread [v2]

2025-04-01 Thread Andy Goryachev
On Wed, 5 Mar 2025 18:19:53 GMT, Andy Goryachev wrote: >> Allows MenuBar to be created in a background thread by delaying >> MenuBarSkin::rebuildUI() call until after the MenuBar becomes a part of the >> scene graph. > > Andy Goryachev has updated the pull request with a new target base due to

Re: RFR: 8350976: MenuBarSkin: exception initializing in a background thread [v2]

2025-04-01 Thread Jayathirth D V
On Wed, 5 Mar 2025 18:19:53 GMT, Andy Goryachev wrote: >> Allows MenuBar to be created in a background thread by delaying >> MenuBarSkin::rebuildUI() call until after the MenuBar becomes a part of the >> scene graph. > > Andy Goryachev has updated the pull request with a new target base due to

Re: RFR: 8350976: MenuBarSkin: exception initializing in a background thread [v2]

2025-04-01 Thread Jayathirth D V
On Wed, 5 Mar 2025 18:19:53 GMT, Andy Goryachev wrote: >> Allows MenuBar to be created in a background thread by delaying >> MenuBarSkin::rebuildUI() call until after the MenuBar becomes a part of the >> scene graph. > > Andy Goryachev has updated the pull request with a new target base due to

Re: RFR: 8350976: MenuBarSkin: exception initializing in a background thread [v2]

2025-04-01 Thread Andy Goryachev
On Tue, 1 Apr 2025 14:36:39 GMT, Jayathirth D V wrote: >> Andy Goryachev has updated the pull request with a new target base due to a >> merge or a rebase. The incremental webrev excludes the unrelated changes >> brought in by the merge/rebase. The pull request contains six additional >> commi

Re: RFR: 8350976: MenuBarSkin: exception initializing in a background thread [v2]

2025-04-01 Thread Jayathirth D V
On Wed, 5 Mar 2025 18:19:53 GMT, Andy Goryachev wrote: >> Allows MenuBar to be created in a background thread by delaying >> MenuBarSkin::rebuildUI() call until after the MenuBar becomes a part of the >> scene graph. > > Andy Goryachev has updated the pull request with a new target base due to

Re: RFR: 8350976: MenuBarSkin: exception initializing in a background thread [v2]

2025-03-28 Thread Andy Goryachev
On Fri, 28 Mar 2025 09:21:56 GMT, Jayathirth D V wrote: > menuBar test passes even without the fix in macOS. That's strange: for me it fails in the current master branch (the line number is different bc I just appended the test to the end of file). macOS 15.3.2. java.lang.IllegalStateExcepti

Re: RFR: 8350976: MenuBarSkin: exception initializing in a background thread [v2]

2025-03-28 Thread Jayathirth D V
On Wed, 5 Mar 2025 18:19:53 GMT, Andy Goryachev wrote: >> Allows MenuBar to be created in a background thread by delaying >> MenuBarSkin::rebuildUI() call until after the MenuBar becomes a part of the >> scene graph. > > Andy Goryachev has updated the pull request with a new target base due to

Re: RFR: 8350976: MenuBarSkin: exception initializing in a background thread [v2]

2025-03-13 Thread Andy Goryachev
On Thu, 13 Mar 2025 04:00:42 GMT, Michael Strauß wrote: >> Andy Goryachev has updated the pull request with a new target base due to a >> merge or a rebase. The incremental webrev excludes the unrelated changes >> brought in by the merge/rebase. The pull request contains six additional >> comm

Re: RFR: 8350976: MenuBarSkin: exception initializing in a background thread [v2]

2025-03-12 Thread Michael Strauß
On Wed, 5 Mar 2025 18:19:53 GMT, Andy Goryachev wrote: >> Allows MenuBar to be created in a background thread by delaying >> MenuBarSkin::rebuildUI() call until after the MenuBar becomes a part of the >> scene graph. > > Andy Goryachev has updated the pull request with a new target base due to

Re: RFR: 8350976: MenuBarSkin: exception initializing in a background thread [v2]

2025-03-05 Thread Andy Goryachev
> Allows MenuBar to be created in a background thread by delaying > MenuBarSkin::rebuildUI() call until after the MenuBar becomes a part of the > scene graph. Andy Goryachev has updated the pull request with a new target base due to a merge or a rebase. The incremental webrev excludes the unrel