Re: Review Request of 8146321: [macosx] JInternalFrame frame icon in wrong position on Mac L&F if icon is not ImageIcon

2016-02-09 Thread Rajeev Chamyal
The fix looks good to me.   Regards, Rajeev Chamyal   From: Avik Niyogi Sent: 09 February 2016 11:43 To: Sergey Bylokhov; Alexandr Scherbatiy; Rajeev Chamyal Cc: swing-dev@openjdk.java.net Subject: Re: Review Request of 8146321: [macosx] JInternalFrame frame icon in wrong position on Mac L

Re: Review request for JDK-8139213 : Mac OS X Aqua Look and Feel: JOptionPane can truncate the first button.

2016-01-20 Thread Rajeev Chamyal
Hello Alexandr, Thanks for the review. I have updated the webrev as suggested. http://cr.openjdk.java.net/~rchamyal/8139213/webrev.01/ Regards, Rajeev Chamyal -Original Message- From: Alexander Scherbatiy Sent: 20 January 2016 17:04 To: Rajeev Chamyal Cc: Sergey Bylokhov; Prasanta

Re: Review request for 8015748: JProgressbar with Aqua LaF ignores JProgressbar#applyComponentOrientation(ComponentOrientation.RIGHT_TO_LEFT) call

2016-01-20 Thread Rajeev Chamyal
Looks good to me. Regards, Rajeev Chamyal From: Avik Niyogi Sent: 20 January 2016 12:23 To: Rajeev Chamyal; Alexander Scherbatiy; Sergey Bylokhov Cc: swing-dev@openjdk.java.net Subject: Re: Review request for 8015748: JProgressbar with Aqua LaF ignores JProgressbar

Re: Review request for JDK-8146276 : Right aligned ToolBar component does not appear

2016-01-19 Thread Rajeev Chamyal
sider preferred size of child comps. The preferred layout size is not getting called in this case. I have replaced the minimumLayoutSize with preferredLayoutSize and updated the webrev. http://cr.openjdk.java.net/~rchamyal/8146276/webrev.01/ Regards, Rajeev Chamyal -Original Message- From: Alex

Re: Review request for 8015748: JProgressbar with Aqua LaF ignores JProgressbar#applyComponentOrientation(ComponentOrientation.RIGHT_TO_LEFT) call

2016-01-19 Thread Rajeev Chamyal
Hello Avik, All exception caught during test should mark the test as failed. For example not able to set any LAF should also be considered as test failure. Regards, Rajeev Chamyal From: Avik Niyogi Sent: 20 January 2016 10:20 To: Rajeev Chamyal Cc: Alexander Scherbatiy; Sergey

Re: Review request for JDK-8139213 : Mac OS X Aqua Look and Feel: JOptionPane can truncate the first button.

2016-01-19 Thread Rajeev Chamyal
needs to be added to the minimum size obtained from the parent class. The current webrev code I feel is much cleaner than getting the size from parent class. Regards, Rajeev Chamyal From: Alexander Scherbatiy Sent: 19 January 2016 21:00 To: Rajeev Chamyal; Sergey Bylokhov; Prasanta

Re: Review request for JDK-8139213 : Mac OS X Aqua Look and Feel: JOptionPane can truncate the first button.

2016-01-19 Thread Rajeev Chamyal
Hello All, Gentle reminder for review. Regards, Rajeev Chamyal -Original Message- From: Rajeev Chamyal Sent: 13 January 2016 16:37 To: Sergey Bylokhov; Alexander Scherbatiy; Prasanta Sadhukhan; swing-dev@openjdk.java.net Subject: Review request for JDK-8139213 : Mac OS X Aqua Look and

Review request for JDK-8139213 : Mac OS X Aqua Look and Feel: JOptionPane can truncate the first button.

2016-01-13 Thread Rajeev Chamyal
AquaButtonAreaLayout to consider default button width and height while calculating minimum size for JOptionPane. Regards, Rajeev Chamyal

Re: Review request for JDK-8075084 JOptionPane.showMessageDialog causes JScrollBar to move

2016-01-12 Thread Rajeev Chamyal
Hello All, Gentle reminder to review the fix. http://cr.openjdk.java.net/~rchamyal/8075084/webrev.02/ Regards, Rajeev Chamyal -Original Message- From: Rajeev Chamyal Sent: 27 December 2015 20:32 To: Sergey Bylokhov Cc: swing-dev@openjdk.java.net Subject: Re: Review request for JDK

Review request for JDK-8146276 : Right aligned ToolBar component does not appear

2016-01-11 Thread Rajeev Chamyal
SynthToolBar is not checking if preferred size is set for the components. Fix: Updated the minimumLayoutSize method of SynthToolBarUI.java to check preferred size of components as well. Regards, Rajeev Chamyal

Re: Review request for JDK-8145060 Minimizing a JInternal frame not shifting focus to frame below it

2016-01-11 Thread Rajeev Chamyal
Hello Sergey, Could you please review the updated webrev. http://cr.openjdk.java.net/~rchamyal/8145060/webrev.01/ Regards, Rajeev Chamyal -Original Message- From: Rajeev Chamyal Sent: 30 December 2015 13:33 To: Sergey Bylokhov; Alexander Scherbatiy; Prasanta Sadhukhan; swing-dev

Re: Review request for JDK-8145896 JInternalFrame setMaximum before adding to desktop throws null pointer exception

2016-01-11 Thread Rajeev Chamyal
Hello Sergey, Could you please review the updated webrev. http://cr.openjdk.java.net/~rchamyal/8145896/webrev.01/ Regards, Rajeev Chamyal -Original Message- From: Rajeev Chamyal Sent: 02 January 2016 11:46 To: Sergey Bylokhov; swing-dev@openjdk.java.net Subject: RE: Review request for

Re: Review request for JDK-8145896 JInternalFrame setMaximum before adding to desktop throws null pointer exception

2016-01-01 Thread Rajeev Chamyal
Hello Sergey, Thanks for review I have updated webrev. There was one more issue with fix to fix it , I have updated BasicInternalFrameUI.java and added it to webrev. http://cr.openjdk.java.net/~rchamyal/8145896/webrev.01/ Regards, Rajeev Chamyal -Original Message- From: Sergey

Re: Review request for JDK-8145060 Minimizing a JInternal frame not shifting focus to frame below it

2015-12-30 Thread Rajeev Chamyal
Hello Sergey, I have updated the webrev. http://cr.openjdk.java.net/~rchamyal/8145060/webrev.01/ Regards, Rajeev Chamyal -Original Message- From: Sergey Bylokhov Sent: 29 December 2015 19:48 To: Rajeev Chamyal; Alexander Scherbatiy; Prasanta Sadhukhan; swing-dev@openjdk.java.net

Re: Review request for JDK-8145896 JInternalFrame setMaximum before adding to desktop throws null pointer exception

2015-12-29 Thread Rajeev Chamyal
Hello All, I need one more review for this webrev. HYPERLINK "http://cr.openjdk.java.net/%7Erchamyal/8145896/webrev.00/"http://cr.openjdk.java.net/~rchamyal/8145896/webrev.00/ Regards, Rajeev Chamyal From: Alexander Scherbatiy Sent: 23 December 2015 19:44 To: Rajeev

Re: Review request for JDK-4769772 JInternalFrame.setIcon(true) before JDesktopPane.add(JIF) causes wrong state

2015-12-29 Thread Rajeev Chamyal
Hello All, I need one more review for this webrev. http://cr.openjdk.java.net/~rchamyal/4769772/webrev.03/ Regards, Rajeev Chamyal -Original Message- From: Alexey Ivanov Sent: 28 December 2015 19:15 To: Rajeev Chamyal; swing-dev@openjdk.java.net Subject: Re: Review request for JDK

Re: Review request for JDK-4769772 JInternalFrame.setIcon(true) before JDesktopPane.add(JIF) causes wrong state

2015-12-28 Thread Rajeev Chamyal
waste time calculating the position if we ignore the operation eventually? [Rajeev Chamyal] Updated the code as suggested. 2) Yet your new errorMessage filed has the same issue. It should be marked volatile since it's accessed from different threads without synchronization. Actually I su

Re: Review request for JDK-8145060 Minimizing a JInternal frame not shifting focus to frame below it

2015-12-27 Thread Rajeev Chamyal
Hello Sergey, Thanks for the review. I have updated the code. http://cr.openjdk.java.net/~rchamyal/8145060/webrev.01/ Regards, Rajeev Chamyal -Original Message- From: Sergey Bylokhov Sent: 25 December 2015 21:23 To: Rajeev Chamyal; Alexander Scherbatiy; Prasanta Sadhukhan; swing-dev

Re: Review request for JDK-4769772 JInternalFrame.setIcon(true) before JDesktopPane.add(JIF) causes wrong state

2015-12-27 Thread Rajeev Chamyal
Hello Alexey, Thanks for the review. I have updated the code as suggested. http://cr.openjdk.java.net/~rchamyal/4769772/webrev.02/ Please see response to questions inline. Regards, Rajeev Chamyal -Original Message- From: Alexey Ivanov Sent: 25 December 2015 18:14 To: Rajeev Chamyal

Re: Review request for JDK-8075084 JOptionPane.showMessageDialog causes JScrollBar to move

2015-12-27 Thread Rajeev Chamyal
BasicScrollBarUI::scrollByUnit method is again getting called recursively, Because of which the scroll bar pointer keeps on moving till the end of scrollbar. With this new implementation these issue are not seen. Regards, Rajeev Chamyal -Original Message- From: Sergey Bylokhov Sent: 25

Re: Review request for JDK-4769772 JInternalFrame.setIcon(true) before JDesktopPane.add(JIF) causes wrong state

2015-12-23 Thread Rajeev Chamyal
Hello Alexey, Thanks for the review. I have updated webrev as per review comments. http://cr.openjdk.java.net/~rchamyal/4769772/webrev.01/ Regards, Rajeev Chamyal -Original Message- From: Alexey Ivanov Sent: 23 December 2015 19:27 To: swing-dev@openjdk.java.net Subject: Re: Review

Re: Review request for JDK-8078489 JSpinner using SpinnerNumberModel doesn't honor orientation flag

2015-12-23 Thread Rajeev Chamyal
Hello Sergey, Thanks for the review. The issues is reproducible with mac Aqua LAF only with other LAF/platforms issue is not reproducible. I have updated webrev as per review comments. http://cr.openjdk.java.net/~rchamyal/8078489/webrev.01/ Regards, Rajeev Chamyal -Original Message

Re: Review request for JDK-8075084 JOptionPane.showMessageDialog causes JScrollBar to move

2015-12-23 Thread Rajeev Chamyal
://cr.openjdk.java.net/~rchamyal/8075084/webrev.02/ Regards, Rajeev Chamyal -Original Message- From: Alexander Scherbatiy Sent: 22 December 2015 05:13 To: Rajeev Chamyal Cc: Sergey Bylokhov; Prasanta Sadhukhan; swing-dev@openjdk.java.net Subject: Re: Review request for JDK-8075084

Review request for JDK-8078489 JSpinner using SpinnerNumberModel doesn't honor orientation flag

2015-12-22 Thread Rajeev Chamyal
Component orientation is missing. Fix: Updated the property change listener of Aqua LAF to update editor text orientation. Regards, Rajeev Chamyal

Review request for JDK-8145896 JInternalFrame setMaximum before adding to desktop throws null pointer exception

2015-12-21 Thread Rajeev Chamyal
parent and Desktop pane are missing Fix: Added null checks for parent and desktop pane. Verified the fix on windows,Ubuntu and Mac with all supported LAF. Regards, Rajeev Chamyal

Re: Review request for 8041894: Test javax/swing/JSpinner/8008657/bug8008657.java failed on Mac

2015-12-21 Thread Rajeev Chamyal
Fix looks good to me. Regards, Rajeev Chamyal From: Avik Niyogi Sent: 21 December 2015 09:47 To: Sergey Bylokhov Cc: Rajeev Chamyal; Alexander Scherbatiy; swing-dev@openjdk.java.net Subject: Re: Review request for 8041894: Test javax/swing/JSpinner/8008657/bug8008657.java failed on Mac

Re: Review request for JDK-4769772 JInternalFrame.setIcon(true) before JDesktopPane.add(JIF) causes wrong state

2015-12-21 Thread Rajeev Chamyal
Hello Sergey, Yes, similar issue is there with maximize as well. I will raise a separate bug for it. Regards, Rajeev Chamyal -Original Message- From: Sergey Bylokhov Sent: 19 December 2015 03:51 To: Rajeev Chamyal; Alexander Scherbatiy; Prasanta Sadhukhan; swing-dev@openjdk.java.net

Re: Review request for JDK-8075084 JOptionPane.showMessageDialog causes JScrollBar to move

2015-12-21 Thread Rajeev Chamyal
Hello Alexandr, I have updated the fix. Please review it. http://cr.openjdk.java.net/~rchamyal/8075084/webrev.01/ Regards, Rajeev Chamyal -Original Message- From: Alexander Scherbatiy Sent: 10 December 2015 16:59 To: Rajeev Chamyal Cc: Sergey Bylokhov; Prasanta Sadhukhan; swing-dev

Review request for JDK-4769772 JInternalFrame.setIcon(true) before JDesktopPane.add(JIF) causes wrong state

2015-12-18 Thread Rajeev Chamyal
JDesktopPane.java as part of fix. Regards, Rajeev Chamyal

Re: JDK9 Review Request for JDK-7104635 HTMLEditorKit fails to write down some html files

2015-12-17 Thread Rajeev Chamyal
Hello, Gentle reminder for review. Regards, Rajeev Chamyal -Original Message- From: Rajeev Chamyal Sent: 10 December 2015 22:42 To: Sergey Bylokhov; Alexander Scherbatiy; swing-dev@openjdk.java.net; Prasanta Sadhukhan Subject: RE: JDK9 Review Request for JDK-7104635 HTMLEditorKit

Review request for JDK-8145060 Minimizing a JInternal frame not shifting focus to frame below it

2015-12-13 Thread Rajeev Chamyal
has already updated the cache so Current frame is not found in cache and focus shift fails. Fix: Internal frame removal from container is done after focus shift. Regards, Rajeev Chamyal

Re: JDK9 Review Request for JDK-7104635 HTMLEditorKit fails to write down some html files

2015-12-10 Thread Rajeev Chamyal
Hello Sergey, Thanks for the review. indentLevel is decremented in AbstractWriter :: incrIndent method as well. Here also we have a check before decrementing indentLevel. Regards, Rajeev Chamyal -Original Message- From: Sergey Bylokhov Sent: 09 December 2015 01:49 To: Rajeev Chamyal

Re: Review request for JDK-8075084 JOptionPane.showMessageDialog causes JScrollBar to move

2015-12-03 Thread Rajeev Chamyal
Hello Alexandr, Thanks for the review. When we open a JOption dialog from AdjustmentListener the scroll bar arrow button is not receiving the mouse release event. As a result the JScrollBar: scrollTimer is not getting stopped and its becoming a recursive call. Regards, Rajeev Chamyal

Re: Review Request for 8139169: Action registered for keyboard shortcut is called twice

2015-12-02 Thread Rajeev Chamyal
Hello Avik,   Can you use @requires (os.family == "mac") instead of a method to check the OS type.   Regards, Rajeev Chamyal   From: Avik Niyogi Sent: 30 November 2015 17:22 To: Rajeev Chamyal; Alexander Scherbatiy; Sergey Bylokhov; swing-dev@openjdk.java.net Subject: Review R

Re: JDK9 Review Request for 8067660: JFileChooser create new folder fails silently

2015-11-29 Thread Rajeev Chamyal
t disabled on windows. Regards, Rajeev Chamyal -Original Message- From: Alexandr Scherbatiy Sent: 27 November 2015 19:21 To: Rajeev Chamyal Cc: Prasanta Sadhukhan; Sergey Bylokhov; swing-dev@openjdk.java.net Subject: Re: JDK9 Review Request for 8067660: JFileChooser create new fo

Re: JDK9 Review Request for 8067660: JFileChooser create new folder fails silently

2015-11-27 Thread Rajeev Chamyal
Hello Alexandr, I have updated the webrev as per review comments. http://cr.openjdk.java.net/~rchamyal/8067660/webrev.01/ Attached image shows the error message. Regards, Rajeev Chamyal -Original Message- From: Alexander Scherbatiy Sent: 26 November 2015 16:18 To: Rajeev Chamyal Cc

JDK9 Review Request for JDK-7104635 HTMLEditorKit fails to write down some html files

2015-11-26 Thread Rajeev Chamyal
AbstractWriter::indent method is passing negative length of indentChars to writer. Fix: Added checks for negative value while decrementing the AbstractWriter::indentlevel. Regards, Rajeev Chamyal

Re: JDK9 Review Request for 8067660: JFileChooser create new folder fails silently

2015-11-25 Thread Rajeev Chamyal
, along with all necessary parent directories; false otherwise. Please let me know if your inputs on this.   Regards, Rajeev Chamyal   From: Alexandr Scherbatiy Sent: 25 November 2015 21:30 To: Rajeev Chamyal Cc: Prasanta Sadhukhan; Sergey Bylokhov; swing-dev@openjdk.java.net Subject: Re: JDK9

JDK9 Review Request for 8067660: JFileChooser create new folder fails silently

2015-11-19 Thread Rajeev Chamyal
from GenericFileSystemView:: createNewFolder, WindowsFileSystemView:: createNewFolder and UnixFileSystemView:: createNewFolder if mkdirs call returns false. Regards, Rajeev Chamyal

Re: Review request for 7124218: Space should select cell in the JTable

2015-11-17 Thread Rajeev Chamyal
The fix looks good to me. Regards, Rajeev Chamyal -Original Message- From: Alexander Scherbatiy Sent: 17 November 2015 15:34 To: Avik Niyogi Cc: Rajeev Chamyal; swing-dev@openjdk.java.net Subject: Re: Review request for 7124218: Space should select cell in the JTable The fix looks

Re: Review for 8132770: Test javax/swing/JRadioButton/FocusTraversal/FocusTraversal.java fails in MacOSX

2015-11-17 Thread Rajeev Chamyal
The fix looks good to me. Regards, Rajeev Chamyal From: Avik Niyogi Sent: 17 November 2015 11:16 To: Rajeev Chamyal Cc: Alexander Scherbatiy; swing-dev@openjdk.java.net Subject: Re: Review for 8132770: Test javax/swing/JRadioButton/FocusTraversal/FocusTraversal.java fails in MacOSX

Re: Review request for JDK-6288609 JInternalFrame.setDefaultCloseOperation() interferes with "close" behavior

2015-11-16 Thread Rajeev Chamyal
Hello Prasanta, Please review the updated webrev. I have updated it as per review comments. http://cr.openjdk.java.net/~rchamyal/6288609/webrev.01 Regards, Rajeev Chamyal -Original Message- From: prasanta sadhukhan Sent: 16 November 2015 12:32 To: Rajeev Chamyal Cc: Alexander

Re: RFR: [9] [JDK-8081491] The case print incomplete.

2015-11-15 Thread Rajeev Chamyal
The fix looks good to me.   Regards, Rajeev Chamyal   Subject: Re: RFR: [9] [JDK-8081491] The case print incomplete. Date: Fri, 16 Oct 2015 13:19:37 +0300 From: Alexander Scherbatiy HYPERLINK "mailto:alexandr.scherba...@oracle.com"; To: prasanta sadhukhan

Re: Review for 8132770: Test javax/swing/JRadioButton/FocusTraversal/FocusTraversal.java fails in MacOSX

2015-11-15 Thread Rajeev Chamyal
Hello Avik, In case of test failure also frame should be disposed. Please add a dispose call before RuntimeException line 190. Regards, Rajeev Chamyal From: Avik Niyogi Sent: 16 November 2015 12:20 To: Rajeev Chamyal Cc: Alexander Scherbatiy; swing-dev@openjdk.java.net Subject: Re

Re: Review request for JDK-8030099 Memory usage of java process increases after pressing start button in test window

2015-11-15 Thread Rajeev Chamyal
Hello Phil, I have updated the test case as per your review comments. Could you please review it. http://cr.openjdk.java.net/~rchamyal/8030099/webrev.02/ Regards, Rajeev Chamyal -Original Message- From: Sergey Bylokhov Sent: 12 November 2015 16:02 To: Rajeev Chamyal; Philip Race

Review request for JDK-8075084 JOptionPane.showMessageDialog causes JScrollBar to move

2015-11-12 Thread Rajeev Chamyal
repeatedly till it reaches the other end of scrollbar. Fix: Added checks in the BasicScrollBarUI action listener to stop the recursion. Regards, Rajeev Chamyal

Re: Review for 8132770: Test javax/swing/JRadioButton/FocusTraversal/FocusTraversal.java fails in MacOSX

2015-11-12 Thread Rajeev Chamyal
Hello Avik, 1) In the tryLookAndFeel exception handling is skipped. createUI and other calls should be skipped in case of exception.. 2) What all platforms you have tested the bug? Regards, Rajeev Chamyal From: Avik Niyogi Sent: 04 November 2015 14:51 To: Rajeev Chamyal

Re: Review request for JDK-6288609 JInternalFrame.setDefaultCloseOperation() interferes with "close" behavior

2015-11-12 Thread Rajeev Chamyal
themselves. I have tested the fix with all look and feels on Windows and Ubuntu and on MAC with Aqua LAF as well. Regards, Rajeev Chamyal -Original Message- From: Alexander Scherbatiy Sent: 06 November 2015 19:41 To: Rajeev Chamyal Cc: Sergey Bylokhov; swing-dev@openjdk.java.net Subject

Re: Review request for JDK-8079253 Test javax/swing/SwingUtilities/TestBadBreak/TestBadBreak.java fails

2015-11-10 Thread Rajeev Chamyal
Hello, Please review the updated webrev. http://cr.openjdk.java.net/~rchamyal/8079253/webrev.01/ Removed the line JFrame.setDefaultLookAndFeelDecorated as Frame size issue will be fixed once JDK-8073710 is resolved. Updated the bug description for JDK-8079253. Regards, Rajeev Chamyal

Re: Review request for JDK-8079253 Test javax/swing/SwingUtilities/TestBadBreak/TestBadBreak.java fails

2015-11-06 Thread Rajeev Chamyal
Hello Alexander, Thanks for the review. We need to set the opaque property on the label as well. Regards, Rajeev Chamyal -Original Message- From: Alexander Scherbatiy Sent: 06 November 2015 15:39 To: Rajeev Chamyal Cc: Sergey Bylokhov; swing-dev@openjdk.java.net Subject: Re: Review

Review request for JDK-8079253 Test javax/swing/SwingUtilities/TestBadBreak/TestBadBreak.java fails

2015-11-06 Thread Rajeev Chamyal
e. Also we need set the opaque property on label component which makes sure that all of the bits contained within component's rectangular bounds are painted. Fix: Setting the default look and feel decoration for frame size issue and making label opaque. Regards, Rajeev Chamyal

Re: Review request for JDK-8030099 Memory usage of java process increases after pressing start button in test window

2015-11-02 Thread Rajeev Chamyal
Hello Phil, Please review the updated webrev. http://cr.openjdk.java.net/~rchamyal/8030099/webrev.02/ The test case has been updated as per review comments. Added tests for both Parallel and default collector. Regards, Rajeev Chamyal -Original Message- From: Phil Race Sent: 30

Re: Review request for JDK-8030099 Memory usage of java process increases after pressing start button in test window

2015-10-29 Thread Rajeev Chamyal
Hello Sergey, Please review the updated test case. Webrev : http://cr.openjdk.java.net/~rchamyal/8030099/webrev.01/ As this issue is reported for Parallel GC collector. So, I have added -XX:+UseParallelGC to child process VM arguments in test case. Regards, Rajeev Chamyal -Original

Review request for JDK-6288609 JInternalFrame.setDefaultCloseOperation() interferes with "close" behavior

2015-10-29 Thread Rajeev Chamyal
by dispose method DefaultDeskTopManager:closeFrame is unable to find reference to previous frame. Fix: Removed the selection change code from Dispose method. Regards, Rajeev Chamyal

Re: JDK9 Review Request for 8138881: Bug in OSInfo.java

2015-10-20 Thread Rajeev Chamyal
Hello Sergey, I have updated the test below is the updated webrev. Webrev : http://cr.openjdk.java.net/~rchamyal/8138881/webrev.02/ Regards, Rajeev Chamyal -Original Message- From: Sergey Bylokhov Sent: 20 October 2015 04:30 To: Rajeev Chamyal; Alexander Scherbatiy; swing-dev

Re: Review request for JDK-8030099 Memory usage of java process increases after pressing start button in test window

2015-10-20 Thread Rajeev Chamyal
.) JDK version build 1.9.0-ea-b86. Max java process size after fix while running test case(webrev) : 130MB(approx.). JDK9 version 1.9.0-internal-_2015_10_15_21_12-b00. Regards, Rajeev Chamyal -Original Message- From: Rajeev Chamyal Sent: 20 October 2015 12:10 To: Sergey Bylokhov; Philip

Re: Review request for JDK-8030099 Memory usage of java process increases after pressing start button in test window

2015-10-19 Thread Rajeev Chamyal
different reference types in different files ( files registering instances with Disposer) e.g. StrikeCache uses SoftReferences. Regards, Rajeev Chamyal -Original Message- From: Sergey Bylokhov Sent: 20 October 2015 03:54 To: Philip Race; Rajeev Chamyal Cc: Alexander Scherbatiy; swing

Re: Review request for JDK-8030099 Memory usage of java process increases after pressing start button in test window

2015-10-19 Thread Rajeev Chamyal
Hello Phil, Thanks for the review. Yes, I have run jtreg on it. Regards, Rajeev Chamyal From: Philip Race Sent: 20 October 2015 03:21 To: Rajeev Chamyal Cc: Sergey Bylokhov; Alexander Scherbatiy; swing-dev@openjdk.java.net Subject: Re: Review request for JDK-8030099 Memory usage

Review request for JDK-8030099 Memory usage of java process increases after pressing start button in test window

2015-10-19 Thread Rajeev Chamyal
, Rajeev Chamyal

Re: JDK9 Review Request for 8138881: Bug in OSInfo.java

2015-10-18 Thread Rajeev Chamyal
Hello Sergey, Thanks for the review. The test was working before the fix as well. The current implementation of OSInfo.getWindowsVersion() adds the missing values also to windowsVersionMap. So test always passes. Regards, Rajeev Chamyal -Original Message- From: Sergey Bylokhov Sent

Re: JDK9 Review Request for 8138881: Bug in OSInfo.java

2015-10-15 Thread Rajeev Chamyal
Hello Sergey, Thanks for the review. I have updated the webrev with review comments. WebRev : http://cr.openjdk.java.net/~rchamyal/8138881/webrev.01/ Regards, Rajeev Chamyal -Original Message- From: Sergey Bylokhov Sent: 14 October 2015 20:44 To: Rajeev Chamyal; Alexander Scherbatiy

JDK9 Review Request for 8138881: Bug in OSInfo.java

2015-10-14 Thread Rajeev Chamyal
Hello, Please review the following fix for Jdk9: WebRev : http://cr.openjdk.java.net/~rchamyal/8138881/webrev.00/ Bug Id: https://bugs.openjdk.java.net/browse/JDK-8138881 Issue: There was a typo in OSInfo.java. Regards, Rajeev Chamyal

Re: JDK9 Review Request for 8037575: JFrame on Windows doesn't animate when setting ICONIFIED state

2015-10-13 Thread Rajeev Chamyal
Hello All, Please review the following webrev. Bug: https://bugs.openjdk.java.net/browse/JDK-8037575 Webrev: http://cr.openjdk.java.net/~psadhukhan/rajeev/8037575/webrev.00/ Regards, Rajeev Chamyal From: Rajeev Chamyal Sent: 29 September 2015 16:41 To: Sergey Bylokhov; Alexander

Re: Review request for JDK-8067346: Swing submenu has a changed starting offset

2015-10-05 Thread Rajeev Chamyal
Hello Alexey, I have created the following bug and assigned it to me. https://bugs.openjdk.java.net/browse/JDK-8138881 Regards, Rajeev Chamyal -Original Message- From: Alexey Ivanov Sent: Monday, October 05, 2015 6:57 PM To: Rajeev Chamyal; Alexander Scherbatiy Cc: swing-dev

Re: Review request for JDK-8067346: Swing submenu has a changed starting offset

2015-10-04 Thread Rajeev Chamyal
Hello Alexey, Thanks for noticing it. Yes, we need to resolve it. Alexander, Should I create a new bug for this or I can submit against the existing bug. Regards, Rajeev Chamyal -Original Message- From: Alexey Ivanov Sent: Friday, October 02, 2015 7:30 PM To: Alexander Scherbatiy

Re: Review request for JDK-8067346: Swing submenu has a changed starting offset

2015-09-30 Thread Rajeev Chamyal
Hello Alexander, Thanks for the review. I have updated webrev as per review comments. Bug: https://bugs.openjdk.java.net/browse/JDK-8067346 Webrev: http://cr.openjdk.java.net/~psadhukhan/rajeev/8067346/webrev.02/ Regards, Rajeev Chamyal -Original Message- From: Alexander Scherbatiy

JDK9 Review Request for 8037575: JFrame on Windows doesn't animate when setting ICONIFIED state

2015-09-29 Thread Rajeev Chamyal
. Fix: Added a call to ShowWindow function to set the specified Frame show state ICONIFIED/ZOOM etc. Regards, Rajeev Chamyal

Re: Review request for JDK-8067346: Swing submenu has a changed starting offset

2015-09-29 Thread Rajeev Chamyal
Hello All, Please review the following updated webrev. Bug: https://bugs.openjdk.java.net/browse/JDK-8067346 Webrev: http://cr.openjdk.java.net/~psadhukhan/rajeev/8067346/webrev.01/ Verified the fix on windows10 and windows8. Regards, Rajeev Chamyal -Original Message- From: Sergey

Review request for JDK-8067346: Swing submenu has a changed starting offset

2015-09-22 Thread Rajeev Chamyal
. Current implementation is applying Vista defaults to Windows7 as well and as a result submenu on windows7 has changed offset. Added checks in WindowsLookAndFeel.java to skip Vista menu defaults for Windows7. Verified changes on Netbeans9 Dev build and SwingSet2 applications. Regards, Rajeev

Re: Bug: JDK-7190596 Nimbus: preferred sizes of components not less than that of tabbed pane

2015-09-20 Thread Rajeev Chamyal
Hello Alexander, Thanks for the review. Please push the changes to JDK9. Regards, Rajeev Chamyal -Original Message- From: Alexander Scherbatiy Sent: Friday, September 18, 2015 9:22 PM To: Rajeev Chamyal Cc: Alexander Zvegintsev; swing-dev@openjdk.java.net; Sergey Bylokhov Subject: Re

Re: Bug: JDK-7190596 Nimbus: preferred sizes of components not less than that of tabbed pane

2015-09-17 Thread Rajeev Chamyal
/7190596/webrev.00/ Regards, Rajeev Chamyal -Original Message- From: Alexander Scherbatiy Sent: Thursday, September 17, 2015 8:23 PM To: Rajeev Chamyal Cc: Alexander Zvegintsev; swing-dev@openjdk.java.net; Sergey Bylokhov Subject: Re: Bug: JDK-7190596 Nimbus: preferred sizes of components not

Bug: JDK-7190596 Nimbus: preferred sizes of components not less than that of tabbed pane

2015-09-16 Thread Rajeev Chamyal
Hi All, Please review the following fix for jdk9: Bug: https://bugs.openjdk.java.net/browse/JDK-7190596 Webrev : http://cr.openjdk.java.net/~psadhukhan/rajeev/7190596/webrev.00/ Test being moved from closed to open repository. Regards, Rajeev Chamyal

Re: Request for review fo bug JDK-8032568 Test javax/swing/JInternalFrame/8020708/bug8020708.java fails on Windows virtual hosts

2015-09-09 Thread Rajeev Chamyal
Hi Alexander, Could you please push the fix to JDK9. Regards, Rajeev Chamyal -Original Message- From: Sergey Bylokhov Sent: Wednesday, September 09, 2015 6:03 PM To: Rajeev Chamyal; Alexander Zvegintsev; swing-dev@openjdk.java.net; Alexander Scherbatiy Subject: Re: Request for review

Re: Request for review fo bug JDK-8032568 Test javax/swing/JInternalFrame/8020708/bug8020708.java fails on Windows virtual hosts

2015-09-09 Thread Rajeev Chamyal
Hello All, I need one more review for this fix. Can someone please review it. Webrev : http://cr.openjdk.java.net/~psadhukhan/rajeev/8032568/webrev.02/ Regards, Rajeev Chamyal -Original Message- From: Rajeev Chamyal Sent: Tuesday, September 08, 2015 2:38 PM To: Sergey Bylokhov

Re: Request for review fo bug JDK-8032568 Test javax/swing/JInternalFrame/8020708/bug8020708.java fails on Windows virtual hosts

2015-09-08 Thread Rajeev Chamyal
Hello All, I need one more review for this fix. Can someone please review it. Webrev : http://cr.openjdk.java.net/~psadhukhan/rajeev/8032568/webrev.02/ Regards, Rajeev Chamyal -Original Message- From: Alexander Scherbatiy Sent: Tuesday, September 08, 2015 2:27 PM To: Rajeev Chamyal Cc

Re: Request for review fo bug JDK-8032568 Test javax/swing/JInternalFrame/8020708/bug8020708.java fails on Windows virtual hosts

2015-09-08 Thread Rajeev Chamyal
Hello All, Please review the below webrev. http://cr.openjdk.java.net/~psadhukhan/rajeev/8032568/webrev.02/ Regards, Rajeev Chamyal -Original Message- From: Alexander Scherbatiy Sent: Monday, September 07, 2015 6:21 PM To: Rajeev Chamyal Cc: Philip Race; Alexander Zvegintsev; swing

Re: Request for review fo bug JDK-8032568 Test javax/swing/JInternalFrame/8020708/bug8020708.java fails on Windows virtual hosts

2015-09-06 Thread Rajeev Chamyal
Hello All, Please review the following webrev. I have updated the fix as per review comments. http://cr.openjdk.java.net/~psadhukhan/rajeev/8032568/webrev.01/ Regards, Rajeev Chamyal -Original Message- From: Phil Race Sent: Thursday, September 03, 2015 11:26 PM To: Rajeev Chamyal Cc

Re: Request for review fo bug JDK-8032568 Test javax/swing/JInternalFrame/8020708/bug8020708.java fails on Windows virtual hosts

2015-09-03 Thread Rajeev Chamyal
Hello All, I need one more review for this fix. Can someone please review it. Webrev : http://cr.openjdk.java.net/~psadhukhan/rajeev/8032568/webrev.00/ Regards, Rajeev Chamyal -Original Message- From: Sergey Bylokhov Sent: Thursday, September 03, 2015 9:05 PM To: Rajeev Chamyal

Request for review fo bug JDK-8032568 Test javax/swing/JInternalFrame/8020708/bug8020708.java fails on Windows virtual hosts

2015-09-02 Thread Rajeev Chamyal
different languages. For example in German close mnemonic is now S and for French its now F. To fix the issue we are now finding the correct mnemonic character and passing it to robo. Regards, Rajeev Chamyal

Re: Bug: JDK-7190596 Nimbus: preferred sizes of components not less than that of tabbed pane

2015-09-02 Thread Rajeev Chamyal
Hello All, Please review the below webrev. Bug: https://bugs.openjdk.java.net/browse/JDK-7190596 Webrev : http://cr.openjdk.java.net/~psadhukhan/rajeev/7190596/webrev.00/ Regards, Rajeev Chamyal From: Rajeev Chamyal Sent: Monday, August 31, 2015 6:14 PM To: Alexander Scherbatiy

Re: Bug : JDK-8078831 Mismatch of getPreferredSize() and getPreferredScrollableViewportSize() values in WindowsClassicLookAndFeel

2015-09-01 Thread Rajeev Chamyal
Hello All, I need one more review for this fix. Can someone please review it. http://cr.openjdk.java.net/~psadhukhan/rajeev/5042886/webrev.open/ http://cr.openjdk.java.net/~psadhukhan/rajeev/5042886/webrev.closed/ Regards, Rajeev Chamyal -Original Message- From: Rajeev Chamyal Sent

Bug: JDK-7190596 Nimbus: preferred sizes of components not less than that of tabbed pane

2015-08-31 Thread Rajeev Chamyal
and child components. Added a default Insets value for Nimbus look and feel, similar to other layouts. Regards, Rajeev Chamyal

Re: Bug : JDK-8078831 Mismatch of getPreferredSize() and getPreferredScrollableViewportSize() values in WindowsClassicLookAndFeel

2015-08-31 Thread Rajeev Chamyal
Hi All, Please review the following webrev. http://cr.openjdk.java.net/~psadhukhan/rajeev/5042886/webrev.open/ http://cr.openjdk.java.net/~psadhukhan/rajeev/5042886/webrev.closed/ Test has been moved to open repository. Regards, Rajeev Chamyal -Original Message- From: Alexander

Re: RFR: [JDK-8025082] The behaviour of the highlight will be lost after clicking the set button

2015-08-30 Thread Rajeev Chamyal
Hello Sergey, Please review the updated webrev. http://cr.openjdk.java.net/~psadhukhan/rajeev/8025082/webrev.03/ Regards, Rajeev Chamyal -Original Message- From: Sergey Bylokhov Sent: Thursday, August 27, 2015 6:18 PM To: Rajeev Chamyal; Alexander Scherbatiy; Alexander Zvegintsev

Re: RFR: [JDK-8025082] The behaviour of the highlight will be lost after clicking the set button

2015-08-27 Thread Rajeev Chamyal
Hello All, Please review the updated webrev. Webrev: http://cr.openjdk.java.net/~psadhukhan/rajeev/8025082/webrev.02/ Bug: https://bugs.openjdk.java.net/browse/JDK-8025082 Regards, Rajeev Chamyal -Original Message- From: Alexander Scherbatiy Sent: Wednesday, August 26, 2015 6:39 PM

Re: Bug : JDK-8078831 Mismatch of getPreferredSize() and getPreferredScrollableViewportSize() values in WindowsClassicLookAndFeel

2015-08-27 Thread Rajeev Chamyal
Hello All, Please review the updated fix with latest review comments. http://cr.openjdk.java.net/~psadhukhan/rajeev/5042886/webrev.01/ Regards, Rajeev Chamyal -Original Message- From: Alexander Scherbatiy Sent: Wednesday, August 26, 2015 6:42 PM To: Rajeev Chamyal Cc: Alexander

Bug : JDK-8078831 Mismatch of getPreferredSize() and getPreferredScrollableViewportSize() values in WindowsClassicLookAndFeel

2015-08-26 Thread Rajeev Chamyal
less size. Test condition was checking for inequality and throwing Runtime exception. Regards, Rajeev Chamyal

Re: RFR: [JDK-8025082] The behaviour of the highlight will be lost after clicking the set button

2015-08-26 Thread Rajeev Chamyal
Hello All, Please review the updated fix for bug JDK-8025082. Bug: https://bugs.openjdk.java.net/browse/JDK-8025082 Webrev : http://cr.openjdk.java.net/~psadhukhan/rajeev/8025082/webrev.01/ Regards, Rajeev Chamyal -Original Message- From: Alexander Scherbatiy Sent: Tuesday, August

Re: RFR: [JDK-8025082] The behaviour of the highlight will be lost after clicking the set button

2015-08-25 Thread Rajeev Chamyal
Hello All, Please review the below fix for JDK9. Bug: https://bugs.openjdk.java.net/browse/JDK-8025082 webrev: http://cr.openjdk.java.net/~psadhukhan/rajeev/8025082/webrev.00/ Regards, Rajeev Chamyal From: Rajeev Chamyal Sent: Friday, August 21, 2015 5:40 PM To: swing-dev

RFR: [JDK-8025082] The behaviour of the highlight will be lost after clicking the set button

2015-08-21 Thread Rajeev Chamyal
focus. Added a method to update the selection ownership for caret when text is selected/unselected in the JTextPane/JTextArea. Regards, Rajeev Chamyal

<    1   2