[7u] Bug ID 8008289 : Request for review and approval/backport: 7189299: DefaultButtonModel instance keeps stale listeners of JButton in case of multiple SwingUtilities.updateComponentTreeUI() calls
Frank Ding
dingxmin at linux.vnet.ibm.com
Fri May 17 02:21:12 PDT 2013
Good catch. Thanks.
Best regards,
Frank
On 5/10/2013 8:54 PM, Seán Coffey wrote:
> Thanks Frank. One last request before you push if possible. Given that
> 7189299 is no longer a valid bug reference (I can't find it in bug
> system) - can you rename the testcase directory also. It'll help
> future code maintenance.
>
> change 7189299/ to 8008289/
>
> No need to re-submit a webrev for that!
>
> regards,
> Sean.
>
> On 10/05/13 04:48, Frank Ding wrote:
>> Hi Sean,
>> I have updated review patch according to your comment @
>> http://cr.openjdk.java.net/~dingxmin/jdk7u/7189299/webrev.01
>>
>> Best regards,
>> Frank
>>
>> On 5/10/2013 2:43 AM, Seán Coffey wrote:
>>> Frank,
>>>
>>> looks like bug 7189299 was migrated over to the JDK project and
>>> became bug ID 8008289 as per jdk8 changeset.
>>>
>>> http://bugs.sun.com/view_bug.do?bug_id=8008289
>>>
>>> Can you update the bug reference in the src code before pushing ?
>>> "// when a new UI component is replaced. See bug 7189299."
>>>
>>> Likewise, update the @bug tag in the testcase to 8008289
>>>
>>> Approved for 7u-dev.
>>>
>>> regards,
>>> Sean.
>>>
>>> On 09/05/2013 07:55, Frank Ding wrote:
>>>> Hi all,
>>>>
>>>> I'd like to request for approval to push the following change
>>>> into 7u. The patch in 7u is exactly identical to the one in 8.
>>>>
>>>> Bug: http://bugs.sun.com/bugdatabase/view_bug.do?bug_id=7189299
>>>> (the link is not available)
>>>>
>>>> Changeset in jdk8
>>>> http://hg.openjdk.java.net/jdk8/awt/jdk/rev/4bf242def958
>>>> Reviewed by alexsch, malenkov
>>>>
>>>> Webrev for 7u
>>>> http://cr.openjdk.java.net/~dingxmin/jdk7u/7189299/webrev.00/
>>>>
>>>> Starting email discussion thread
>>>> http://mail.openjdk.java.net/pipermail/swing-dev/2012-August/002182.html
>>>>
>>>>
>>>> Best regards,
>>>> Frank
>>>>
>>>
>>
>
More information about the jdk7u-dev
mailing list