Hi Sergey, The fix looks good to me.
Thanks, Dmitry > On 14 Sep 2017, at 23:46, Sergey Bylokhov <[email protected]> wrote: > > Hello, > Please review the fix for jdk10. > > Bug: https://bugs.openjdk.java.net/browse/JDK-8186559 > Webrev can be found at: http://cr.openjdk.java.net/~serb/8186559/webrev.00 > > When our NSView is embedded in native to the SWT-based NSWindow we > incorrectly fetch the location of the NSView. The reason is that we use wrong > screen when we flip the location. Currently we use [mainScreen] - which > "returns the screen object containing the window with the keyboard focus", > but we need the first screen which contains 0,0 coordinates. > > In the fix the utility method is reused which internally uses > [[NSScreen screens] objectAtIndex:0] > > -- > Best regards, Sergey.
