Re: [9] Review Request for 8036915: setLocationRelativeTo stopped working in Ubuntu 13.10 (Unity)

2015-10-05 Thread Semyon Sadetsky
Sorry. I meant it is not guaranteed to be sent by every WM. On 10/6/2015 9:03 AM, Semyon Sadetsky wrote: In is extended event. In does not guaranteed to be sent by any WM. On 10/5/2015 6:12 PM, Sergey Bylokhov wrote: Why we cannot treat such a XA_NET_FRAME_EXTENTS as a ConfigureNotify in which

Re: [9] Review Request for 8036915: setLocationRelativeTo stopped working in Ubuntu 13.10 (Unity)

2015-10-05 Thread Semyon Sadetsky
In is extended event. In does not guaranteed to be sent by any WM. On 10/5/2015 6:12 PM, Sergey Bylokhov wrote: Why we cannot treat such a XA_NET_FRAME_EXTENTS as a ConfigureNotify in which only insets are changed, and the window bounds/insets should be revalidated? On 05.10.15 14:56, Semyon

Re: [9] Review Request for 8080395: consider making sun.awt.CausedFocusEvent functionality public

2015-10-05 Thread Sergey Bylokhov
A few notes: - Typo in the header of the test: "Copyright (c) 20015", also please dispose the window before "throw new RuntimeException" - FocusEvent.java: some of the new javadoc overlap the 80 chars per line, also remove double spaces in the text. - I wonder about serialization, the new field

Re: [9] Review request for: 8086038 [macosx] No available data flavors when copying from Microsoft Word for Mac

2015-10-05 Thread Alexander Scherbatiy
The fix looks good to me. Thanks, Alexandr. On 10/5/2015 6:16 PM, Sergey Bylokhov wrote: Looks fine. On 05.10.15 17:44, mikhail cherkasov wrote: I reverted removing of "text/uri": http://cr.openjdk.java.net/~mcherkas/8086038/webrev.02/ it doesn't relate to original fix, so we can go wit

Re: [9] Review request for: 8086038 [macosx] No available data flavors when copying from Microsoft Word for Mac

2015-10-05 Thread Sergey Bylokhov
Looks fine. On 05.10.15 17:44, mikhail cherkasov wrote: I reverted removing of "text/uri": http://cr.openjdk.java.net/~mcherkas/8086038/webrev.02/ it doesn't relate to original fix, so we can go without this change. Please file a bug to not forget this cleanup. On 10/5/2015 17:10, Sergey Byl

Re: [9] Review Request for 8036915: setLocationRelativeTo stopped working in Ubuntu 13.10 (Unity)

2015-10-05 Thread Sergey Bylokhov
Why we cannot treat such a XA_NET_FRAME_EXTENTS as a ConfigureNotify in which only insets are changed, and the window bounds/insets should be revalidated? On 05.10.15 14:56, Semyon Sadetsky wrote: Hello, Please review fix for JDK9: bug: https://bugs.openjdk.java.net/browse/JDK-8036915 webrev

Re: Question: Area changes order of points

2015-10-05 Thread Hruda, Steve
Hi, is nobody able to answer my question? http://mail.openjdk.java.net/pipermail/awt-dev/2015-September/010037.html Regards, Steve Diese E-Mail wurde versandt im Auftrag des Unternehmens Intergraph Ges.m.b.H. Vertretungsberechtigte Geschäftsführer: Maximilian Weber Sitz der Gesellschaft

Re: [9] Review request for: 8086038 [macosx] No available data flavors when copying from Microsoft Word for Mac

2015-10-05 Thread mikhail cherkasov
I reverted removing of "text/uri": http://cr.openjdk.java.net/~mcherkas/8086038/webrev.02/ it doesn't relate to original fix, so we can go without this change. On 10/5/2015 17:10, Sergey Bylokhov wrote: On 01.10.15 17:22, mikhail cherkasov wrote: On 9/30/2015 18:48, Sergey Bylokhov wrote: Hi,

Re: [9] Review request for: 8086038 [macosx] No available data flavors when copying from Microsoft Word for Mac

2015-10-05 Thread Sergey Bylokhov
On 01.10.15 17:22, mikhail cherkasov wrote: On 9/30/2015 18:48, Sergey Bylokhov wrote: Hi, Mikhail. Why you have removed "text/uri" from the flavormap.properties? http://cr.openjdk.java.net/~mcherkas/8086038/webrev.01/src/java.datatransfer/macosx/classes/sun/datatransfer/resources/flavormap.pro

[9] Review request for 8137571 Linux HiDPI Graphics support

2015-10-05 Thread Alexander Scherbatiy
Hello, Could you review the fix: bug: https://bugs.openjdk.java.net/browse/JDK-8137571 webrev: http://cr.openjdk.java.net/~alexsch/8137571/webrev.00 This is an initial part of the HiDPI Graphics support on Linux for the JEP 263: HiDPI Graphics on Windows and Linux http://openjdk.jav

Re: [9] Review Request for 8067470: Examine if the replacement for sun.awt.SunToolkit.setLWRequestStatus should be provided

2015-10-05 Thread Sergey Bylokhov
On 05.10.15 15:03, Semyon Sadetsky wrote: Sergey, any other thoughts? I suggest to postpone the fix as discussed in private(latest massage was added to the CR). On 9/28/2015 12:43 PM, Semyon Sadetsky wrote: On 9/27/2015 1:13 AM, Sergey Bylokhov wrote: My understanding according to th

Re: [9] Review Request for 8067470: Examine if the replacement for sun.awt.SunToolkit.setLWRequestStatus should be provided

2015-10-05 Thread Semyon Sadetsky
Sergey, any other thoughts? On 9/28/2015 12:43 PM, Semyon Sadetsky wrote: On 9/27/2015 1:13 AM, Sergey Bylokhov wrote: My understanding according to the new javadoc: "* If the window does transfer focus to other * heavyweight containers and the synchronous lightweight focus requests * are

[9] Review Request for 8036915: setLocationRelativeTo stopped working in Ubuntu 13.10 (Unity)

2015-10-05 Thread Semyon Sadetsky
Hello, Please review fix for JDK9: bug: https://bugs.openjdk.java.net/browse/JDK-8036915 webrev: http://cr.openjdk.java.net/~ssadetsky/8036915/webrev.00/ The test scenario attached to the jira contains potential errors because Componet.getLocation() won't return the location at any moment of t

[9] RFR 8138838: docs cleanup for java.desktop

2015-10-05 Thread Alexander Stepanov
Sorry; wrong subject... On 10/5/2015 2:12 PM, Alexander Stepanov wrote: Hello, Could you please review the fix for https://bugs.openjdk.java.net/browse/JDK-8138838 Patch + webrev zipped + specdiff report: http://cr.openjdk.java.net/~avstepan/8138838 Just some cosmetic changes for docs (... ->

Re: P.S.: RFR [9] 8133651: automated replacing of old-style tags in docs

2015-10-05 Thread Alexander Stepanov
Hello, Could you please review the fix for https://bugs.openjdk.java.net/browse/JDK-8138838 Patch + webrev zipped + specdiff report: http://cr.openjdk.java.net/~avstepan/8138838 Just some cosmetic changes for docs (... -> {@code ...} replacement) + some misprints fixed. Not sure if these cha

Re: [awt-dev] Request for Approval and Review: JDK-8136592 [TEST_BUG] Fix 2 platform-specific closed regtests for jigsaw

2015-10-05 Thread Renjith Alexander
Hi All, Please find the updated webrev at: http://cr.openjdk.java.net/~pchopra/8136592/webrev.01/ Regards, Renjith. -Original Message- From: Yuri Nesterenko Sent: Wednesday, September 30, 2015 5:20 PM To: Renjith Alexander; awt-dev@openjdk.java.net Subject: Re: [awt-dev] Request for A