Re: RFR: 8236987: Remove call to System.out.println from ImageIcon.loadImage [v3]

2022-03-23 Thread Tejesh R
> Removed the println() line from the Interrupted catch block. Since > waitForID() Interrupt indicates completion of Image Loading, println as > Interrupt handling was not required. Tejesh R has updated the pull request with a new target base due to a merge or a rebase. The incremental webrev e

Re: RFR: 8283426: Fix 'exeption' typo [v3]

2022-03-23 Thread Andrey Turbanov
> Fix repeated typo `exeption` Andrey Turbanov has updated the pull request incrementally with one additional commit since the last revision: 8283426: Fix 'exeption' typo fix more typos, found by Sean Coffey - Changes: - all: https://git.openjdk.java.net/jdk/pull/7879/files

Re: RFR: JDK-8283015: Create a test for JDK-4715496 [v3]

2022-03-23 Thread Srinivas Mandalika
> Create a test for JDK-4715496 > > Issue observed in the above bug is that the > JTable.AccessibleJTableCell.getAccessible name returns the cell instance > string instead of the cell text. > The test is being added to verify the same. > This review is for migrating tests from a closed test suit

RFR: 8280193: summary javadoc for java.awt.GraphicsEnvironment#preferPropotionalFonts broken

2022-03-23 Thread James Nord
https://bugs.openjdk.java.net/browse/JDK-8280193 Javadoc broke the first sentence at the full stop space (newline) so it rendered with `Indicates a preference for proportional over non-proportional (e.g.` This is an incorrect sentence, and so this change fixes it by replacing the `e.g.` to `f

Re: RFR: 8280193: summary javadoc for java.awt.GraphicsEnvironment#preferPropotionalFonts broken

2022-03-23 Thread Daniel Jeliński
On Tue, 18 Jan 2022 14:05:55 GMT, James Nord wrote: > https://bugs.openjdk.java.net/browse/JDK-8280193 > > > Javadoc broke the first sentence at the full stop space (newline) so it > rendered with `Indicates a preference for proportional over non-proportional > (e.g.` > > This is an incorrec

Re: RFR: 8280193: summary javadoc for java.awt.GraphicsEnvironment#preferPropotionalFonts broken

2022-03-23 Thread James Nord
On Tue, 18 Jan 2022 14:05:55 GMT, James Nord wrote: > https://bugs.openjdk.java.net/browse/JDK-8280193 > > > Javadoc broke the first sentence at the full stop space (newline) so it > rendered with `Indicates a preference for proportional over non-proportional > (e.g.` > > This is an incorrec

Re: RFR: 8282526: Default icon is not painted properly

2022-03-23 Thread Jeremy
On Mon, 14 Mar 2022 02:29:47 GMT, Jeremy wrote: >> Detect the situation where we do need to perform interpolation during >> ImageIcon >> painting and set a hint to the rendering to perform bicubic approximation so >> image details are preserved during transition. > > test/jdk/javax/swing/ImageIc

Re: RFR: JDK-8283015: Create a test for JDK-4715496 [v4]

2022-03-23 Thread Srinivas Mandalika
> Create a test for JDK-4715496 > > Issue observed in the above bug is that the > JTable.AccessibleJTableCell.getAccessible name returns the cell instance > string instead of the cell text. > The test is being added to verify the same. > This review is for migrating tests from a closed test suit

Re: RFR: 8280193: summary javadoc for java.awt.GraphicsEnvironment#preferPropotionalFonts broken

2022-03-23 Thread James Nord
On Tue, 18 Jan 2022 14:05:55 GMT, James Nord wrote: > https://bugs.openjdk.java.net/browse/JDK-8280193 > > > Javadoc broke the first sentence at the full stop space (newline) so it > rendered with `Indicates a preference for proportional over non-proportional > (e.g.` > > This is an incorrec

Re: RFR: JDK-8283015: Create a test for JDK-4715496 [v5]

2022-03-23 Thread Srinivas Mandalika
> Create a test for JDK-4715496 > > Issue observed in the above bug is that the > JTable.AccessibleJTableCell.getAccessible name returns the cell instance > string instead of the cell text. > The test is being added to verify the same. > This review is for migrating tests from a closed test suit

Re: RFR: 8283426: Fix 'exeption' typo [v3]

2022-03-23 Thread Alexey Ivanov
On Wed, 23 Mar 2022 07:31:23 GMT, Andrey Turbanov wrote: >> Fix repeated typo `exeption` > > Andrey Turbanov has updated the pull request incrementally with one > additional commit since the last revision: > > 8283426: Fix 'exeption' typo > fix more typos, found by Sean Coffey Marked as re

Re: RFR: JDK-8282046: Create a regression test for JDK-8000326 [v6]

2022-03-23 Thread Alexey Ivanov
On Wed, 23 Mar 2022 06:19:14 GMT, Srinivas Mandalika wrote: >> Create a regression test for CCC8000326 >> >> Issue is identified by >> [JDK-8000326](https://bugs.openjdk.java.net/browse/JDK-8000326), which >> identifies that after focus moves into JMenuBar, whose focus traversal key >> is di

RFR: 8283323: libharfbuzz optimization level results in extreme build times

2022-03-23 Thread Magnus Ihse Bursie
[JDK-8247872](https://bugs.openjdk.java.net/browse/JDK-8247872) (upgrade HarfBuzz to 2.7.2) caused build time to go up with 24 seconds on my reference linux machine. This was one of the four culprits that caused a 25-30% build time regression over the last two years. The problem here was that t

Re: RFR: 8236987: Remove call to System.out.println from ImageIcon.loadImage [v4]

2022-03-23 Thread Tejesh R
> Removed the println() line from the Interrupted catch block. Since > waitForID() Interrupt indicates completion of Image Loading, println as > Interrupt handling was not required. Tejesh R has updated the pull request incrementally with three additional commits since the last revision: - Me

Re: RFR: 8283323: libharfbuzz optimization level results in extreme build times

2022-03-23 Thread Erik Joelsson
On Wed, 23 Mar 2022 12:25:08 GMT, Magnus Ihse Bursie wrote: > [JDK-8247872](https://bugs.openjdk.java.net/browse/JDK-8247872) (upgrade > HarfBuzz to 2.7.2) caused build time to go up with 24 seconds on my reference > linux machine. This was one of the four culprits that caused a 25-30% build >

Re: RFR: 8236987: Remove call to System.out.println from ImageIcon.loadImage [v5]

2022-03-23 Thread Tejesh R
> Removed the println() line from the Interrupted catch block. Since > waitForID() Interrupt indicates completion of Image Loading, println as > Interrupt handling was not required. Tejesh R has updated the pull request incrementally with one additional commit since the last revision: Update

Re: RFR: 8236987: Remove call to System.out.println from ImageIcon.loadImage [v2]

2022-03-23 Thread Tejesh R
On Tue, 22 Mar 2022 11:30:31 GMT, Alexey Ivanov wrote: >> Tejesh R has updated the pull request incrementally with one additional >> commit since the last revision: >> >> Updated based on review comments > > Changes requested by aivanov (Reviewer). @aivanov-jdk , could you please review now

Re: RFR: 8283323: libharfbuzz optimization level results in extreme build times

2022-03-23 Thread Aleksey Shipilev
On Wed, 23 Mar 2022 12:25:08 GMT, Magnus Ihse Bursie wrote: > [JDK-8247872](https://bugs.openjdk.java.net/browse/JDK-8247872) (upgrade > HarfBuzz to 2.7.2) caused build time to go up with 24 seconds on my reference > linux machine. This was one of the four culprits that caused a 25-30% build >

Re: RFR: 8236987: Remove call to System.out.println from ImageIcon.loadImage [v5]

2022-03-23 Thread Alexey Ivanov
On Wed, 23 Mar 2022 13:55:11 GMT, Tejesh R wrote: >> Removed the println() line from the Interrupted catch block. Since >> waitForID() Interrupt indicates completion of Image Loading, println as >> Interrupt handling was not required. > > Tejesh R has updated the pull request incrementally with

Re: RFR: 8279614: The left line of the TitledBorder is not painted on 150 scale factor [v6]

2022-03-23 Thread Alisen Chung
On Thu, 17 Mar 2022 23:50:21 GMT, Alisen Chung wrote: >> Changed the drawing area to be increased by 0.5 on the left side to prevent >> clipping > > Alisen Chung has updated the pull request incrementally with one additional > commit since the last revision: > > fixed apostrophe in comment o

Re: RFR: 8279614: The left line of the TitledBorder is not painted on 150 scale factor [v7]

2022-03-23 Thread Alisen Chung
> Changed the drawing area to be increased by 0.5 on the left side to prevent > clipping Alisen Chung has updated the pull request incrementally with one additional commit since the last revision: scale stroke width at higher scalings - Changes: - all: https://git.openjdk.java

Re: RFR: 8279614: The left line of the TitledBorder is not painted on 150 scale factor [v6]

2022-03-23 Thread Alexey Ivanov
On Wed, 23 Mar 2022 18:19:58 GMT, Alisen Chung wrote: > I don't have a monitor that can go higher than 175%, so I'm not able to test > the code well... Can you try the recent change to see if it works at 200+%? > If not, I'll try using the area approach. You can pass `-Dsun.java2d.uiScale=2.0`

Re: RFR: 8279614: The left line of the TitledBorder is not painted on 150 scale factor [v6]

2022-03-23 Thread Alisen Chung
On Wed, 23 Mar 2022 18:23:54 GMT, Alexey Ivanov wrote: > > I don't have a monitor that can go higher than 175%, so I'm not able to > > test the code well... Can you try the recent change to see if it works at > > 200+%? If not, I'll try using the area approach. > > You can pass `-Dsun.java2d.u

Re: RFR: 8283323: libharfbuzz optimization level results in extreme build times

2022-03-23 Thread Magnus Ihse Bursie
On Wed, 23 Mar 2022 12:25:08 GMT, Magnus Ihse Bursie wrote: > [JDK-8247872](https://bugs.openjdk.java.net/browse/JDK-8247872) (upgrade > HarfBuzz to 2.7.2) caused build time to go up with 24 seconds on my reference > linux machine. This was one of the four culprits that caused a 25-30% build >

Re: RFR: 8283323: libharfbuzz optimization level results in extreme build times

2022-03-23 Thread Erik Joelsson
On Wed, 23 Mar 2022 12:25:08 GMT, Magnus Ihse Bursie wrote: > [JDK-8247872](https://bugs.openjdk.java.net/browse/JDK-8247872) (upgrade > HarfBuzz to 2.7.2) caused build time to go up with 24 seconds on my reference > linux machine. This was one of the four culprits that caused a 25-30% build >

Re: RFR: 8283323: libharfbuzz optimization level results in extreme build times

2022-03-23 Thread Phil Race
On Wed, 23 Mar 2022 12:25:08 GMT, Magnus Ihse Bursie wrote: > [JDK-8247872](https://bugs.openjdk.java.net/browse/JDK-8247872) (upgrade > HarfBuzz to 2.7.2) caused build time to go up with 24 seconds on my reference > linux machine. This was one of the four culprits that caused a 25-30% build >

Re: RFR: 8283426: Fix 'exeption' typo [v4]

2022-03-23 Thread Andrey Turbanov
> Fix repeated typo `exeption` Andrey Turbanov has updated the pull request incrementally with one additional commit since the last revision: 8283426: Fix 'exeption' typo Co-authored-by: Alexey Ivanov <[email protected]> - Changes: - all: https:

RFR: 8273355: Flickering on tooltip appearance IntelliJ IDEA 2021.2.1

2022-03-23 Thread Alexey Ushakov
Used setOpaque() method to set correct background of platform window Extracted setting background to the particular method in PlatformWindow. Provided appropriate implementation in CPlatformWindow. - Commit messages: - 8273355: Flickering on tooltip appearance IntelliJ IDEA 2021.2.1

Re: RFR: 8283323: libharfbuzz optimization level results in extreme build times

2022-03-23 Thread Magnus Ihse Bursie
On Wed, 23 Mar 2022 19:35:22 GMT, Phil Race wrote: >> [JDK-8247872](https://bugs.openjdk.java.net/browse/JDK-8247872) (upgrade >> HarfBuzz to 2.7.2) caused build time to go up with 24 seconds on my >> reference linux machine. This was one of the four culprits that caused a >> 25-30% build time

Re: RFR: 8283323: libharfbuzz optimization level results in extreme build times

2022-03-23 Thread Phil Race
On Wed, 23 Mar 2022 12:25:08 GMT, Magnus Ihse Bursie wrote: > [JDK-8247872](https://bugs.openjdk.java.net/browse/JDK-8247872) (upgrade > HarfBuzz to 2.7.2) caused build time to go up with 24 seconds on my reference > linux machine. This was one of the four culprits that caused a 25-30% build >

Re: RFR: 8283323: libharfbuzz optimization level results in extreme build times [v2]

2022-03-23 Thread Magnus Ihse Bursie
> [JDK-8247872](https://bugs.openjdk.java.net/browse/JDK-8247872) (upgrade > HarfBuzz to 2.7.2) caused build time to go up with 24 seconds on my reference > linux machine. This was one of the four culprits that caused a 25-30% build > time regression over the last two years. > > The problem her

Re: RFR: 8283323: libharfbuzz optimization level results in extreme build times [v2]

2022-03-23 Thread Magnus Ihse Bursie
On Wed, 23 Mar 2022 21:13:30 GMT, Magnus Ihse Bursie wrote: >> [JDK-8247872](https://bugs.openjdk.java.net/browse/JDK-8247872) (upgrade >> HarfBuzz to 2.7.2) caused build time to go up with 24 seconds on my >> reference linux machine. This was one of the four culprits that caused a >> 25-30% b

Re: RFR: 8283323: libharfbuzz optimization level results in extreme build times [v2]

2022-03-23 Thread Phil Race
On Wed, 23 Mar 2022 21:13:30 GMT, Magnus Ihse Bursie wrote: >> [JDK-8247872](https://bugs.openjdk.java.net/browse/JDK-8247872) (upgrade >> HarfBuzz to 2.7.2) caused build time to go up with 24 seconds on my >> reference linux machine. This was one of the four culprits that caused a >> 25-30% b

Re: RFR: 8270331 : [TESTBUG] Error: Not a test or directory containing tests: java/awt/print/PrinterJob/InitToBlack.java

2022-03-23 Thread Phil Race
On Thu, 17 Mar 2022 21:47:12 GMT, lawrence.andrews wrote: > 1) Removed =yesno since it was throwing Error: Not a test or directory > containing tests > 2) Test was just printing the "Test passes" on the printer and user did not > had any option to mark the test as pass or fail and test was alw

Re: RFR: 8270331 : [TESTBUG] Error: Not a test or directory containing tests: java/awt/print/PrinterJob/InitToBlack.java [v2]

2022-03-23 Thread lawrence . andrews
> 1) Removed =yesno since it was throwing Error: Not a test or directory > containing tests > 2) Test was just printing the "Test passes" on the printer and user did not > had any option to mark the test as pass or fail and test was always passing. > so created test UI with test instruction and

Re: RFR: 8270331 : [TESTBUG] Error: Not a test or directory containing tests: java/awt/print/PrinterJob/InitToBlack.java [v2]

2022-03-23 Thread lawrence . andrews
On Wed, 23 Mar 2022 22:51:17 GMT, Phil Race wrote: >> lawrence.andrews has updated the pull request incrementally with one >> additional commit since the last revision: >> >> Fixed minor spelling > > test/jdk/java/awt/print/PrinterJob/InitToBlack.java line 172: > >> 170: if (!CountDo

Re: RFR: 8279614: The left line of the TitledBorder is not painted on 150 scale factor [v6]

2022-03-23 Thread lukeu
On Wed, 23 Mar 2022 18:25:31 GMT, Alisen Chung wrote: > Can you try the recent change to see if it works at 200+%? Sure, if it still helps here's a screenshot at 250% scaling. It shows `TitledBorder` constructed with: * top: The JDK's `EtchedBorder` with the latest commit * bottom: a custom

Re: RFR: 8283323: libharfbuzz optimization level results in extreme build times [v2]

2022-03-23 Thread Magnus Ihse Bursie
On Wed, 23 Mar 2022 22:37:55 GMT, Phil Race wrote: >> Magnus Ihse Bursie has updated the pull request incrementally with one >> additional commit since the last revision: >> >> Restore HIGHEST for entire lib, just set SIZE to two files > > make/modules/java.desktop/lib/Awt2dLibraries.gmk line

Re: RFR: 8283323: libharfbuzz optimization level results in extreme build times [v3]

2022-03-23 Thread Magnus Ihse Bursie
> [JDK-8247872](https://bugs.openjdk.java.net/browse/JDK-8247872) (upgrade > HarfBuzz to 2.7.2) caused build time to go up with 24 seconds on my reference > linux machine. This was one of the four culprits that caused a 25-30% build > time regression over the last two years. > > The problem her

Re: RFR: 8283323: libharfbuzz optimization level results in extreme build times [v3]

2022-03-23 Thread Phil Race
On Wed, 23 Mar 2022 23:17:38 GMT, Magnus Ihse Bursie wrote: >> [JDK-8247872](https://bugs.openjdk.java.net/browse/JDK-8247872) (upgrade >> HarfBuzz to 2.7.2) caused build time to go up with 24 seconds on my >> reference linux machine. This was one of the four culprits that caused a >> 25-30% b

Re: RFR: 8283323: libharfbuzz optimization level results in extreme build times [v2]

2022-03-23 Thread Phil Race
On Wed, 23 Mar 2022 23:13:04 GMT, Magnus Ihse Bursie wrote: >> make/modules/java.desktop/lib/Awt2dLibraries.gmk line 487: >> >>> 485: LIBFONTMANAGER_OPTIMIZATION := HIGHEST >>> 486: >>> 487: ifneq ($(filter $(TOOLCHAIN_TYPE), gcc clang), ) >> >> Can we have a note here that the de-opt is possi

Re: RFR: 8273355: Flickering on tooltip appearance IntelliJ IDEA 2021.2.1

2022-03-23 Thread Sergey Bylokhov
On Wed, 23 Mar 2022 20:01:54 GMT, Alexey Ushakov wrote: > Used setOpaque() method to set correct background of platform window > Extracted setting background to the particular method in PlatformWindow. > Provided appropriate implementation in CPlatformWindow. src/java.desktop/macosx/classes/sun

Re: RFR: 8283323: libharfbuzz optimization level results in extreme build times [v3]

2022-03-23 Thread Sergey Bylokhov
On Wed, 23 Mar 2022 23:17:38 GMT, Magnus Ihse Bursie wrote: >> [JDK-8247872](https://bugs.openjdk.java.net/browse/JDK-8247872) (upgrade >> HarfBuzz to 2.7.2) caused build time to go up with 24 seconds on my >> reference linux machine. This was one of the four culprits that caused a >> 25-30% b

Integrated: 8283493: Create an automated regression test for RFE 4231298

2022-03-23 Thread Manukumar V S
On Tue, 22 Mar 2022 15:30:09 GMT, Manukumar V S wrote: > Create a regression test for > [JDK-4231298](https://bugs.openjdk.java.net/browse/JDK-4231298) > Issue: > When a JComboBox first drops down, the renderer is called for > every element in the list, even those elements that will not > actual

Integrated: 8282860: Write a regression test for JDK-4164779

2022-03-23 Thread Manukumar V S
On Sun, 13 Mar 2022 13:42:15 GMT, Manukumar V S wrote: > Write a regression test for > [JDK-4164779](https://bugs.openjdk.java.net/browse/JDK-4164779) : JSplitPane > keyboard navigation does not support F6 and Ctrl+Tab > > > Issue: > The specified keyboard support for JSplitPane also requires

Re: RFR: 8283323: libharfbuzz optimization level results in extreme build times [v3]

2022-03-23 Thread Magnus Ihse Bursie
On Wed, 23 Mar 2022 23:38:52 GMT, Sergey Bylokhov wrote: >> Magnus Ihse Bursie has updated the pull request incrementally with one >> additional commit since the last revision: >> >> Update comment to show we don't care about these files. > > Do we really want to change the product performanc

Re: RFR: 8241192: [macosx] Wrong letter typed after ´ symbol when using Finnish layout

2022-03-23 Thread Phil Race
On Fri, 28 Jan 2022 09:38:35 GMT, Prasanta Sadhukhan wrote: > If we press " ´ " (on U.S. keyboard layout it is the [+] button just before > the [delete] button), and then press "s" , 2 "´" characters are entered in a > text field instead of "´s" with "Finish" keyboard layout. > This is because

Re: RFR: 8280468: Crashes in getConfigColormap, getConfigVisualId, XVisualIDFromVisual on Linux

2022-03-23 Thread Phil Race
On Fri, 21 Jan 2022 17:02:38 GMT, Maxim Kartashev wrote: > These crashes were not reproducible, so the fix is based on a hypothesis that > there are two possible reasons for them: > 1. `makeDefaultConfig()` returning `NULL`. > 2. A race condition when the number of screens changes. > The race sc

Re: RFR: 8269806: Emoji rendering on Linux

2022-03-23 Thread Phil Race
On Thu, 15 Jul 2021 17:29:01 GMT, Nikita Gubarkov wrote: > It was implemented in JetBrains Runtime a year ago and was ported & > refactored for this PR > It includes: > - Bitmap glyph loading via Freetype > - Manual scaling & transformation of bitmap glyphs with nearest-neighbor or > bilinear-m

Re: RFR: 8276849: Refresh the window icon on graphics configuration changes [v4]

2022-03-23 Thread Phil Race
On Wed, 23 Feb 2022 21:34:29 GMT, Emmanuel Bourg wrote: >> Emmanuel Bourg has updated the pull request incrementally with two >> additional commits since the last revision: >> >> - Test case for the window icon update on DPI change (Java generic font >> names are usually capitalized) >> - Te

Re: RFR: 8176501: Method Shape.getBounds2D() incorrectly includes Bezier control points in bounding box [v12]

2022-03-23 Thread Phil Race
On Fri, 4 Mar 2022 22:38:33 GMT, Jeremy wrote: >> Jeremy has updated the pull request incrementally with one additional commit >> since the last revision: >> >> 8176501: Method Shape.getBounds2D() incorrectly includes Bezier control >> points in bounding box >> >> This is a second follo

Re: RFR: 8278828 : java/awt/a11y/AccessibleTextTest.java's createTextArea() just shows instructions on the UI [v3]

2022-03-23 Thread Phil Race
On Thu, 27 Jan 2022 21:47:04 GMT, lawrence.andrews wrote: >> 1) Test scenario was just showing test instruction with JTextArea. So added >> the JTextArea as the instruction and method says. >> 2) JTextArea is filled with contents so that and an Emoji character making >> sure that screen reade

Re: RFR: 8280193: summary javadoc for java.awt.GraphicsEnvironment#preferPropotionalFonts broken

2022-03-23 Thread Phil Race
On Tue, 18 Jan 2022 14:05:55 GMT, James Nord wrote: > https://bugs.openjdk.java.net/browse/JDK-8280193 > > > Javadoc broke the first sentence at the full stop space (newline) so it > rendered with `Indicates a preference for proportional over non-proportional > (e.g.` > > This is an incorrec

Re: RFR: 8185261: Font fallback sometimes doesn't work in Swing text components [v2]

2022-03-23 Thread Phil Race
On Tue, 1 Feb 2022 21:40:53 GMT, Dmitry Batrak wrote: >> The proposed fix makes fonts with and without fallback components >> distinguishable (in terms of `equals` method), so that >> font metrics cache (and other similar code) can handle them separately. This >> is achieved by adding a new boo

Re: RFR: 8281284 : Write JSlider accessibility test [v7]

2022-03-23 Thread Phil Race
On Mon, 21 Mar 2022 21:41:27 GMT, lawrence.andrews wrote: >> This testcase covers both old and new api ( api added in jdk17 >> getAccessibleAction(), getAccessibleActionCount() , >> getAccessibleActionDescription ) >> >> @shurymury >> @savoptik > > lawrence.andrews has updated the pull requ

Re: RFR: 8274735: javax.imageio.IIOException: Unsupported Image Type while processing a valid JPEG image [v2]

2022-03-23 Thread Phil Race
On Tue, 22 Mar 2022 01:45:30 GMT, Sergey Bylokhov wrote: >> Phil Race has updated the pull request incrementally with one additional >> commit since the last revision: >> >> 8274735: javax.imageio.IIOException: Unsupported Image Type while >> processing a valid JPEG image > > src/java.deskto

Integrated: 8270331 : [TESTBUG] Error: Not a test or directory containing tests: java/awt/print/PrinterJob/InitToBlack.java

2022-03-23 Thread lawrence . andrews
On Thu, 17 Mar 2022 21:47:12 GMT, lawrence.andrews wrote: > 1) Removed =yesno since it was throwing Error: Not a test or directory > containing tests > 2) Test was just printing the "Test passes" on the printer and user did not > had any option to mark the test as pass or fail and test was alw

Re: RFR: 8279614: The left line of the TitledBorder is not painted on 150 scale factor [v7]

2022-03-23 Thread Phil Race
On Wed, 23 Mar 2022 18:24:05 GMT, Alisen Chung wrote: >> Changed the drawing area to be increased by 0.5 on the left side to prevent >> clipping > > Alisen Chung has updated the pull request incrementally with one additional > commit since the last revision: > > scale stroke width at higher

Re: RFR: 8282640: Create a test for JDK-4740761 [v2]

2022-03-23 Thread Phil Race
On Fri, 11 Mar 2022 14:44:25 GMT, Srinivas Mandalika wrote: >> Create a test for >> [JDK-4740761](https://bugs.openjdk.java.net/browse/JDK-4740761) >> >> The issue observed is in a JFrame with a JTextField and a JScrollPane which >> contains focused component. When the JScrollPane was >> is r

Re: RFR: 8283323: libharfbuzz optimization level results in extreme build times [v3]

2022-03-23 Thread Phil Race
On Wed, 23 Mar 2022 23:38:52 GMT, Sergey Bylokhov wrote: >> Magnus Ihse Bursie has updated the pull request incrementally with one >> additional commit since the last revision: >> >> Update comment to show we don't care about these files. > > Do we really want to change the product performanc

Withdrawn: 8278828 : java/awt/a11y/AccessibleTextTest.java's createTextArea() just shows instructions on the UI

2022-03-23 Thread lawrence . andrews
On Wed, 15 Dec 2021 02:21:35 GMT, lawrence.andrews wrote: > 1) Test scenario was just showing test instruction with JTextArea. So added > the JTextArea as the instruction and method says. > 2) JTextArea is filled with contents so that and an Emoji character making > sure that screen reader ca

Withdrawn: 8241192: [macosx] Wrong letter typed after ´ symbol when using Finnish layout

2022-03-23 Thread Prasanta Sadhukhan
On Fri, 28 Jan 2022 09:38:35 GMT, Prasanta Sadhukhan wrote: > If we press " ´ " (on U.S. keyboard layout it is the [+] button just before > the [delete] button), and then press "s" , 2 "´" characters are entered in a > text field instead of "´s" with "Finish" keyboard layout. > This is because

Re: RFR: 8282640: Create a test for JDK-4740761 [v2]

2022-03-23 Thread Srinivas Mandalika
On Thu, 24 Mar 2022 03:29:45 GMT, Phil Race wrote: >> Srinivas Mandalika has updated the pull request incrementally with one >> additional commit since the last revision: >> >> Added @bug keyword > > test/jdk/java/awt/Focus/8282640/ScrollPaneFocusBugTest.java line 102: > >> 100: robo

Re: RFR: 8282640: Create a test for JDK-4740761 [v2]

2022-03-23 Thread Srinivas Mandalika
On Fri, 11 Mar 2022 14:44:25 GMT, Srinivas Mandalika wrote: >> Create a test for >> [JDK-4740761](https://bugs.openjdk.java.net/browse/JDK-4740761) >> >> The issue observed is in a JFrame with a JTextField and a JScrollPane which >> contains focused component. When the JScrollPane was >> is r

Re: RFR: JDK-8283015: Create a test for JDK-4715496 [v5]

2022-03-23 Thread Srinivas Mandalika
On Wed, 23 Mar 2022 10:12:26 GMT, Srinivas Mandalika wrote: >> Create a test for JDK-4715496 >> >> Issue observed in the above bug is that the >> JTable.AccessibleJTableCell.getAccessible name returns the cell instance >> string instead of the cell text. >> The test is being added to verify t

Re: RFR: JDK-8282046: Create a regression test for JDK-8000326 [v7]

2022-03-23 Thread Srinivas Mandalika
> Create a regression test for CCC8000326 > > Issue is identified by > [JDK-8000326](https://bugs.openjdk.java.net/browse/JDK-8000326), which > identifies that after focus moves into JMenuBar, whose focus traversal key is > disabled by default, it never moves to other focusable component. > >

Re: RFR: 8282936: Write a regression test for JDK-4615365 [v2]

2022-03-23 Thread Manukumar V S
On Tue, 22 Mar 2022 06:29:16 GMT, Manukumar V S wrote: >> Write a regression test for >> [JDK-4615365](https://bugs.openjdk.java.net/browse/JDK-4615365) : JSplitPane >> current and last divider positions incorrect when realized >> >> Issue: >> JSplitPane component with a left and right compone

Integrated: 8283437: Refactor imageio classes javadoc to use @throws instead of @exception

2022-03-23 Thread Prasanta Sadhukhan
On Mon, 21 Mar 2022 06:13:24 GMT, Prasanta Sadhukhan wrote: > Prevailing JDK coding practices use "@throws" rather than "@exception". > Refactor existing imageio classes javadoc to use @throws This pull request has now been integrated. Changeset: e6f707aa Author:Prasanta Sadhukhan URL:

RFR: 8283608: Refactor 2d, beans classes javadoc to use @throws instead of @exception

2022-03-23 Thread Prasanta Sadhukhan
Prevailing JDK coding practices use "@throws" rather than "@exception". Need to refactor remaining 2d, beans,sound classes to use @throws - Commit messages: - Fix - Fix Changes: https://git.openjdk.java.net/jdk/pull/7937/files Webrev: https://webrevs.openjdk.java.net/?repo=jdk&pr=