On Tue, 29 Jul 2025 05:38:00 GMT, Jayathirth D V wrote:
> We need to throw appropriate exception when we pass invalid image index
> values to ImageReader.getImageTypes().
> We throw IOOBE in all other plugins except JPEG, Added appropriate check to
> throw IOOBE when JpegIma
On Thu, 28 Aug 2025 12:49:37 GMT, Alexey Ivanov wrote:
>> Jayathirth D V has updated the pull request incrementally with one
>> additional commit since the last revision:
>>
>> Keep verifyImageIndex in try block
>
> test/jdk/javax/imageio/plugins/jpeg/JpegNegat
On Thu, 28 Aug 2025 13:04:53 GMT, Alexey Ivanov wrote:
>> test/jdk/javax/imageio/plugins/jpeg/JpegNegativeImageIndexTest.java line 52:
>>
>>> 50:
>>> 51: private static void testMethod(String methodName,
>>> 52:RunnableWithException method) {
>>
>> The `
> We need to throw appropriate exception when we pass invalid image index
> values to ImageReader.getImageTypes().
> We throw IOOBE in all other plugins except JPEG, Added appropriate check to
> throw IOOBE when JpegImageReader has invalid image index.
Jayathirth D V has upda
On Mon, 25 Aug 2025 20:18:49 GMT, Phil Race wrote:
> The disposer for MemoryCacheImageInputStream isn't needed. All it does is
> clear an ArrayList of byte[]
> GC can take care of that.
Marked as reviewed by jdv (Reviewer).
-
PR Review: https://git.openjdk.org/jdk/pull/26933#pullr
> We need to throw appropriate exception when we pass invalid image index
> values to ImageReader.getImageTypes().
> We throw IOOBE in all other plugins except JPEG, Added appropriate check to
> throw IOOBE when JpegImageReader has invalid image index.
Jayathirth D V has upda
On Tue, 26 Aug 2025 05:49:59 GMT, Prasanta Sadhukhan
wrote:
>> Jayathirth D V has updated the pull request incrementally with two
>> additional commits since the last revision:
>>
>> - Make test more robust
>> - Use thread lock
>
> src/java.desktop/share/
On Mon, 25 Aug 2025 14:12:05 GMT, Alexey Ivanov wrote:
>> Jayathirth D V has updated the pull request incrementally with one
>> additional commit since the last revision:
>>
>> Update image index verification function name
>
> src/java.desktop/share/classe
> We need to throw appropriate exception when we pass invalid image index
> values to ImageReader.getImageTypes().
> We throw IOOBE in all other plugins except JPEG, Added appropriate check to
> throw IOOBE when JpegImageReader has invalid image index.
Jayathirth D V has upda
On Wed, 20 Aug 2025 20:36:21 GMT, Phil Race wrote:
>> Remove javax.imageio.spi.ServiceRegistry.finalize() since it is deprecated
>> for removal.
>> The impact of this should be minimal. The IIORegistry will now be a
>> singleton - which it already was
>> in practice since Applets are no longer
On Fri, 22 Aug 2025 07:13:45 GMT, Prasanta Sadhukhan
wrote:
>> Jayathirth D V has updated the pull request with a new target base due to a
>> merge or a rebase. The incremental webrev excludes the unrelated changes
>> brought in by the merge/rebase. The pull request contai
> We need to throw appropriate exception when we pass invalid image index
> values to ImageReader.getImageTypes().
> We throw IOOBE in all other plugins except JPEG, Added appropriate check to
> throw IOOBE when JpegImageReader has invalid image index.
Jayathirth D V has upda
On Thu, 21 Aug 2025 20:46:41 GMT, Phil Race wrote:
>> Jayathirth D V has updated the pull request with a new target base due to a
>> merge or a rebase. The incremental webrev excludes the unrelated changes
>> brought in by the merge/rebase. The pull request contains five addi
On Tue, 29 Jul 2025 17:22:09 GMT, Phil Race wrote:
>> Jayathirth D V has updated the pull request with a new target base due to a
>> merge or a rebase. The incremental webrev excludes the unrelated changes
>> brought in by the merge/rebase. The pull request contains five addi
On Fri, 1 Aug 2025 00:31:45 GMT, Alisen Chung wrote:
>> Jayathirth D V has updated the pull request with a new target base due to a
>> merge or a rebase. The incremental webrev excludes the unrelated changes
>> brought in by the merge/rebase. The pull request contai
> We need to throw appropriate exception when we pass invalid image index
> values to ImageReader.getImageTypes().
> We throw IOOBE in all other plugins except JPEG, Added appropriate check to
> throw IOOBE when JpegImageReader has invalid image index.
Jayathirth D V has upda
On Wed, 13 Aug 2025 16:07:45 GMT, Nikita Gubarkov wrote:
>> For "true" null objects, reset the ref itself to null. Non-null ref with
>> null content means that the object was GC'ed. GC'ed state always behaves as
>> not-equal to the new one, causing corresponding ops to be written into RQ.
>>
>
On Tue, 12 Aug 2025 12:00:37 GMT, Nikita Gubarkov wrote:
>> For "true" null objects, reset the ref itself to null. Non-null ref with
>> null content means that the object was GC'ed. GC'ed state always behaves as
>> not-equal to the new one, causing corresponding ops to be written into RQ.
>>
>
On Tue, 12 Aug 2025 12:00:37 GMT, Nikita Gubarkov wrote:
>> For "true" null objects, reset the ref itself to null. Non-null ref with
>> null content means that the object was GC'ed. GC'ed state always behaves as
>> not-equal to the new one, causing corresponding ops to be written into RQ.
>>
>
On Tue, 12 Aug 2025 11:48:56 GMT, Laurent Bourgès wrote:
>> test/jdk/java/awt/ColorClass/WeakColorTest.java line 24:
>>
>>> 22: */
>>> 23:
>>> 24: import java.awt.*;
>>
>> Better to expand wild card imports.
>
> If test depends on gc collector + special mode, then set explicitely the
> neede
On Mon, 11 Aug 2025 15:48:58 GMT, Nikita Gubarkov wrote:
>> For "true" null objects, reset the ref itself to null. Non-null ref with
>> null content means that the object was GC'ed. GC'ed state always behaves as
>> not-equal to the new one, causing corresponding ops to be written into RQ.
>>
>
On Fri, 8 Aug 2025 10:03:18 GMT, Jayathirth D V wrote:
>> ByteArrayOutputStream stmt part of try-with-resources block.
>
> I understand, but line no 55 has extra space instead of usual 4 space
> indentation which is followed in next line.
> Also its a good practice to wa
On Thu, 7 Aug 2025 21:27:53 GMT, Harshitha Onkar wrote:
>> test/jdk/java/awt/image/XBMDecoder/XBMDecoderTest.java line 55:
>>
>>> 53: System.out.println("--- Testing " + fileName + " ---");
>>> 54: try (FileInputStream fis = new FileInputStream(file);
>>> 55:
On Fri, 11 Jul 2025 23:01:24 GMT, Jeremy Wood wrote:
>> When a gif frame's disposal code is DISPOSAL_BGCOLOR and we're maintaining a
>> saved image: we have to flood-fill the frame's bounds. And if the
>> transparent pixel index changes across frames: we could sometimes flood fill
>> the wrong
On Mon, 21 Jul 2025 16:51:51 GMT, Phil Race wrote:
> Remove finalize from TIFF ImageIO implementatation classes.
>
> Following copied from the bug description
> The TIFF image (de)compressors can make use of a JPEG reader / writer
> Like all ImageIO readers and writers, once done, the owner sho
On Fri, 25 Jul 2025 23:58:39 GMT, Harshitha Onkar wrote:
>> XBM (X Bitmap) images are small one-bit deep images that were developed as a
>> simple uncompressed format that could easily be included in C source files
>> as include files.
>>
>> Format of XBM image:
>>
>>
>> #define width w
>>
We need to throw appropriate exception when we pass invalid image index values
to ImageReader.getImageTypes().
We throw IOOBE in all other plugins except JPEG, Added appropriate check to
throw IOOBE when JpegImageReader has invalid image index.
-
Commit messages:
- 8364135: JPEGIma
On Fri, 11 Jul 2025 18:59:18 GMT, Jeremy Wood wrote:
>> If there are two consecutive frames that use DISPOSAL_SAVE, but the
>> transparent pixel index changed: we might accidentally send the wrong data
>> to the ImageConsumer.
>>
>> We already had logic that submits info "the hard way" (see co
On Fri, 11 Jul 2025 06:36:31 GMT, Jayathirth D V wrote:
> Latest change looks good to me. I will run CI tests and update.
Applied this PR over latest mainline code and CI test is all green.
-
PR Comment: https://git.openjdk.org/jdk/pull/25264#issuecomment-3061621716
On Thu, 10 Jul 2025 21:20:07 GMT, Jeremy Wood wrote:
>> If there are two consecutive frames that use DISPOSAL_SAVE, but the
>> transparent pixel index changed: we might accidentally send the wrong data
>> to the ImageConsumer.
>>
>> We already had logic that submits info "the hard way" (see co
On Thu, 10 Jul 2025 21:41:51 GMT, Phil Race wrote:
> I did modify the test to dump the frames being generated and the accumulated
> image, just so I could see. Maybe you could add such a feature as a debugging
> option in one of the follow-on PRs.
Followed same approach. As mentioned its bette
On Sun, 1 Jun 2025 07:13:14 GMT, Jeremy Wood wrote:
>> If there are two consecutive frames that use DISPOSAL_SAVE, but the
>> transparent pixel index changed: we might accidentally send the wrong data
>> to the ImageConsumer.
>>
>> We already had logic that submits info "the hard way" (see com
On Wed, 9 Jul 2025 20:54:23 GMT, Jeremy Wood wrote:
>> This resolves a gif parsing bug where an unwanted opaque rectangle could
>> appear under these conditions:
>>
>> 1. The disposal method for frames is 1 (meaning "do not dispose", aka
>> "DISPOSAL_SAVE")
>> 2. The transparent pixel is non-z
On Sun, 1 Jun 2025 06:43:44 GMT, Jeremy Wood wrote:
>> This resolves a gif parsing bug where an unwanted opaque rectangle could
>> appear under these conditions:
>>
>> 1. The disposal method for frames is 1 (meaning "do not dispose", aka
>> "DISPOSAL_SAVE")
>> 2. The transparent pixel is non-z
On Mon, 2 Jun 2025 15:05:44 GMT, Phil Race wrote:
>> test/jdk/java/awt/FontMetrics/ExtremeFontSizeTest.java line 39:
>>
>>> 37: /*
>>> 38: * @test
>>> 39: * @bug 8328896 8357672
>>
>> I thought Freetype rasterizer is used on all platforms(Linux, Windows &
>> macOS) by default. Since Freetype
On Wed, 28 May 2025 20:19:42 GMT, Phil Race wrote:
> Instrumenting the existing ExtremeFontSizeTest.java, it was noted that the
> freetype code, when encountering a null return from getting a scaler context,
> would over-interpret this as a bad font.
> The test was intended to check this wasn't
On Mon, 26 May 2025 05:29:45 GMT, Jeremy Wood wrote:
>> This resolves a gif parsing bug where an unwanted opaque rectangle could
>> appear under these conditions:
>>
>> 1. The disposal method for frames is 1 (meaning "do not dispose", aka
>> "DISPOSAL_SAVE")
>> 2. The transparent pixel is non-
On Fri, 23 May 2025 11:38:32 GMT, Jayathirth D V wrote:
>> In stress based scenarios it is observed that nothing is drawn in UI content
>> when display wakes up from sleep in Metal pipeline of macOS. Unfortunately i
>> am not able to reproduce it, but based on details in the
On Wed, 21 May 2025 06:11:18 GMT, Jayathirth D V wrote:
> In stress based scenarios it is observed that nothing is drawn in UI content
> when display wakes up from sleep in Metal pipeline of macOS. Unfortunately i
> am not able to reproduce it, but based on details in the bug it look
On Thu, 22 May 2025 08:07:34 GMT, Alexey Ushakov wrote:
>> Jayathirth D V has updated the pull request incrementally with one
>> additional commit since the last revision:
>>
>> Add macro
>
> src/java.desktop/macosx/native/libawt_lwawt/java2d/metal/MTLLaye
deprecated from macOS 14 and going forward we need to use
> CADisplayLink/CAMetalDisplayLink. I have raised
> [JDK-8357418](https://bugs.openjdk.org/browse/JDK-8357418) for the same.
Jayathirth D V has updated the pull request incrementally with one additional
commit since the last revision:
Ad
On Mon, 19 May 2025 05:37:05 GMT, Jeremy Wood wrote:
>> Unfortunately I'm unable to reproduce this failure. I slowed the test down;
>> it passes on my machine before and after this PR.
>>
>> I added a little bit of logging that might also help identify what's
>> happening.
>>
>> Does anyone h
On Thu, 22 May 2025 03:04:44 GMT, Sergey Bylokhov wrote:
> Is it possible that it is somehow related to 8304825? could be the layer
> created "during display sleep" and that might be in "sleep" more that 5
> seconds?
This is follow-up fix for JDK-8304825. Basically we have a race condition
be
In stress based scenarios it is observed that nothing is drawn in UI content
when display wakes up from sleep in Metal pipeline of macOS. Unfortunately i am
not able to reproduce it, but based on details in the bug it looks like we are
hitting a race condition. Call to CVDisplayLinkCreateWithAct
On Mon, 19 May 2025 05:37:05 GMT, Jeremy Wood wrote:
>> Unfortunately I'm unable to reproduce this failure. I slowed the test down;
>> it passes on my machine before and after this PR.
>>
>> I added a little bit of logging that might also help identify what's
>> happening.
>>
>> Does anyone h
On Mon, 19 May 2025 05:37:05 GMT, Jeremy Wood wrote:
>> Unfortunately I'm unable to reproduce this failure. I slowed the test down;
>> it passes on my machine before and after this PR.
>>
>> I added a little bit of logging that might also help identify what's
>> happening.
>>
>> Does anyone h
On Mon, 19 May 2025 08:15:29 GMT, Abhishek Kumar wrote:
> Test was failing on macOS 14.7.1 system in CI pipeline. Logged output
> suggests that the mouse exit event is not triggered and thus the test fails.
> Increased number of list items and adjusted the X and Y offset for mouse move
> events
On Mon, 12 May 2025 07:20:43 GMT, Abhishek Kumar wrote:
> Log mouse events output to analyze the failure in CI pipeline.
Marked as reviewed by jdv (Reviewer).
-
PR Review: https://git.openjdk.org/jdk/pull/25176#pullrequestreview-2832349394
On Thu, 17 Apr 2025 04:25:59 GMT, Abhishek Kumar wrote:
>> Jayathirth D V has updated the pull request incrementally with one
>> additional commit since the last revision:
>>
>> Update after review comment
>
> test/jdk/com/sun/java/swing/plaf/windows
On Thu, 17 Apr 2025 05:59:03 GMT, Jayathirth D V wrote:
> javax/swing/AbstractButton/bug4133768.java was automated and open-sourced
> recently and it fails intermittently in CI.
>
> Looks like when we have mouse pointer at same location from where we are
> picking UI color
On Fri, 18 Apr 2025 08:17:10 GMT, Abhishek Kumar wrote:
>> Jayathirth D V has updated the pull request incrementally with one
>> additional commit since the last revision:
>>
>> Update
>
> test/jdk/javax/swing/AbstractButton/bug4133768.java line 1:
>
>
have mouse pointer to be at different place from where we are
> picking roll-over color of buttons.
> Also made other changes to make test more robust. With all these changes test
> now passes all the time in CI.
Jayathirth D V has updated the pull request incrementally with one add
On Thu, 17 Apr 2025 12:31:48 GMT, Abhishek Kumar wrote:
>> Few Modal and Full Screen related tests are converted from applet and open
>> sourced.
>
> Abhishek Kumar has updated the pull request incrementally with one additional
> commit since the last revision:
>
> Typo fix
Marked as review
On Fri, 18 Apr 2025 07:27:57 GMT, Abhishek Kumar wrote:
>> You had any background window while running this test?
>> Also make sure if you have proper color profile for display.
>> It runs properly in my machine and our CI with 50 test runs on all platforms.
>>
>> I can add tolerance, but since
On Tue, 15 Apr 2025 22:37:05 GMT, Damon Nguyen wrote:
> Open-sourcing and standardizing several tests (windows & JSlider)
Marked as reviewed by jdv (Reviewer).
-
PR Review: https://git.openjdk.org/jdk/pull/24671#pullrequestreview-2777857023
On Fri, 18 Apr 2025 05:53:01 GMT, Abhishek Kumar wrote:
>> javax/swing/AbstractButton/bug4133768.java was automated and open-sourced
>> recently and it fails intermittently in CI.
>>
>> Looks like when we have mouse pointer at same location from where we are
>> picking UI color from robot, it
On Wed, 16 Apr 2025 08:42:20 GMT, Jayathirth D V wrote:
> Open source some of the swing LAF tests
This pull request has now been integrated.
Changeset: 4c994894
Author: Jayathirth D V
URL:
https://git.openjdk.org/jdk/commit/4c99489420bd73159eca6bae22442f7b29156c1d
Stats:
On Wed, 16 Apr 2025 08:54:29 GMT, Jayathirth D V wrote:
> Open source some of the swing windows LAF tests
This pull request has now been integrated.
Changeset: dda4b5a4
Author: Jayathirth D V
URL:
https://git.openjdk.org/jdk/commit/dda4b5a4ade2e5d7225117e58fce4038bb0e0f1b
St
On Wed, 16 Apr 2025 20:33:06 GMT, Phil Race wrote:
>> Open source some more Swing menu tests
>
> Phil Race has updated the pull request incrementally with one additional
> commit since the last revision:
>
> 8354451
Can we automate test/jdk/javax/swing/JPopupMenu/bug4188832.java ? Or do we w
On Thu, 17 Apr 2025 06:31:03 GMT, Abhishek Kumar wrote:
>> Few Modal and Full Screen related tests are converted from applet and open
>> sourced.
>
> Abhishek Kumar has updated the pull request incrementally with one additional
> commit since the last revision:
>
> Review fix
Marked as revi
javax/swing/AbstractButton/bug4133768.java was automated and open-sourced
recently and it fails intermittently in CI.
Looks like when we have mouse pointer at same location from where we are
picking UI color from robot, it picks wrong color.
Updated test to have mouse pointer to be at different
> Open source some of the swing windows LAF tests
Jayathirth D V has updated the pull request incrementally with one additional
commit since the last revision:
Update after review comment
-
Changes:
- all: https://git.openjdk.org/jdk/pull/24678/files
- new: ht
> Open source some of the swing LAF tests
Jayathirth D V has updated the pull request incrementally with one additional
commit since the last revision:
Update after review
-
Changes:
- all: https://git.openjdk.org/jdk/pull/24677/files
- new: https://git.openjdk.org/jdk/p
On Thu, 17 Apr 2025 04:24:57 GMT, Abhishek Kumar wrote:
>> Jayathirth D V has updated the pull request incrementally with one
>> additional commit since the last revision:
>>
>> Update after review
>
> test/jdk/com/sun/java/swing/plaf/windows/MenuItem/4685
Open source some of the swing LAF tests
-
Commit messages:
- 8354285: Open source Swing tests Batch 3
Changes: https://git.openjdk.org/jdk/pull/24677/files
Webrev: https://webrevs.openjdk.org/?repo=jdk&pr=24677&range=00
Issue: https://bugs.openjdk.org/browse/JDK-8354285
Stats:
On Fri, 11 Apr 2025 05:52:47 GMT, Jayathirth D V wrote:
> Few swing JList tests are open sourced.
This pull request has now been integrated.
Changeset: 2be5bc84
Author: Jayathirth D V
URL:
https://git.openjdk.org/jdk/commit/2be5bc847a444f08a4ebb41b58e8a2bf4553d621
Stats:
On Mon, 14 Apr 2025 12:53:59 GMT, Prasanta Sadhukhan
wrote:
>> Few JComponent tests are opensourced
>
> Prasanta Sadhukhan has updated the pull request incrementally with one
> additional commit since the last revision:
>
> Redundant method removed
Marked as reviewed by jdv (Reviewer).
---
Few Swing AbstractButton, JList and JMenu tests are open sourced.
-
Commit messages:
- Update
- 8354163: Open source Swing tests Batch 1
Changes: https://git.openjdk.org/jdk/pull/24650/files
Webrev: https://webrevs.openjdk.org/?repo=jdk&pr=24650&range=00
Issue: https://bugs.ope
On Tue, 15 Apr 2025 11:02:39 GMT, Tejesh R wrote:
>> Open source these AWT TextComponent tests:
>>
>> java/awt/TextComponent/SelectionAndCaretColor/SelectionAndCaretColor.java
>> java/awt/TextComponent/SelectionTest/SelectionTest.java
>> java/awt/TextComponent/CorrectTextComponentSelectionTest/C
On Mon, 7 Apr 2025 16:41:37 GMT, Phil Race wrote:
>> Open source a few tests related to Swing menus.
>
> Phil Race has updated the pull request incrementally with one additional
> commit since the last revision:
>
> 8353589
LGTM.
Also tested them on my macOS machine.
-
Marked a
On Tue, 15 Apr 2025 07:09:51 GMT, Jayathirth D V wrote:
> Few Swing AbstractButton, JList and JMenu tests are open sourced.
This pull request has now been integrated.
Changeset: b7837843
Author: Jayathirth D V
URL:
https://git.openjdk.org/jdk/com
On Tue, 15 Apr 2025 07:28:39 GMT, Prasanta Sadhukhan
wrote:
>> test/jdk/javax/swing/AbstractButton/bug4133768.java line 54:
>>
>>> 52: private static AbstractButton[] buttons;
>>> 53: private static Point buttonLocation;
>>> 54: private static int buttonWidth, buttonHeight;
>>
>> d
On Tue, 15 Apr 2025 07:26:44 GMT, Prasanta Sadhukhan
wrote:
>> Jayathirth D V has updated the pull request incrementally with one
>> additional commit since the last revision:
>>
>> Update after review comments
>
> test/jdk/javax/swing/AbstractButton/bug
> Few Swing AbstractButton, JList and JMenu tests are open sourced.
Jayathirth D V has updated the pull request incrementally with one additional
commit since the last revision:
Update after review comments
-
Changes:
- all: https://git.openjdk.org/jdk/pull/24650/files
-
> Few swing JList tests are open sourced.
Jayathirth D V has updated the pull request incrementally with one additional
commit since the last revision:
Update after review comments
-
Changes:
- all: https://git.openjdk.org/jdk/pull/24588/files
- new: https://git.openjdk.
On Mon, 14 Apr 2025 11:26:18 GMT, Abhishek Kumar wrote:
>> Jayathirth D V has updated the pull request incrementally with one
>> additional commit since the last revision:
>>
>> Update after review
>
> test/jdk/javax/swing/JList/bug4193267.java line 44:
>
&g
On Fri, 11 Apr 2025 20:19:30 GMT, Andrey Turbanov wrote:
>> Jayathirth D V has updated the pull request incrementally with one
>> additional commit since the last revision:
>>
>> Update after review
>
> test/jdk/javax/swing/JList/bug4193267.java line 8
On Fri, 11 Apr 2025 10:31:14 GMT, Mikhail Yankelevich
wrote:
>> Jayathirth D V has updated the pull request incrementally with one
>> additional commit since the last revision:
>>
>> Update after review
>
> test/jdk/javax/swing/JList/bug4618767.java line 2
> Few swing JList tests are open sourced.
Jayathirth D V has updated the pull request incrementally with one additional
commit since the last revision:
Update after review
-
Changes:
- all: https://git.openjdk.org/jdk/pull/24588/files
- new: https://git.openjdk.org/jdk/p
On Fri, 11 Apr 2025 17:14:06 GMT, Harshitha Onkar wrote:
>> Jayathirth D V has updated the pull request incrementally with one
>> additional commit since the last revision:
>>
>> Update after review
>
> test/jdk/javax/swing/JList/bug4193267.java line 63:
>
On Mon, 14 Apr 2025 05:57:16 GMT, Abhishek Kumar wrote:
> Font Class, Selection adn Icon related Applet tests are converted to manual
> and open sourced.
Marked as reviewed by jdv (Reviewer).
Marked as reviewed by jdv (Reviewer).
-
PR Review: https://git.openjdk.org/jdk/pull/2461
Few swing JList tests are open sourced.
-
Commit messages:
- 8354214: Open source Swing tests Batch 2
Changes: https://git.openjdk.org/jdk/pull/24588/files
Webrev: https://webrevs.openjdk.org/?repo=jdk&pr=24588&range=00
Issue: https://bugs.openjdk.org/browse/JDK-8354214
Stats:
On Thu, 10 Apr 2025 16:35:33 GMT, Alexander Zvegintsev
wrote:
> Problem list two tests crashing XWayland server
> Corresponding issue [JDK-8354097](https://bugs.openjdk.org/browse/JDK-8354097)
Suggestion:
Since we will be streamlining things to run on Ubuntu 24.04 & Wayland, we can
use/add uti
On Thu, 13 Feb 2025 22:00:26 GMT, Harshitha Onkar wrote:
>> Harshitha Onkar has updated the pull request incrementally with one
>> additional commit since the last revision:
>>
>> javadoc update
>
> src/java.desktop/share/classes/java/awt/color/ICC_Profile.java line 1154:
>
>> 1152: * T
On Thu, 13 Feb 2025 23:15:52 GMT, Harshitha Onkar wrote:
>> Built-in Profiles are singleton objects and if the user happens to modify
>> this shared profile object via setData() then the modified version of the
>> profile is returned each time the same built-in profile is requested via
>> getI
On Tue, 21 Jan 2025 20:06:55 GMT, Harshitha Onkar wrote:
>> ICC_Profile.setData(..) does validation of the specified tag contents and
>> throws an exception if it is not valid. But if the tag represents the
>> header, at least some of the validation is lazy, occurring only when the
>> data is
On Thu, 16 Jan 2025 11:52:14 GMT, Jayathirth D V wrote:
> In PNG we have non-essential compressed text chunks zTXt and iTXt, which can
> consume considerable amount of memory when they contain large amount of
> compressed data.
>
> We should restrict the amount of text
In PNG we have non-essential compressed text chunks zTXt and iTXt, which can
consume considerable amount of memory when they contain large amount of
compressed data.
We should restrict the amount of text data we want to decompress from these
text chunks. This PR adds limit of 256k bytes of infl
On Tue, 14 Jan 2025 20:45:14 GMT, Harshitha Onkar wrote:
>>> our own profiles do not strictly follow these specification, so if lcms
>>> ignores that we can do the same, especially since we fallback to the
>>> "Perceptual intent"
>>>
>>> https://github.com/openjdk/jdk/blob/13a1775718f329b02cde
On Thu, 19 Sep 2024 11:54:56 GMT, Nikita Gubarkov wrote:
>> I moved SurfaceDataProxy cache into a separate class.
>> Now caching level is determined by placement of the
>> SurfaceManager.ProxyCache object instead of a "proxy key". This cache "owns"
>> proxies and therefore strong refs via them
On Tue, 1 Oct 2024 10:34:53 GMT, Jayathirth D V wrote:
> Clean up and open source some of the AWT window tests.
> Automated test `OwnedWindowShowTest.java` is verified in our CI also and it
> passes on all platforms.
This pull request has now been integrated.
Changeset: c8e70d
On Fri, 4 Oct 2024 17:16:26 GMT, Jayathirth D V wrote:
>> Clean up and open source some of the AWT window tests.
>> Automated test `OwnedWindowShowTest.java` is verified in our CI also and it
>> passes on all platforms.
>
> Jayathirth D V has updated the pull reques
On Fri, 4 Oct 2024 13:41:19 GMT, Alexey Ivanov wrote:
>> Jayathirth D V has updated the pull request incrementally with one
>> additional commit since the last revision:
>>
>> Update based on review comments
>
> test/jdk/java/awt/Window/LocationByPlatform/Te
> Clean up and open source some of the AWT window tests.
> Automated test `OwnedWindowShowTest.java` is verified in our CI also and it
> passes on all platforms.
Jayathirth D V has updated the pull request incrementally with one additional
commit since the last revision:
Update f
On Fri, 4 Oct 2024 10:42:23 GMT, Jayathirth D V wrote:
>> Clean up and open source more of the AWT window tests.
>> Automated tests `PopupProxyCrash` and `NoResizeEvent` are also verified in
>> our CI.
>
> Jayathirth D V has updated the pull request incrementally with
On Fri, 4 Oct 2024 07:33:42 GMT, Jayathirth D V wrote:
> Clean up and open source more of the AWT window tests.
> Automated tests `PopupProxyCrash` and `NoResizeEvent` are also verified in
> our CI.
This pull request has now been integrated.
Changeset: db61458d
Author:Jayathirth
On Fri, 4 Oct 2024 09:28:15 GMT, Abhishek Kumar wrote:
>> Jayathirth D V has updated the pull request incrementally with one
>> additional commit since the last revision:
>>
>> Add headful tag in PopupProxyCrash Test
>
> test/jdk/java/awt/Window/
> Clean up and open source more of the AWT window tests.
> Automated tests `PopupProxyCrash` and `NoResizeEvent` are also verified in
> our CI.
Jayathirth D V has updated the pull request incrementally with one additional
commit since the last revision:
Update based on review
> Clean up and open source more of the AWT window tests.
Jayathirth D V has updated the pull request incrementally with one additional
commit since the last revision:
Add headful tag in PopupProxyCrash Test
-
Changes:
- all: https://git.openjdk.org/jdk/pull/21348/files
-
On Thu, 3 Oct 2024 19:21:29 GMT, Phil Race wrote:
>> Open source several AWT tests, mostly related to menu short cuts.
>
> Phil Race has updated the pull request incrementally with one additional
> commit since the last revision:
>
> 834
Marked as reviewed by jdv (Reviewer).
---
1 - 100 of 329 matches
Mail list logo