Re: [OpenJDK 2D-Dev] [9] Review request for 8073320 Windows HiDPI Graphics support

2015-11-16 Thread Jim Graham
Note that the test case in JDK-8069348 seems to work OK for a scaled graphics (it actually fails only if I set the uiScale to 1 for some reason), but Swing scrolling is completely broken. I dug down to the call to D3D->StretchRect and it was returning D3DERR_INVALIDCALL and simply not doing an

Re: [OpenJDK 2D-Dev] [9] Review request for 8073320 Windows HiDPI Graphics support

2015-11-16 Thread Jim Graham
I ran SwingSet2 and JDK-8069348 immediately jumped out as being broken to the point where SwingSet2 was not usable. We should definitely make that a high priority to fix ASAP. There were a couple of other very minor rendering issues, but they didn't really affect usability like that bug...

Re: [OpenJDK 2D-Dev] [9] Review request for 8076545 Text size is twice bigger under Windows L&F on Win 8.1 with HiDPI display

2015-11-16 Thread Jim Graham
Hi Alexandr, One bug - we should be using LOGPIXELSY instead of X. Otherwise, the changes look OK except for the one question/issue which I don't know/remember the answer for: How does this interact with per-monitor APIs? I believe LOGPIXELSX will not change until the user logs off and back

Re: [OpenJDK 2D-Dev] [9] Review request for 8076545 Text size is twice bigger under Windows L&F on Win 8.1 with HiDPI display

2015-11-16 Thread Jim Graham
Note that LOGPIXELSY is global and static between reboots so it doesn't really matter which monitor is used to get the value. Also, the issue is that the measurements are in pixels, so if we convert them into a resolution independent measurement then the rest of the scaling in the AWT/2D will

P.S.: Re: Review Request for 8142861: [TEST_BUG] MultiResolution image: add a manual test for two-display configuration (HiDPI + non-HiDPI)

2015-11-16 Thread Alexander Stepanov
P.S.: The behavior with Mac OS X mission control should also be checked which is definitely a manual job (it seems to be strange sometimes; not sure if buggy). So the test instructions should be appended a bit... On 11/16/2015 3:24 PM, Alexander Stepanov wrote: Hello Sergey, Thank you for the

Re: Review Request for 8142861: [TEST_BUG] MultiResolution image: add a manual test for two-display configuration (HiDPI + non-HiDPI)

2015-11-16 Thread Alexander Stepanov
Hello Sergey, Thank you for the notes. > Do you have some thoughts why this test cannot be converted to auto test? The following parts of the test scenario: - "Please try to drag both parent and child, do it fast several times and check if no artifacts occur." (not very valuable part of the t

Re: Review request for 8143064 Icons are not properly rendered with Windows L&F on HiDPI display

2015-11-16 Thread Sergey Bylokhov
Hi, Alexander. 109 AffineTransform tx = ((Graphics2D) g).getTransform(); 110 int sw = tx.isIdentity() ? w : (int) Math.round(w * tx.getScaleX()); 111 int sh = tx.isIdentity() ? h : (int) Math.round(h * tx.getScaleY()); I think that it is not necessary that !isIdentit

Review request for 8060137: Removing Text from TextField / TextArea is not possible after typing

2015-11-16 Thread Ambarish Rapte
Dear All,       Please review the fix for JDK9,     Bug: https://bugs.openjdk.java.net/browse/JDK-8060137     Webrev: http://cr.openjdk.java.net/~arapte/8060137/webrev.00/   Issue: 1.   Type any character in TextArea or TextField 2.   Call setText

Re: [OpenJDK 2D-Dev] [9] Review request for 8076545 Text size is twice bigger under Windows L&F on Win 8.1 with HiDPI display

2015-11-16 Thread Alexander Scherbatiy
Hello, Could you review the updated fix: http://cr.openjdk.java.net/~alexsch/8076545/webrev.02 - round is used instead of ceil - inverted scales are used Thanks, Alexandr. On 10/30/2015 10:40 PM, Jim Graham wrote: In this case round may be better. ceil() is more for cases wh

Re: Review Request for 8142861: [TEST_BUG] MultiResolution image: add a manual test for two-display configuration (HiDPI + non-HiDPI)

2015-11-16 Thread Sergey Bylokhov
Hi, Alexander. - Note that MultiRes image can be created at runtime, it will be good to cover this also. - Do you have some thoughts why this test cannot be converted to auto test? Probably some api can be added to jdk to simplify creation of such tests? like already added "-Dsun.java2d.uiScal