Hi Alexander,
Thank you for the update. It works for me.
One additional remark. You do not call ::CoUninitialize() in
awt_Taskbar.cpp.
And I'm not sure that COM init/uninit can be done from arbitrary thread
without aftereffects. MSDN states the next:
A thread must call CoUninitialize once fo
Please review the updated webrev:
http://cr.openjdk.java.net/~ssadetsky/8080395/webrev.04/
- CausedFocusEvent is restored to avoid CNFE during deserialization.
- readResolve() is added to FocusEvent to handle the null cause test.
- deserialization compatibility test scenarios added
--Semyon
Hi Ambarish,
Updated test .
Webrev: http://cr.openjdk.java.net/~arapte/prem/8062946/webrev.01/
Regards,
Prem
From: Ambarish Rapte
Sent: Tuesday, February 02, 2016 2:47 PM
To: Alexander Scherbatiy; Prem Balakrishnan; Semyon Sadetsky; Sergey Bylokhov;
Rajeev Chamyal; swing-...@openj
The fix looks good to me.
Thanks,
Alexandr.
On 2/2/2016 1:00 AM, Ambarish Rapte wrote:
Hi Ajit,
Fix looks good to me.
Thanks,
Ambarish
*From:*Ajit Ghaisas
*Sent:* Monday, February 01, 2016 4:47 PM
*To:* Ambarish Rapte; Sergey Bylokhov; Semyon Sadetsky;
awt-dev@openjdk.java.net
*Subject:*
The fix looks good to me.
Thanks,
Alexandr.
On 2/2/2016 12:15 AM, Semyon Sadetsky wrote:
Hello Anton,
Thanks for the explanation.
The fix looks good to me.
--Semyon
On 2/1/2016 6:29 PM, Anton Litvinov wrote:
Hello Semyon,
Thank you for review of this fix. This issue depends on race
condit
Hi Prem,
The test passes without including the fix.
The test should fail without fix, and should pass with fix.
Thanks,
Ambarish
From: Alexandr Scherbatiy
Sent: Monday, February 01, 2016 5:15 PM
To: Prem Balakrishnan; Semyon Sadetsky; Sergey Bylokhov; Amb
Hi Ajit,
Fix looks good to me.
Thanks,
Ambarish
From: Ajit Ghaisas
Sent: Monday, February 01, 2016 4:47 PM
To: Ambarish Rapte; Sergey Bylokhov; Semyon Sadetsky; awt-dev@openjdk.java.net
Subject: RE: Review request for 8005918 : [TESTBUG] There are no
'Fra
Hello Anton,
Thanks for the explanation.
The fix looks good to me.
--Semyon
On 2/1/2016 6:29 PM, Anton Litvinov wrote:
Hello Semyon,
Thank you for review of this fix. This issue depends on race
conditions. In fact this issue is reproducible sometimes also, when
the components are initially