<html>
<head>
<meta content="text/html; charset=UTF-8" http-equiv="Content-Type">
</head>
<body bgcolor="#FFFFFF" text="#000000">
<div class="moz-cite-prefix">You will need to convince me of the
appropriateness of opening these methods.<br>
It seems to me that are for the focus system, not for
applications.<br>
<br>
Why should an application be allowed to say "I would like
component X" to<br>
receive focus and tell it the reason is a mouse event" when in
fact it is nothing of the kind.<br>
<br>
I would like to hear from others if they see a valid use case and
that there are no<br>
down-sides to mis-use.<br>
<br>
-phil.<br>
<br>
On 04/19/2016 02:30 AM, Semyon Sadetsky wrote:<br>
</div>
<blockquote cite="mid:5715FABE.7050109@oracle.com" type="cite">
<meta http-equiv="content-type" content="text/html; charset=UTF-8">
Hello, <br>
<br>
Please review fix for JDK9:<br>
<br>
bug: <a moz-do-not-send="true" class="moz-txt-link-freetext"
href="https://bugs.openjdk.java.net/browse/JDK-8154434">https://bugs.openjdk.java.net/browse/JDK-8154434</a><br>
webrev: <a moz-do-not-send="true" class="moz-txt-link-freetext"
href="http://cr.openjdk.java.net/%7Essadetsky/8154434/webrev.00/">http://cr.openjdk.java.net/~ssadetsky/8154434/webrev.00/</a><br>
<br>
To support the new FocusEvent cause concept introduced by
JDK-8080395 the next package-private methods of the
java.awt.Component class are opened in the fix:<br>
<br>
boolean requestFocus(FocusEvent.Cause cause) -> public void
requestFocus(FocusEvent.Cause cause)
<meta http-equiv="content-type" content="text/html; charset=UTF-8">
<meta http-equiv="content-type" content="text/html; charset=UTF-8">
<meta http-equiv="content-type" content="text/html; charset=UTF-8">
<br>
<br>
<meta http-equiv="content-type" content="text/html; charset=UTF-8">
boolean requestFocus(boolean temporary, FocusEvent.Cause cause)
-> protected boolean requestFocus(boolean temporary,
FocusEvent.Cause cause)<br>
<br>
<meta http-equiv="content-type" content="text/html; charset=UTF-8">
boolean requestFocusInWindow(FocusEvent.Cause cause) ->
<meta http-equiv="content-type" content="text/html; charset=UTF-8">
public boolean requestFocusInWindow(FocusEvent.Cause cause)<br>
<br>
The methods are changed to be symmetric with the focus request
methods of the same class which do not accept the cause parameter.<br>
The method requestFocus(FocusEvent.Cause cause) was changed to
return no value similarly to the requestFocus() because the
returning boolean true cannot guarantee the focus gain.<br>
<br>
--Semyon<br>
<br>
<br>
</blockquote>
<br>
</body>
</html>