The fix looks good to me.
I have also added awt-dev alias because fix changes the native part.
Thanks,
Alexandr.
On 26/09/16 16:22, Anton Tarasov wrote:
Hi Pete,
The fix looks fine to me.
Thanks,
Anton.
On 9/24/2016 2:59 AM, Pete Brunet wrote:
New webrev:
http://cr.openjdk.java.net/~ptbrunet/JDK-8160893/webrev.02/
I added a null check at line 688 like the one at line 693.
Pete
On 9/23/16 2:46 PM, Pete Brunet wrote:
Please review the patch for JDK-8160893.
Bug: https://bugs.openjdk.java.net/browse/JDK-8160893
Patch: http://cr.openjdk.java.net/~ptbrunet/JDK-8160893/webrev.01/
This patch adds support to post menu related events:
kAXMenuOpenedNotification, kAXMenuClosedNotification, and
kAXMenuItemSelectedNotification. Also note that these changes had
impacts on the current combo box behavior so you'll see combo boxes
filtered out from this fix. The combo box behavior is currently broken
and will be fixed at a later date but at least for now I didn't want
this patch to change that (broken) behavior.
TiA,
Pete
