Hi, Petr.
The fix looks good.
BTW:This code was added on windows as a fix for:
https://bugs.openjdk.java.net/browse/JDK-4399204
https://bugs.openjdk.java.net/browse/JDK-4393453 <-- the code change is attached

On 16.01.2014 18:12, Petr Pchelko wrote:
Hello, AWT Team.

Please review the fix for the issue:
https://bugs.openjdk.java.net/browse/JDK-8031964
The fix is available at:
http://cr.openjdk.java.net/~pchelko/9/8031964/webrev.00/ <http://cr.openjdk.java.net/%7Epchelko/9/8031964/webrev.00/>

The fix resolves the problem we've had with the URL pasteboard type and the problem with overriding the method of the superclass.

Additional comments are needed about the removed code about HTML: I've run all tests I know about HTML transfer and I could not find what fails because of this code. So I've removed it as it was never called any way. However, we have a bug JDK-8027366 <https://bugs.openjdk.java.net/browse/JDK-8027366> which is about the problem with an overriding method. After I push this changes I'll cross-reference that bug with this, so that we could restore the deleted code when we find out why was it needed before the Denis's data transfer fix.

With best regards. Petr.


--
Best regards, Sergey.

Reply via email to