[9] Review request for 8157213: HiDPI support for Linux creates unnecessary dependency

[Jim Graham]

Ah, this is probably me returning a -1 as an uint.  If you change the "defval" used (line 167) in the call to query the 
property from -1 to 0 does it work as intended?

			...jim

On 5/18/16 11:59 AM, Erik De Rijcke wrote:
> I tested the patch.
>
> Without the dependency I get a an enormous stage (I explicitly set it to be 100x100). Debugging shows me the screen is
> initialized with these values:
>
> Screen:
>     ptr:0
>     adapter:0
>     depth:24
>     x:0
>     y:0
>     width:0
>     height:0
>     platformX:0
>     platformY:0
>     platformWidth:1680
>     platformHeight:946
>     visibleX:0
>     visibleY:0
>     visibleWidth:0
>     visibleHeight:0
>     platformScaleX:4.2949673E9
>     platformScaleY:4.2949673E9
>     outputScaleX:4.2949673E9
>     outputScaleY:4.2949673E9
>     resolutionX:0
>     resolutionY:0
>
>
> I assume the scaling factor is not what it should be?
>
> With the dependency installed the stage looks fine.
>
> On Wed, May 18, 2016 at 5:52 AM, Jim Graham <james.graham at oracle.com <mailto:james.graham at oracle.com>> wrote:
>
>     bug: https://bugs.openjdk.java.net/browse/JDK-8157213
>     webrev: http://cr.openjdk.java.net/~flar/JDK-8157213/webrev-00/
>
>     Details of what was fixed are listed in the bug report.  This will hopefully fix all of the dependencies that Erik
>     ran into in his Gentoo environment...
>
>                             ...jim
>
>