[Rev 02] RFR: 8244418: MenuBar: IOOB exception on requestFocus on empty bar

Ajit Ghaisas aghaisas at openjdk.java.net
Mon Jun 15 11:50:41 UTC 2020


On Sat, 13 Jun 2020 15:03:00 GMT, Kevin Rushforth <kcr at openjdk.org> wrote:

>> darn .. overlooked that fixed index ..
>> 
>> As to the naming: personally, I hate violation of naming conventions even for test-only methods, so would tend to make
>> setFocusedMenuIndex  package and doc as being used for testing as well as internally.
>> We might consider aligning the corresponding methods in the shim (they are also slightly confusing in
>> get/set/FocusedIndex).
>
> Good suggestion.

Thanks @kevinrushforth  for catching this. It was a big overlook on my part.

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

PR: https://git.openjdk.java.net/jfx/pull/216


More information about the openjfx-dev mailing list