<AWT Dev> Review request for 8151385: [hidpi] JOptionPane-Icons only partially visible when using Windows 10 L&F
hs at tagtraum.com
Sat Mar 12 11:23:58 UTC 2016
Can you also review it?
> On Mar 11, 2016, at 19:06, Alexander Scherbatiy <alexandr.scherbatiy at oracle.com> wrote:
> I have uploaded the webrev to the cr.openjdk.java.net:
> http://cr.openjdk.java.net/~alexsch/hendrik.schreiber/8151385/webrev.00/ <http://cr.openjdk.java.net/~alexsch/hendrik.schreiber/8151385/webrev.00/>
> On 11/03/16 20:10, Hendrik Schreiber wrote:
>> Hey there,
>> I would very much appreciate a review (and a sponsor) for this JDK 8u fix.
>> Thanks so much!
>>> On Mar 7, 2016, at 16:47, Hendrik Schreiber <hs at tagtraum.com <mailto:hs at tagtraum.com>> wrote:
>>> Hey there,
>>> Could someone please review this fix for JDK8u.
>>> https://bugs.openjdk.java.net/browse/JDK-8151385 <https://bugs.openjdk.java.net/browse/JDK-8151385>
>>> https://www.beatunes.com/download/webrev-8151385.zip <https://www.beatunes.com/download/webrev-8151385.zip>
>>> On Windows 10 with a HiDPI display, the icons in JOptionsPanes are only partially displayed (screenshot in the bug report).
>>> Changed the methods that create BufferedImages from Windows bitmaps so the actual bitmap size is taken into account.
>>> Background: Windows 10 delivers differently sized bitmaps based on display resolution.
>>> Automated Test:
>>> The error icon displayed by JOptionsPanes is round and as such the pixels in all four corners are identical (transparent).
>>> If the error occurs, at least the pixel in the lower right corner is not the same as the one on the upper left corner. The test checks, that the pixels in all four corners are identical. This makes sure the test also succeeds on standard DPI displays.
>>> For the test to actually fail (without the fix), this test needs to be run on a Windows 10 system with a HiDPI display.
>>> I have tested this on Windows 7 and Windows 10.
>>> I have not tested it on Vista or Windows 8.x.
-------------- next part --------------
An HTML attachment was scrubbed...
More information about the awt-dev