<div dir="ltr"><div><div><br>Thanks a lot, <span name="Anthony Petrov" class="">Anthony!</span></div>Please let me know when it will be backported to 1.7... As I understarding it will be backported to 1.7 also. Is it true?<br>
<br></div>Best Regards,<br>Vladimir<br><div><br><br></div></div><div class="gmail_extra"><br><br><div class="gmail_quote">2013/4/26 Anthony Petrov <span dir="ltr">&lt;<a href="mailto:anthony.petrov@oracle.com" target="_blank">anthony.petrov@oracle.com</a>&gt;</span><br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Hi Vladimir,<br>
<br>
I&#39;ve just pushed your fix:<br>
<br>
<a href="http://hg.openjdk.java.net/jdk8/awt/jdk/rev/854f60ec4bfb" target="_blank">http://hg.openjdk.java.net/<u></u>jdk8/awt/jdk/rev/854f60ec4bfb</a><br>
<br>
Thank you for contributing the patch!<br>
<br>
--<br>
best regards,<br>
Anthony<div class="HOEnZb"><div class="h5"><br>
<br>
On 04/25/2013 04:40 PM, Anthony Petrov wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
The fix looks good to me as well. If no objections I can push it to the<br>
AWT repository tomorrow.<br>
<br>
--<br>
best regards,<br>
Anthony<br>
<br>
On 04/25/13 16:21, Sergey Bylokhov wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
Hi, Vladimir.<br>
Fix looks good.<br>
<br>
On <a href="tel:24.04.2013%2016" value="+12404201316" target="_blank">24.04.2013 16</a>:40, Vladimir Kravets wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
Hi guys,<br>
<br>
Patch was updated according to yours notices.<br>
- Fix copyright in the XWindowPeer<br>
- Add test for such behavior:<br>
   * Applicable to Metacity WM<br>
   * Cover such behavior:<br>
      1. going to fullscreen<br>
      2. showing the modal dialog<br>
      3. check if dialog appeared above fullscreen window or not<br>
<br>
You can look at it on the same link<br>
<a href="http://vkravets.github.io/awt-fixes/8012586/webrev.00/index.html" target="_blank">http://vkravets.github.io/awt-<u></u>fixes/8012586/webrev.00/index.<u></u>html</a><br>
(webrev original folder can be taken from<br>
<a href="https://github.com/vkravets/awt-fixes/tree/master/8012586" target="_blank">https://github.com/vkravets/<u></u>awt-fixes/tree/master/8012586</a>)<br>
<br>
Thanks a lot for cooperative work!<br>
<br>
Waiting merging in the master...<br>
<br>
Best Regards,<br>
Vladimir<br>
<br>
<br>
<br>
2013/4/23 Sergey Bylokhov &lt;<a href="mailto:Sergey.Bylokhov@oracle.com" target="_blank">Sergey.Bylokhov@oracle.com</a><br>
&lt;mailto:<a href="mailto:Sergey.Bylokhov@oracle.com" target="_blank">Sergey.Bylokhov@<u></u>oracle.com</a>&gt;&gt;<br>
<br>
    Hi, Vladimir.<br>
<br>
    On 23.04.2013 14:36, Vladimir Kravets wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
    Hi Sergey,<br>
<br>
    Could you please point me where I can do(add/create) this?<br>
</blockquote>
    It depends on how you write your test.<br>
    Location of tests for a full screen mode:<br>
    jdk/test/java/awt/FullScreen/<br>
    Location of tests for Dialogs:<br>
    jdk/test/java/awt/Dialog/<br>
<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
    Thanks,<br>
    Vladimir<br>
<br>
<br>
    2013/4/23 Sergey Bylokhov &lt;<a href="mailto:Sergey.Bylokhov@oracle.com" target="_blank">Sergey.Bylokhov@oracle.com</a><br>
    &lt;mailto:<a href="mailto:Sergey.Bylokhov@oracle.com" target="_blank">Sergey.Bylokhov@<u></u>oracle.com</a>&gt;&gt;<br>
<br>
        Hello, Vladimir.<br>
        Testcase will be welcome, since the issue was not found by<br>
        our regression tests.<br>
<br>
<br>
        On 23.04.2013 12:50, Anthony Petrov wrote:<br>
<br>
            Thanks Vladimir. The fix looks fine to me. We need to get<br>
            at least one more review, after which I&#39;ll be able to<br>
            push it to the repository. Could anyone review this<br>
            please? Here&#39;s a link to the webrev:<br>
<br>
<br>
<a href="http://vkravets.github.io/awt-fixes/8012586/webrev.00/index.html" target="_blank">http://vkravets.github.io/awt-<u></u>fixes/8012586/webrev.00/index.<u></u>html</a><br>
<br>
            Regarding other WMs and the DIALOG type - yes, I agree<br>
            with you. I don&#39;t known of any WMs that would do<br>
            something strange for DIALOG windows, so as I said this<br>
            is just a theoretical concern. The fix should work fine<br>
            now for most WMs. If we encounter a problem with a<br>
            particular WM in the future, we&#39;ll see how to deal with it.<br>
<br>
            --<br>
            best regards,<br>
            Anthony<br>
<br>
            On 04/22/2013 06:33 PM, Vladimir Kravets wrote:<br>
<br>
                Done. You can look at the same link. BTW the full<br>
                listing webrev<br>
                directory you can find on the<br>
<br>
<a href="https://github.com/vkravets/awt-fixes/tree/master/8012586" target="_blank">https://github.com/vkravets/<u></u>awt-fixes/tree/master/8012586</a><br>
                link.<br>
<br>
                &gt; I assume you have tested this fix and it works as<br>
                expected for your<br>
                use cases?<br>
                Of course =)<br>
<br>
                &gt;Although I&#39;m a little concerned about other window<br>
                managers that may<br>
                present DIALOG windows somehow completely differently<br>
                (as some Mac-style<br>
                sheets for example). In this case the fix may fail<br>
                since it marks<br>
                regular owned windows as DIALOGs, too. This is purely<br>
                a theoretical<br>
                concern, but still something to think about.<br>
                Shouldn&#39;t we check the<br>
                window type instead of just checking whether the<br>
                window has an owner or not?<br>
<br>
                Since this is is not ICCCM-compliant atom, thus I<br>
                guess that all another<br>
                WM&#39;s should compliant by this spec from freedesktop.<br>
<br>
<a href="http://standards.freedesktop.org/wm-spec/latest/ar01s05.html#idp1333344" target="_blank">http://standards.freedesktop.<u></u>org/wm-spec/latest/ar01s05.<u></u>html#idp1333344</a><br>
<br>
                In this section very clearly specified how should<br>
                react WM&#39;s on<br>
                different wndow type. BTW before fix AWT was not<br>
                fully compliant with<br>
                this spec =(<br>
<br>
                Best Regards,<br>
                Vladimir<br>
<br>
<br>
<br>
        --<br>
        Best regards, Sergey.<br>
<br>
<br>
</blockquote>
<br>
<br>
    --<br>
    Best regards, Sergey.<br>
<br>
<br>
</blockquote>
<br>
<br>
--<br>
Best regards, Sergey.<br>
<br>
</blockquote></blockquote>
</div></div></blockquote></div><br></div>