Re: RFR: 8182043: Access to Windows Large Icons [v16]

2021-05-26 Thread Alexander Zuev
> Fix updated after first round of review. Alexander Zuev has updated the pull request incrementally with one additional commit since the last revision: Test comment fixed. - Changes: - all: https://git.openjdk.java.net/jdk/pull/2875/files - new: https://git.openjdk.java.net/

Re: RFR: 8182043: Access to Windows Large Icons [v15]

2021-05-26 Thread Alexander Zuev
On Wed, 26 May 2021 21:06:49 GMT, Sergey Bylokhov wrote: > Did somebody run the test on Linux and macOS, Does our stub implementation > there confirms the specification? I did run fill tier4 and visually compared the JFileChooser appearance on both Linux and Mac OS X - this change introduced n

Re: RFR: 8182043: Access to Windows Large Icons [v15]

2021-05-26 Thread Alexander Zuev
On Wed, 26 May 2021 16:52:38 GMT, Alexander Zuev wrote: >> Fix updated after first round of review. > > Alexander Zuev has updated the pull request incrementally with two additional > commits since the last revision: > > - Fixed small errors >Adjustments in the test > - Grammar fix in met

Re: RFR: 8182043: Access to Windows Large Icons [v15]

2021-05-26 Thread Alexander Zuev
On Thu, 27 May 2021 00:18:13 GMT, Phil Race wrote: >> Alexander Zuev has updated the pull request incrementally with two >> additional commits since the last revision: >> >> - Fixed small errors >>Adjustments in the test >> - Grammar fix in method documentation > > test/jdk/javax/swing/JF

Re: RFR: 8182043: Access to Windows Large Icons [v15]

2021-05-26 Thread Phil Race
On Wed, 26 May 2021 16:52:38 GMT, Alexander Zuev wrote: >> Fix updated after first round of review. > > Alexander Zuev has updated the pull request incrementally with two additional > commits since the last revision: > > - Fixed small errors >Adjustments in the test > - Grammar fix in met

Re: RFR: 8182043: Access to Windows Large Icons [v15]

2021-05-26 Thread Phil Race
On Wed, 26 May 2021 21:37:44 GMT, Alexander Zuev wrote: >> test/jdk/javax/swing/JFileChooser/FileSystemView/SystemIconTest.java line 77: >> >>> 75: int[] sizes = new int[] {16, 32, 48, 64, 128}; >>> 76: for (int size : sizes) { >>> 77: ImageIcon icon = (ImageIcon) fsv

Re: RFR: 8182043: Access to Windows Large Icons [v15]

2021-05-26 Thread Alexander Zuev
On Wed, 26 May 2021 21:12:16 GMT, Sergey Bylokhov wrote: >> Alexander Zuev has updated the pull request incrementally with two >> additional commits since the last revision: >> >> - Fixed small errors >>Adjustments in the test >> - Grammar fix in method documentation > > test/jdk/javax/sw

Re: RFR: 8182043: Access to Windows Large Icons [v15]

2021-05-26 Thread Sergey Bylokhov
On Wed, 26 May 2021 16:52:38 GMT, Alexander Zuev wrote: >> Fix updated after first round of review. > > Alexander Zuev has updated the pull request incrementally with two additional > commits since the last revision: > > - Fixed small errors >Adjustments in the test > - Grammar fix in met

Re: RFR: 8182043: Access to Windows Large Icons [v7]

2021-05-26 Thread Sergey Bylokhov
On Fri, 21 May 2021 19:37:19 GMT, Alexander Zuev wrote: >> Didn't you answer your question already? If `FileSystemView.getRoots()` >> returns Desktop in Windows shell namespace. Otherwise, I don't know a way to >> get a reference to a *virtual* folder in Windows shell namespace which >> doesn'

Re: RFR: 8182043: Access to Windows Large Icons [v15]

2021-05-26 Thread Sergey Bylokhov
On Wed, 26 May 2021 16:52:38 GMT, Alexander Zuev wrote: >> Fix updated after first round of review. > > Alexander Zuev has updated the pull request incrementally with two additional > commits since the last revision: > > - Fixed small errors >Adjustments in the test > - Grammar fix in met

Re: RFR: 8266079: Lanai: AlphaComposite shows differences on Metal compared to OpenGL

2021-05-26 Thread Sergey Bylokhov
On Wed, 26 May 2021 13:49:24 GMT, Alexey Ushakov wrote: > Set black color for underlying window background to perform correct blending > operations in metal with window surface destination I think this will resurrect the JDK-8033786, but with a different color - black instead of white. --

Re: RFR: 8182043: Access to Windows Large Icons [v14]

2021-05-26 Thread Alexey Ivanov
On Wed, 26 May 2021 17:40:44 GMT, Phil Race wrote: >> src/java.desktop/share/classes/javax/swing/filechooser/FileSystemView.java >> line 300: >> >>> 298: >>> 299: if(!f.exists()) { >>> 300: return null; >> >> Shall it throw `FileNotFoundException` or `IllegalArgumentExcept

Re: RFR: 8264125: Specification of Taskbar::getIconImage doesn't mention that the returned image might not be equal to the Taskbar::setIconImage one. (eg on Mac OS) [v2]

2021-05-26 Thread Alexander Zvegintsev
> This fix is to explicitly specify that `Taskbar::getIconImage` may return an > object different from passed to `Taskbar::setIconImage`. > > Actually it is always returns a different object on macOS(the only OS which > supports this feature), but I want to save some options if we decide to > r

Re: RFR: 8182043: Access to Windows Large Icons [v14]

2021-05-26 Thread Phil Race
On Wed, 26 May 2021 15:15:42 GMT, Alexey Ivanov wrote: >> Alexander Zuev has updated the pull request incrementally with one >> additional commit since the last revision: >> >> null file now properly causes IllegalArgumentException >> Small fixed in JavaDoc > > src/java.desktop/share/classe

Re: RFR: 8182043: Access to Windows Large Icons [v15]

2021-05-26 Thread Alexey Ivanov
On Wed, 26 May 2021 16:52:38 GMT, Alexander Zuev wrote: >> Fix updated after first round of review. > > Alexander Zuev has updated the pull request incrementally with two additional > commits since the last revision: > > - Fixed small errors >Adjustments in the test > - Grammar fix in met

Re: RFR: 8182043: Access to Windows Large Icons [v15]

2021-05-26 Thread Alexander Zuev
> Fix updated after first round of review. Alexander Zuev has updated the pull request incrementally with two additional commits since the last revision: - Fixed small errors Adjustments in the test - Grammar fix in method documentation - Changes: - all: https://git.openjdk.

Re: RFR: 8182043: Access to Windows Large Icons [v14]

2021-05-26 Thread Alexander Zuev
On Wed, 26 May 2021 16:03:02 GMT, Alexander Zuev wrote: >> test/jdk/javax/swing/JFileChooser/FileSystemView/SystemIconTest.java line 67: >> >>> 65: if (!gotException) { >>> 66: throw new RuntimeException("Negative size icon should throw >>> invalid argument exception"); >>>

Re: RFR: 8182043: Access to Windows Large Icons [v14]

2021-05-26 Thread Alexander Zuev
On Wed, 26 May 2021 15:07:30 GMT, Alexey Ivanov wrote: >> Alexander Zuev has updated the pull request incrementally with one >> additional commit since the last revision: >> >> null file now properly causes IllegalArgumentException >> Small fixed in JavaDoc > > src/java.desktop/share/classe

Integrated: 8263202: Update Hebrew/Indonesian/Yiddish ISO 639 language codes to current

2021-05-26 Thread Naoto Sato
On Mon, 17 May 2021 16:55:35 GMT, Naoto Sato wrote: > Please review the changes to the subject issue. java.util.Locale class has a > long-standing issue for those obsolete ISO 639 languages where its > normalization ends up in the obsolete codes. This change intends to flip the > normalization

Re: RFR: 8182043: Access to Windows Large Icons [v14]

2021-05-26 Thread Alexey Ivanov
On Tue, 25 May 2021 23:36:43 GMT, Alexander Zuev wrote: >> Fix updated after first round of review. > > Alexander Zuev has updated the pull request incrementally with one additional > commit since the last revision: > > null file now properly causes IllegalArgumentException > Small fixed in

Re: RFR: 8182043: Access to Windows Large Icons [v14]

2021-05-26 Thread Phil Race
On Tue, 25 May 2021 23:36:43 GMT, Alexander Zuev wrote: >> Fix updated after first round of review. > > Alexander Zuev has updated the pull request incrementally with one additional > commit since the last revision: > > null file now properly causes IllegalArgumentException > Small fixed in

RFR: 8266079: Lanai: AlphaComposite shows differences on Metal compared to OpenGL

2021-05-26 Thread Alexey Ushakov
Set black color for underlying window background to perform correct blending operations in metal with window surface destination - Commit messages: - 8266079: Lanai: AlphaComposite shows differences on Metal compared to OpenGL Changes: https://git.openjdk.java.net/jdk/pull/4206/fil

Integrated: 8264302: Create implementation for Accessibility native peer for Splitpane java role

2021-05-26 Thread Pankaj Bansal
On Tue, 25 May 2021 08:24:21 GMT, Pankaj Bansal wrote: > Create implementation for Accessibility native peer for Splitpane java role. > I have tested it using the SwingSet2 and splitpane demo from > https://docs.oracle.com/javase/tutorial/uiswing/examples/components/index.html#SplitPaneDemo. >

Re: RFR: 8264302: Create implementation for Accessibility native peer for Splitpane java role [v2]

2021-05-26 Thread Alexander Zuev
On Wed, 26 May 2021 04:53:41 GMT, Pankaj Bansal wrote: >> Create implementation for Accessibility native peer for Splitpane java role. >> I have tested it using the SwingSet2 and splitpane demo from >> https://docs.oracle.com/javase/tutorial/uiswing/examples/components/index.html#SplitPaneDemo.