[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

[Seán Coffey]

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
>>>
>>
>