RFR: 8230492: font-family not set in HTMLEditor if font name has a number in it

2019-10-30 Thread Arun Joseph
In the HTMLEditor, when positioning the caret in a text and trying to set a font-family that has a number in it is not working. Issue: In [CSSPropertyParser.cpp](https://github.com/openjdk/jfx/blob/master/modules/javafx.web/src/main/native/Source/WebCore/css/parser/CSSPropertyParser.cpp),

Re: RFR: 8230492: font-family not set in HTMLEditor if font name has a number in it

2019-11-04 Thread Arun Joseph
On Wed, 30 Oct 2019 13:27:32 GMT, Kevin Rushforth wrote: > On Wed, 30 Oct 2019 11:09:38 GMT, Hadzic Samir wrote: > >> On Wed, 30 Oct 2019 10:07:42 GMT, Arun Joseph >> wrote: >> >>> In the HTMLEditor, when positioning the caret in a text and trying to set a

RFR: 8231188: Update SQLite to version 3.30.1

2019-11-06 Thread Arun Joseph
We currently use SQLite version 3.28.0. We should update to the latest stable release version 3.30.1 released. https://www.sqlite.org/index.html Commits: - 24c6375d: 8231188: Update SQLite to version 3.30.1 Changes: https://git.openjdk.java.net/jfx/pull/31/files Webrev:

RFR: 8234239: [TEST_BUG] Reenable few ignored web tests

2019-11-15 Thread Arun Joseph
The following tests can be reenabled: CookieManager.testPutOverwriteExpired CookieManager.testPutPurgeDomainAfterExpiry CookieManager.testPutPurgeCookiesGlobally2 CookieManager.testPutPurgeCookiesGlobally3 CookieManager.testPutPurgeCookiesGloballyAfterExpiry CallbackTest.testCustomPopup

Re: [Rev 01] RFR: 8234239: [TEST_BUG] Reenable few ignored web tests

2019-11-15 Thread Arun Joseph
The pull request has been updated with additional changes. Added commits: - ca460353: Remove ignore imports and update copyright Changes: - all: https://git.openjdk.java.net/jfx/pull/42/files - new: https://git.openjdk.java.net/jfx/pull/42/files/f2f121d9..ca460353

Re: [Rev 01] RFR: 8234239: [TEST_BUG] Reenable few ignored web tests

2019-11-17 Thread Arun Joseph
On Fri, 15 Nov 2019 23:03:46 GMT, Kevin Rushforth wrote: > On Fri, 15 Nov 2019 10:01:51 GMT, Arun Joseph wrote: > >> The pull request has been updated with additional changes. >> >> >> >> Added commits: >> - ca460353:

Re: [Rev 02] RFR: 8234239: [TEST_BUG] Reenable few ignored web tests

2019-11-17 Thread Arun Joseph
The pull request has been updated with additional changes. Added commits: - 8c2b988d: Modified LeakTest.testOleg Changes: - all: https://git.openjdk.java.net/jfx/pull/42/files - new: https://git.openjdk.java.net/jfx/pull/42/files/ca460353..8c2b988d Webrevs: - full:

RFR: 8234056: Upgrade to libxslt 1.1.34

2019-11-18 Thread Arun Joseph
We need to update the version of libxslt to the latest version, 1.1.34. Commits: - 2d9c0bf3: 8234056: Upgrade to libxslt 1.1.34 Changes: https://git.openjdk.java.net/jfx/pull/45/files Webrev: https://webrevs.openjdk.java.net/jfx/45/webrev.00 Issue:

Re: RFR: WIP: 8230231: font-family not updated in HTMLEditor

2019-11-05 Thread Arun Joseph
On Tue, 5 Nov 2019 09:26:56 GMT, Arun Joseph wrote: > On Thu, 10 Oct 2019 14:56:19 GMT, Hadzic Samir wrote: > >> On Thu, 10 Oct 2019 12:19:18 GMT, Kevin Rushforth wrote: >> >>> On Thu, 10 Oct 2019 09:12:57 GMT, Hadzic Samir wrote: >>> >>>>

Re: RFR: WIP: 8230231: font-family not updated in HTMLEditor

2019-11-05 Thread Arun Joseph
On Thu, 10 Oct 2019 14:56:19 GMT, Hadzic Samir wrote: > On Thu, 10 Oct 2019 12:19:18 GMT, Kevin Rushforth wrote: > >> On Thu, 10 Oct 2019 09:12:57 GMT, Hadzic Samir wrote: >> >>> On Wed, 9 Oct 2019 20:09:58 GMT, Kevin Rushforth wrote: >>> On Wed, 9 Oct 2019 16:09:58 GMT, Hadzic Samir

Re: [Rev 01] RFR: 8230492: font-family not set in HTMLEditor if font name has a number in it

2019-11-05 Thread Arun Joseph
The pull request has been updated with additional changes. Added commits: - afc7f17a: Minor formatting Changes: - all: https://git.openjdk.java.net/jfx/pull/27/files - new: https://git.openjdk.java.net/jfx/pull/27/files/5a1fbade..afc7f17a Webrevs: - full:

RFR: 8234593: Mark LeakTest.testGarbageCollectability as unstable

2019-11-21 Thread Arun Joseph
The test.javafx.scene.web.LeakTest.testGarbageCollectability unit test is failing intermittently. Commits: - 1f7c5c8e: 8234593: Mark LeakTest.testGarbageCollectability as unstable Changes: https://git.openjdk.java.net/jfx/pull/49/files Webrev:

Re: [Rev 02] RFR: 8233747: JVM crash in com.sun.webkit.dom.DocumentImpl.createAttribute

2019-12-04 Thread Arun Joseph
The pull request has been updated with additional changes. Added commits: - 7f6ed5bf: Added test for createAttribute Changes: - all: https://git.openjdk.java.net/jfx/pull/47/files - new: https://git.openjdk.java.net/jfx/pull/47/files/2bd56c11..7f6ed5bf Webrevs: - full:

Re: RFR: 8233747: JVM crash in com.sun.webkit.dom.DocumentImpl.createAttribute

2019-12-04 Thread Arun Joseph
On Thu, 28 Nov 2019 13:16:19 GMT, Arun Joseph wrote: > On Wed, 20 Nov 2019 15:04:07 GMT, Kevin Rushforth wrote: > >> On Wed, 20 Nov 2019 07:05:40 GMT, Arun Joseph wrote: >> >>> Issue: Native part of WebView throws a DOMException and then, continues >>>

Re: [Rev 03] RFR: 8233747: JVM crash in com.sun.webkit.dom.DocumentImpl.createAttribute

2019-12-04 Thread Arun Joseph
The pull request has been updated with additional changes. Added commits: - 472d77cf: Added spaces Changes: - all: https://git.openjdk.java.net/jfx/pull/47/files - new: https://git.openjdk.java.net/jfx/pull/47/files/7f6ed5bf..472d77cf Webrevs: - full:

RFR: 8234471: Canvas in webview displayed with wrong scale on Windows

2019-12-09 Thread Arun Joseph
This bug can be reproduced when the screen resolution is at 125%, 150% and 175% for Windows, which correpsonds to `pixelScale` values of 1.25, 1.5 and 1.75, respectively. Issue: The rectangle inside canvas is rendered on `pixelScale` while the borders are rendered on `Math.ceil(pixelScale)`

RFR: 8210955: DOMTest::testEventListenerCascade fails

2019-11-18 Thread Arun Joseph
Modified the test to work with WebView's ordering of capturing and bubbling event listeners. Commits: - cbef0069: DOMTest::testEventListenerCascade fails Changes: https://git.openjdk.java.net/jfx/pull/46/files Webrev: https://webrevs.openjdk.java.net/jfx/46/webrev.00

Re: [Rev 01] RFR: 8233747: JVM crash in com.sun.webkit.dom.DocumentImpl.createAttribute

2019-11-28 Thread Arun Joseph
The pull request has been updated with additional changes. Added commits: - 2bd56c11: Modified only raiseOnDOMError Changes: - all: https://git.openjdk.java.net/jfx/pull/47/files - new: https://git.openjdk.java.net/jfx/pull/47/files/acc52780..2bd56c11 Webrevs: - full:

Re: RFR: 8233747: JVM crash in com.sun.webkit.dom.DocumentImpl.createAttribute

2019-11-28 Thread Arun Joseph
On Wed, 20 Nov 2019 15:04:07 GMT, Kevin Rushforth wrote: > On Wed, 20 Nov 2019 07:05:40 GMT, Arun Joseph wrote: > >> Issue: Native part of WebView throws a DOMException and then, continues >> executing the rest of the function assuming that value is present. This >>

RFR: 8218640: Update ICU4C to version 64.2

2019-10-09 Thread Arun Joseph
We currently use ICU4C version 62.1. We should update to the latest stable version 64.2. http://site.icu-project.org/home Commits: - b56b720e: 8218640: Update ICU4C to version 64.2 Changes: https://git.openjdk.java.net/jfx/pull/10/files Webrev:

RFR: 8240211: Stack overflow on Windows 32-bit can lead to crash

2020-03-05 Thread Arun Joseph
Issue: The stack pointer is checked close to the stack limit during the last iteration of calling frameLoaded() and then, grows beyond the thread's stack range causing a stack overflow and crashes. This occurs as the stack grows by an amount larger than the reserved zone at the end of the

Re: [Rev 01] RFR: 8234471: Canvas in webview displayed with wrong scale on Windows

2020-02-24 Thread Arun Joseph
> This bug can be reproduced when the screen resolution is at 125%, 150% and > 175% for Windows, which correpsonds to `pixelScale` values of 1.25, 1.5 and > 1.75, respectively. > > Issue: The rectangle inside canvas is rendered on `pixelScale` while the > borders are rendered on

Re: RFR: 8233942: Update to 609.1 version of WebKit

2020-01-22 Thread Arun Joseph
On Mon, 20 Jan 2020 16:39:15 GMT, Guru Hb wrote: > Updated GTK Webkit 2.26 (609.1) into jfx. Marked as reviewed by ajoseph (Author). - PR: https://git.openjdk.java.net/jfx/pull/91

RFR: 8239454: LLIntData : invalid opcode returned for 16 and 32 bit wide instructions

2020-02-19 Thread Arun Joseph
With new bytecode format introduced in Webkit 608.1, the unlinked and linked bytecodes were replaced by narrow (1-byte operand) and wide (4-byte operand) bytecodes. These were extended to narrow, wide16 and wide32 bytecodes in WebKit 609.1. In narrow instructions, each argument of the opcode

Re: [Rev 01] RFR: 8237889: Update libxml2 to version 2.9.10

2020-02-20 Thread Arun Joseph
> We currently use libxml2 version 2.9.9. We should update to the latest stable > release 2.9.10. The pull request has been updated with 1 additional commit. - Added commits: - 9c76239b: 8237889: Update libxml2 to version 2.9.10 Changes: - all:

RFR: 8237889: Update libxml2 to version 2.9.10

2020-02-20 Thread Arun Joseph
We currently use libxml2 version 2.9.9. We should update to the latest stable release 2.9.10. - Commits: - eaf8d1b1: 8237889: Update libxml2 to version 2.9.10 Changes: https://git.openjdk.java.net/jfx/pull/120/files Webrev: https://webrevs.openjdk.java.net/jfx/120/webrev.00

Re: RFR: 8239109: Update SQLite to version 3.31.1

2020-02-19 Thread Arun Joseph
On Thu, 20 Feb 2020 05:58:03 GMT, Guru Hb wrote: > Updating SQLite to version 3.31.1 (Currently used version is 3.30.1). Marked as reviewed by ajoseph (Author). - PR: https://git.openjdk.java.net/jfx/pull/119

FYI: 8243112: Skip failing test SVGTest.testSVGRenderingWithPattern

2020-04-18 Thread Arun Joseph
SVGTest.testSVGRenderingWithPattern is failing frequently due to [JDK-8243110](https://bugs.openjdk.java.net/browse/JDK-8243110). We should skip this test until it is fixed. - Commit messages: - 8243112: Skip failing test SVGTest.testSVGRenderingWithPattern Changes:

Re: RFR: 8230809: HTMLEditor formatting lost when selecting all (CTRL-A)

2020-04-07 Thread Arun Joseph
On Wed, 1 Apr 2020 10:49:22 GMT, Ajit Ghaisas wrote: > Bug : https://bugs.openjdk.java.net/browse/JDK-8230809 > > Root Cause : > Turned out to be a simpler issue than thought. > Atomicity check was missed while updating toolbar state (which in turn > updates styles). > > Fix : > Added the

RFR: 8223298: SVG patterns are drawn wrong

2020-04-08 Thread Arun Joseph
Issue: Assuming the pixelScale is 2, the tile image size is doubled at the native side which is propagated to the java side as well. But, as transform initialization takes place after scaling, the transform is reset to default value. Fix: Override scale() method in WCBufferedContext class to

Re: RFR: WIP: 8230231: font-family not updated in HTMLEditor

2020-04-06 Thread Arun Joseph
On Wed, 12 Feb 2020 20:34:55 GMT, Kevin Rushforth wrote: >> I have indeed let this bug on the side. I would like to investigate but my >> time is limited these days. Especially that >> my patch is apparently creating another issue. I will try to give it another >> shot in the upcoming week.

Re: [Rev 01] RFR: 8223298: SVG patterns are drawn wrong

2020-04-08 Thread Arun Joseph
BufferedContext class to call init() before > scaling. Arun Joseph has updated the pull request incrementally with one additional commit since the last revision: Update copyright year - Changes: - all: https://git.openjdk.java.net/jfx/pull/164/files - new: https://git.openjdk

Re: [Rev 02] RFR: 8234471: Canvas in webview displayed with wrong scale on Windows

2020-03-25 Thread Arun Joseph
d on `Math.ceil(pixelScale)` > > Fix: Use `Math.ceil(pixelScale)` for calculating `pixelScaleTransform` Arun Joseph has updated the pull request incrementally with one additional commit since the last revision: Updated test - Changes: - all: https://git.openjdk.java.net/jfx/

RFR: 8242209: Increase web native thread stack size for x86 mode

2020-04-06 Thread Arun Joseph
CLoop interpreter used in 32-bit Windows uses 87 KB of stack space each time CLoop::execute() is called. For web native threads which has a default stack size of 320 KB, a Stack Overflow Error is raised just after two calls to execute() function. While 64-bit windows has a default stack size of

Re: RFR: 8242861: Update ImagePattern to apply SVG pattern transforms

2020-04-29 Thread Arun Joseph
On Mon, 27 Apr 2020 23:56:37 GMT, Kevin Rushforth wrote: >> fillPath() and fillRect() functions in >> [GraphicsContextJava.cpp](https://github.com/openjdk/jfx/blob/master/modules/javafx.web/src/main/native/Source/WebCore/platform/graphics/java/GraphicsContextJava.cpp) >> use Image::drawPattern()

RFR: 8242861: Update ImagePattern to apply SVG pattern transforms

2020-04-22 Thread Arun Joseph
fillPath() and fillRect() functions in [GraphicsContextJava.cpp](https://github.com/openjdk/jfx/blob/master/modules/javafx.web/src/main/native/Source/WebCore/platform/graphics/java/GraphicsContextJava.cpp) use Image::drawPattern() for applying patterns as fill. But drawPattern() doesn't use

Re: [Rev 01] RFR: 8242861: Update ImagePattern to apply SVG pattern transforms

2020-04-23 Thread Arun Joseph
> doesn't use patternTransform argument as > ImagePattern doesn't have the same attribute. So, the final image won't be > transformed. Arun Joseph has updated the pull request incrementally with one additional commit since the last revision: Add CanvasTest - Chang

Re: RFR: 8242861: Update ImagePattern to apply SVG pattern transforms

2020-04-29 Thread Arun Joseph
On Wed, 29 Apr 2020 06:45:04 GMT, Arun Joseph wrote: >> Since this is in a common method used by all shapes, and not just WebView, >> we will need to ensure no regressions. > > All ImagePattern objects, other than the one from WebView, uses the second > constructor whic

Re: RFR: 8171898: [WebView] ScrollBarWidget thickness calculation is not correct

2020-04-29 Thread Arun Joseph
On Tue, 28 Apr 2020 21:41:19 GMT, Kevin Rushforth wrote: >> ScrollbarThemeJava delegates scrollbar thickness calculation to >> ScrollBarWidget::initializeThickness() >> [ScrollBarWidget.java] method. Since >> "ScrollbarThemeJava::scrollbarThickness" is not associated with any >>

Re: [Rev 02] RFR: 8223298: SVG patterns are drawn wrong

2020-04-15 Thread Arun Joseph
BufferedContext class to call init() before > scaling. Arun Joseph has updated the pull request incrementally with one additional commit since the last revision: Added SVGTest - Changes: - all: https://git.openjdk.java.net/jfx/pull/164/files - new: https://git.openjdk.jav

Re: [Rev 02] RFR: 8218973: SVG with masking is not rendering image with mask effect

2020-05-17 Thread Arun Joseph
On Sun, 10 May 2020 20:49:07 GMT, Bhawesh Choudhary wrote: >> Root cause of issue is Specifying a image mask from GraphicsContextJava.cpp >> in WebKit was not implemented, so masking >> doesn't take place at all while rendering SVGRect. to fix this issue add >> implementation of function

Re: [Closed] RFR: 8171898: [WebView] ScrollBarWidget thickness calculation is not correct

2020-05-05 Thread Arun Joseph
On Mon, 27 Apr 2020 08:17:50 GMT, Arun Joseph wrote: > ScrollbarThemeJava delegates scrollbar thickness calculation to > ScrollBarWidget::initializeThickness() > [ScrollBarWidget.java] method. Since "ScrollbarThemeJava::scrollbarThickness" > is not associated with

Re: [Rev 03] RFR: 8218973: SVG with masking is not rendering image with mask effect

2020-05-18 Thread Arun Joseph
On Mon, 18 May 2020 08:00:38 GMT, Bhawesh Choudhary wrote: >> Root cause of issue is Specifying a image mask from GraphicsContextJava.cpp >> in WebKit was not implemented, so masking >> doesn't take place at all while rendering SVGRect. to fix this issue add >> implementation of function

Re: RFR: 8239107: Update libjpeg to version 9d

2020-03-17 Thread Arun Joseph
On Thu, 12 Mar 2020 10:43:37 GMT, Ambarish Rapte wrote: > Currently we use IJG libjpeg version 9c released on 14-Jan-2018 > We should update the libjpeg to latest version [9d](http://www.ijg.org/) > released on 12-Jan-2020 > > reference: https://jpegclub.org/reference/reference-sources/

RFR: 8244579: Windows "User Objects" leakage with WebView

2020-05-20 Thread Arun Joseph
Cause: The Window Class `RunLoopMessageWindow` is never registered (this happens because registerRunLoopMessageWindowClass() is moved to MainThreadWin.cpp while openjfx uses MainThreadJava.cpp) and this causes every setTimer() to create a new user object instead of reusing the same object over

Re: RFR: 8244579: Windows "User Objects" leakage with WebView

2020-05-20 Thread Arun Joseph
On Wed, 20 May 2020 15:16:11 GMT, Kevin Rushforth wrote: >> Cause: The Window Class `RunLoopMessageWindow` is never registered (this >> happens because >> registerRunLoopMessageWindowClass() is moved to MainThreadWin.cpp while >> openjfx uses MainThreadJava.cpp) and this >> causes every

Re: [Integrated] RFR: 8244579: Windows "User Objects" leakage with WebView

2020-05-21 Thread Arun Joseph
On Wed, 20 May 2020 14:30:11 GMT, Arun Joseph wrote: > Cause: The Window Class `RunLoopMessageWindow` is never registered (this > happens because > registerRunLoopMessageWindowClass() is moved to MainThreadWin.cpp while > openjfx uses MainThreadJava.cpp) and this > causes every

Re: RFR: 8202990: javafx webview css filter property with display scaling [v2]

2020-09-01 Thread Arun Joseph
On Tue, 1 Sep 2020 06:29:17 GMT, Bhawesh Choudhary wrote: >> ImageJava.cpp ignores CompositeOperator parameter in drawImage function due >> to which shadow was getting drawn on top of >> actual image. apply given composite operator to graphics context before >> drawing image to fix this

Re: RFR: 8202990: javafx webview css filter property with display scaling [v2]

2020-09-01 Thread Arun Joseph
On Tue, 1 Sep 2020 06:29:17 GMT, Bhawesh Choudhary wrote: >> ImageJava.cpp ignores CompositeOperator parameter in drawImage function due >> to which shadow was getting drawn on top of >> actual image. apply given composite operator to graphics context before >> drawing image to fix this

RFR: 8252062: WebKit build fails with recent VS 2019 compiler

2020-09-02 Thread Arun Joseph
The WebKit build fails with recent VS 2019 compiler. Bug: This MediaQueryEvaluator constructor takes a String parameter, but the conversion of char* to String is failing. Fix: Use the default constructor of MediaQueryEvaluator, which also returns true for "all". Test: Build webkit with the

Integrated: 8252062: WebKit build fails with recent VS 2019 compiler

2020-09-02 Thread Arun Joseph
On Wed, 2 Sep 2020 11:22:45 GMT, Arun Joseph wrote: > The WebKit build fails with recent VS 2019 compiler. > > Bug: This MediaQueryEvaluator constructor takes a String parameter, but the > conversion of char* to String is failing. > > Fix: Use the default constructor of M

Re: RFR: 8202990: javafx webview css filter property with display scaling [v3]

2020-09-03 Thread Arun Joseph
On Thu, 3 Sep 2020 06:07:10 GMT, Bhawesh Choudhary wrote: >> ImageJava.cpp ignores CompositeOperator parameter in drawImage function due >> to which shadow was getting drawn on top of >> actual image. apply given composite operator to graphics context before >> drawing image to fix this

Integrated: 8253696: WebEngine refuses to load local "file:///" CSS stylesheets when using JDK 15

2020-10-05 Thread Arun Joseph
On Mon, 5 Oct 2020 07:08:33 GMT, Arun Joseph wrote: > Issue: In JDK15, > [URLConnection](https://github.com/openjdk/jdk/blob/master/src/java.base/share/classes/sun/net/www/URLConnection.java) > class overrode the getHeaderFields() methods to return properties, which > previou

RFR: 8253696: WebEngine refuses to load local "file:///" CSS stylesheets when using JDK 15

2020-10-05 Thread Arun Joseph
Issue: In JDK15, [URLConnection](https://github.com/openjdk/jdk/blob/master/src/java.base/share/classes/sun/net/www/URLConnection.java) class overrode the getHeaderFields() methods to return properties, which previously returned an empty map from its super class. Fix: Extract headers only when

Re: RFR: 8242861: Update ImagePattern to apply SVG pattern transforms [v6]

2020-10-11 Thread Arun Joseph
On Fri, 9 Oct 2020 22:14:15 GMT, Kevin Rushforth wrote: >> Arun Joseph has updated the pull request incrementally with two additional >> commits since the last revision: >> >> - Update copyright year >> - Minor refactoring > > modules/javafx.web/src/m

Re: RFR: 8242861: Update ImagePattern to apply SVG pattern transforms [v6]

2020-10-11 Thread Arun Joseph
On Fri, 9 Oct 2020 21:48:40 GMT, Kevin Rushforth wrote: >> Arun Joseph has updated the pull request incrementally with two additional >> commits since the last revision: >> >> - Update copyright year >> - Minor refactoring > > modules/javafx.grap

Re: RFR: 8242861: Update ImagePattern to apply SVG pattern transforms [v7]

2020-10-11 Thread Arun Joseph
> doesn't use patternTransform argument as > ImagePattern doesn't have the same attribute. So, the final image won't be > transformed. Arun Joseph has updated the pull request incrementally with one additional commit since the last revision: Fix incorrect concat param order

Re: RFR: 8242861: Update ImagePattern to apply SVG pattern transforms [v6]

2020-10-12 Thread Arun Joseph
On Mon, 12 Oct 2020 13:42:08 GMT, Kevin Rushforth wrote: >> Instead of negation, it should have been at the opposite positions. I have >> fixed this in the latest commit. The arg >> list order for >>

Re: RFR: 8254049: Update WebView to public suffix list 2020-04-24

2020-10-19 Thread Arun Joseph
On Mon, 19 Oct 2020 04:29:13 GMT, Arun Joseph wrote: > We should use the public_suffix_list.dat file in the JDK instead. Reading the > public_suffix_list.dat file is modified > to be similar to > [DomainName.java](https://github.com/openjdk/jdk/blob/master/src/java.base/share

RFR: 8254049: Update WebView to public suffix list 2020-04-24

2020-10-19 Thread Arun Joseph
We should use the public_suffix_list.dat file in the JDK instead. Reading the public_suffix_list.dat file is modified to be similar to [DomainName.java](https://github.com/openjdk/jdk/blob/master/src/java.base/share/classes/sun/security/util/DomainName.java). If the file is not present,

Re: RFR: 8202990: javafx webview css filter property with display scaling

2020-08-27 Thread Arun Joseph
On Tue, 11 Aug 2020 16:18:37 GMT, Bhawesh Choudhary wrote: > ImageJava.cpp ignores CompositeOperator parameter in drawImage function due > to which shadow was getting drawn on top of > actual image. apply given composite operator to graphics context before > drawing image to fix this issue.

Re: RFR: 8202990: javafx webview css filter property with display scaling

2020-08-26 Thread Arun Joseph
On Wed, 12 Aug 2020 15:59:20 GMT, Kevin Rushforth wrote: > With your patch installed, I see a race condition that manifests in one of > two ways: I couldn't reproduce the race condition or crash with the fix. > I suspect this might be related to the gc.setCompositeOperation call. This may

Integrated: 8252381: Cherry pick GTK WebKit 2.28.4 changes

2020-08-28 Thread Arun Joseph
On Thu, 27 Aug 2020 14:59:56 GMT, Arun Joseph wrote: > Update to GTK WebKit 2.28.4 > https://webkitgtk.org/2020/07/28/webkitgtk2.28.4-released.html This pull request has now been integrated. Changeset: 88c0f978 Author: Arun Joseph URL: https://git.openjdk.java.net/jfx/commit/88

Re: RFR: 8242861: Update ImagePattern to apply SVG pattern transforms [v4]

2020-08-20 Thread Arun Joseph
> doesn't use patternTransform argument as > ImagePattern doesn't have the same attribute. So, the final image won't be > transformed. Arun Joseph has updated the pull request incrementally with one additional commit since the last revision: Incorect axis of rotational -

RFR: 8252381: Cherry pick GTK WebKit 2.28.4 changes

2020-08-27 Thread Arun Joseph
Update to GTK WebKit 2.28.4 https://webkitgtk.org/2020/07/28/webkitgtk2.28.4-released.html - Commit messages: - 8252381: Cherry pick GTK WebKit 2.28.4 changes Changes: https://git.openjdk.java.net/jfx/pull/289/files Webrev: https://webrevs.openjdk.java.net/jfx/289/webrev.00

Re: RFR: 8242861: Update ImagePattern to apply SVG pattern transforms [v5]

2020-08-24 Thread Arun Joseph
> doesn't use patternTransform argument as > ImagePattern doesn't have the same attribute. So, the final image won't be > transformed. Arun Joseph has updated the pull request incrementally with three additional commits since the last revision: - Exclude TYPE_IDENTITY case - Anothe

Re: RFR: 8242861: Update ImagePattern to apply SVG pattern transforms [v6]

2020-08-24 Thread Arun Joseph
> doesn't use patternTransform argument as > ImagePattern doesn't have the same attribute. So, the final image won't be > transformed. Arun Joseph has updated the pull request incrementally with two additional commits since the last revision: - Update copyright year -

Re: RFR: 8240969: WebView does not allow to load style sheet in modularized applications [v4]

2020-08-14 Thread Arun Joseph
On Fri, 14 Aug 2020 06:23:18 GMT, Tobias Diez wrote: >> Currently, loading a style sheet file using >> `WebView.getEngine().setUserStyleSheetLocation(url)` fails if the url >> start's with `jrt`, i.e. if the file is packaged in an application image >> using jlink. This is fixed with this PR.

Re: RFR: 8218973: SVG with masking is not rendering image with mask effect [v13]

2020-09-17 Thread Arun Joseph
On Mon, 14 Sep 2020 13:47:28 GMT, Bhawesh Choudhary wrote: >> Root cause of issue is Specifying a image mask from GraphicsContextJava.cpp >> in WebKit was not implemented, so masking >> doesn't take place at all while rendering SVGRect. to fix this issue add >> implementation of function

Re: RFR: 8218973: SVG with masking is not rendering image with mask effect [v13]

2020-09-17 Thread Arun Joseph
On Thu, 17 Sep 2020 06:29:36 GMT, Arun Joseph wrote: >> Bhawesh Choudhary has updated the pull request incrementally with one >> additional commit since the last revision: >> >> Updates as per review comments > > The fix works when the shape is displayed init

RFR: 8243110: SVGTest.testSVGRenderingWithPattern fails intermittently

2020-05-28 Thread Arun Joseph
Issue: Snapshot taken before webpage is rendered Fix: Use stateProperty and sleep to wait for the page to finish rendering - Commit messages: - Add sleep - 8243110: SVGTest.testSVGRenderingWithPattern fails intermittently Changes: https://git.openjdk.java.net/jfx/pull/240/files

RFR: 8234540: javafx.web LeakTest.testGarbageCollectability fails intermittently

2020-05-26 Thread Arun Joseph
Cause: The test assumes that WebView/WebPage will be GCed right after `System.gc()` Fix: Use `runFinalization()` and `sleep()` to wait for objects to get GCed - Commit messages: - Update copyright year - Remove unwanted line - 8234540: javafx.web

Re: [Rev 01] RFR: 8243110: SVGTest.testSVGRenderingWithPattern fails intermittently

2020-05-29 Thread Arun Joseph
> Issue: Snapshot taken before webpage is rendered > > Fix: Use stateProperty and sleep to wait for the page to finish rendering Arun Joseph has updated the pull request incrementally with one additional commit since the last revision: Review comments - Changes: - a

Re: [Integrated] RFR: 8243110: SVGTest.testSVGRenderingWithPattern fails intermittently

2020-05-29 Thread Arun Joseph
On Thu, 28 May 2020 08:45:36 GMT, Arun Joseph wrote: > Issue: Snapshot taken before webpage is rendered > > Fix: Use stateProperty and sleep to wait for the page to finish rendering This pull request has now been integrated. Changeset: 9edba9cc Author:Arun Joseph URL:

Re: [Integrated] RFR: 8234540: javafx.web LeakTest.testGarbageCollectability fails intermittently

2020-05-27 Thread Arun Joseph
On Tue, 26 May 2020 09:27:52 GMT, Arun Joseph wrote: > Cause: The test assumes that WebView/WebPage will be GCed right after > `System.gc()` > > Fix: Use `runFinalization()` and `sleep()` to wait for objects to get GCed This pull request has now been integrated. Changeset: 891

Re: CFV: New OpenJFX Committer: Jeanette Winzenburg

2020-06-02 Thread Arun Joseph
Vote: YES — Arun Joseph > On 01-Jun-2020, at 7:22 PM, Kevin Rushforth > wrote: > > I hereby nominate Jeanette Winzenburg [1] to OpenJFX Committer. > > Jeanette is an OpenJFX community member, who has contributed 16 changesets > [2][3] to OpenJFX. > > Vot

Re: CFV: New OpenJFX Committer: Jose Pereda

2020-06-02 Thread Arun Joseph
Vote: YES — Arun Joseph > On 01-Jun-2020, at 7:23 PM, Kevin Rushforth > wrote: > > I hereby nominate Jose Pereda [1] to OpenJFX Committer. > > Jose is an OpenJFX community member, who has contributed 16 changesets [2][3] > to OpenJFX. > > Votes are due by June

Re: CFV: New OpenJFX Committer: Pankaj Bansal

2020-10-23 Thread Arun Joseph
Vote: YES > On 23-Oct-2020, at 8:08 PM, Kevin Rushforth > wrote: > > I hereby nominate Pankaj Bansal [1] to OpenJFX Committer. > > Pankaj is a member of Java client team at Oracle, who has contributed 11 > commits [2] to OpenJFX. > > Votes are due by November 6, 2020. > > Only current

RFR: 8252596: [TESTBUG] WebPageShim::paint is not thread-safe

2020-10-21 Thread Arun Joseph
Issue: Tests using `WebPageShim::paint` fails around 1/10 times. Fix: Execute `WebPageShim::paint` in the render thread using `PrismInvoker`. Ran tests over 100 times, no failures. Used the initial unit test in the PR for [JDK-8202990](https://bugs.openjdk.java.net/browse/JDK-8202990).

Re: RFR: 8181775: JavaFX WebView does not calculate border-radius properly [v4]

2020-08-09 Thread Arun Joseph
On Thu, 23 Jul 2020 09:43:36 GMT, Bhawesh Choudhary wrote: >> root cause of issue is prism's fillRoundedRect() API doesn't allow rendering >> of rounded corner rectangle if four >> corners have different radii. but same can be achieved via Path. to fix the >> issue, in GraphicsContextJava.cpp

Integrated: 8247963: Update SQLite to version 3.32.3

2020-07-02 Thread Arun Joseph
On Wed, 1 Jul 2020 17:14:30 GMT, Arun Joseph wrote: > We currently use SQLite version 3.31.1. We should update to the latest stable > release version 3.32.3 released on > 2020-06-18. https://www.sqlite.org/index.html This pull request has now been integrated. Changeset: f3a044

RFR: 8247963: Update SQLite to version 3.32.3

2020-07-01 Thread Arun Joseph
We currently use SQLite version 3.31.1. We should update to the latest stable release version 3.32.3 released on 2020-06-18. https://www.sqlite.org/index.html - Commit messages: - 8247963: Update SQLite to version 3.32.3 Changes: https://git.openjdk.java.net/jfx/pull/261/files

RFR: 8245284: Update to 610.1 version of WebKit

2020-07-09 Thread Arun Joseph
Update JavaFX WebKit to GTK WebKit 2.28 (610.1) - Commit messages: - Fix exec files - 8245284: Update to 610.1 version of WebKit Changes: https://git.openjdk.java.net/jfx/pull/263/files Webrev: https://webrevs.openjdk.java.net/jfx/263/webrev.00 Issue:

Re: [Rev 02] RFR: 8208169: can not print selected pages of web page

2020-06-14 Thread Arun Joseph
On Sun, 14 Jun 2020 19:13:41 GMT, Bhawesh Choudhary wrote: >> Print function of WebEngine.java ignores page range setting and prints given >> number of pages starting from first page, >> which is the root cause of this issue. To fix it, put check for page ranges >> and if it available, use it

Re: RFR: 8246357: Allow static build of webkit library on linux

2020-06-03 Thread Arun Joseph
On Tue, 2 Jun 2020 17:52:37 GMT, Johan Vos wrote: > add changes to build a static version of libjfxwebkit.a > Fox for JDK-8246357 Changes looks good to me. - Marked as reviewed by ajoseph (Committer). PR: https://git.openjdk.java.net/jfx/pull/245

Integrated: 8249839: Cherry pick GTK WebKit 2.28.3 changes

2020-07-24 Thread Arun Joseph
On Thu, 23 Jul 2020 05:34:50 GMT, Arun Joseph wrote: > Update to GTK WebKit 2.28.3 > https://webkitgtk.org/2020/07/09/webkitgtk2.28.3-released.html This pull request has now been integrated. Changeset: 926b5b6b Author: Arun Joseph URL: https://git.openjdk.java.net/jfx/commit/92

Re: RFR: 8218973: SVG with masking is not rendering image with mask effect [v6]

2020-07-24 Thread Arun Joseph
On Tue, 21 Jul 2020 21:50:39 GMT, Bhawesh Choudhary wrote: >> Root cause of issue is Specifying a image mask from GraphicsContextJava.cpp >> in WebKit was not implemented, so masking >> doesn't take place at all while rendering SVGRect. to fix this issue add >> implementation of function

RFR: 8249839: Cherry pick GTK WebKit 2.28.3 changes

2020-07-22 Thread Arun Joseph
Update to GTK WebKit 2.28.3 https://webkitgtk.org/2020/07/09/webkitgtk2.28.3-released.html - Commit messages: - Cherry pick GTK WebKit 2.28.3 changes Changes: https://git.openjdk.java.net/jfx/pull/271/files Webrev: https://webrevs.openjdk.java.net/jfx/271/webrev.00 Issue:

Integrated: 8245284: Update to 610.1 version of WebKit

2020-07-16 Thread Arun Joseph
On Fri, 10 Jul 2020 05:26:39 GMT, Arun Joseph wrote: > Update JavaFX WebKit to GTK WebKit 2.28 (610.1) This pull request has now been integrated. Changeset: 5e7e452c Author: Arun Joseph URL: https://git.openjdk.java.net/jfx/commit/5e7e452c Stats: 300855 lines in 5370 fi

Re: [jfx15] RFR: 8246343: Fix mistakes in FX API docs [v4]

2020-07-28 Thread Arun Joseph
On Mon, 27 Jul 2020 21:16:41 GMT, Nir Lisker wrote: >> Fixes https://bugs.openjdk.java.net/browse/JDK-8246343 and some additional >> fixes in the vicinity. > > Nir Lisker has updated the pull request incrementally with one additional > commit since the last revision: > > Addressed review

Re: RFR: 8242861: Update ImagePattern to apply SVG pattern transforms [v3]

2020-07-20 Thread Arun Joseph
> doesn't use patternTransform argument as > ImagePattern doesn't have the same attribute. So, the final image won't be > transformed. Arun Joseph has updated the pull request incrementally with one additional commit since the last revision: Modify SW pipeline - Chang

Re: RFR: 8254049: Update WebView to public suffix list 2020-04-24 [v4]

2020-11-27 Thread Arun Joseph
not present, `isPublicSuffix()` returns `false`, which is > similar to how WebKit ignores the public suffix check when it is disabled. > > Test: Run PublicSuffixesTest.java Arun Joseph has updated the pull request incrementally with one additional commit since the last revision: A

RFR: 8257897: Fix webkit build for XCode 12

2020-12-08 Thread Arun Joseph
The WebKit build fails with recent Xcode 12. Bug: `getVTablePointer()` should return a const void* Fix: Use const void* and remove dependency on JavaVM.framework in Xcode Upstream fix: https://bugs.webkit.org/show_bug.cgi?id=207871 Test: Build webkit with the Xcode 12 compiler with and without

Integrated: 8257897: Fix webkit build for XCode 12

2020-12-08 Thread Arun Joseph
On Tue, 8 Dec 2020 17:29:56 GMT, Arun Joseph wrote: > The WebKit build fails with recent Xcode 12. > > Bug: `getVTablePointer()` should return a const void* > > Fix: Use const void* and remove dependency on JavaVM.framework in Xcode > Upstream fix: https://bugs.webkit.o

Re: RFR: 8254049: Update WebView to public suffix list 2020-04-24 [v3]

2020-11-24 Thread Arun Joseph
not present, `isPublicSuffix()` returns `false`, which is > similar to how WebKit ignores the public suffix check when it is disabled. > > Test: Run PublicSuffixesTest.java Arun Joseph has updated the pull request incrementally with one additional commit since the last revision: Cr

Integrated: 8254049: Update WebView to public suffix list 2020-04-24

2020-12-03 Thread Arun Joseph
On Mon, 19 Oct 2020 04:29:13 GMT, Arun Joseph wrote: > We should use the public_suffix_list.dat file in the JDK instead. Reading the > public_suffix_list.dat file is modified to be similar to > [DomainName.java](https://github.com/openjdk/jdk/blob/master/src/java.base/share/cl

Re: RFR: 8254049: Update WebView to public suffix list 2020-04-24 [v5]

2020-11-30 Thread Arun Joseph
not present, `isPublicSuffix()` returns `false`, which is > similar to how WebKit ignores the public suffix check when it is disabled. > > Test: Run PublicSuffixesTest.java Arun Joseph has updated the pull request incrementally with one additional commit since the last revision:

Re: RFR: 8242361: JavaFX Web View crashes with Segmentation Fault, when HTML contains Data-URIs [v3]

2020-12-16 Thread Arun Joseph
On Tue, 15 Dec 2020 16:57:13 GMT, Matthias Bläsing wrote: >> The code in WTF::scheduleDispatchFunctionsOnMainThread assumes, that >> the java class com.sun.webkit.MainThread can be found be the JNI >> function FindClass. This is only true if the class is loadable by the >> system class loader.

Re: RFR: 8242361: JavaFX Web View crashes with Segmentation Fault, when HTML contains Data-URIs [v6]

2020-12-16 Thread Arun Joseph
On Wed, 16 Dec 2020 19:07:17 GMT, Matthias Bläsing wrote: >> The code in WTF::scheduleDispatchFunctionsOnMainThread assumes, that >> the java class com.sun.webkit.MainThread can be found be the JNI >> function FindClass. This is only true if the class is loadable by the >> system class loader.

Re: RFR: 8254049: Update WebView to public suffix list 2020-04-24 [v2]

2020-11-16 Thread Arun Joseph
not present, `isPublicSuffix()` returns `false`, which is > similar to how WebKit ignores the public suffix check when it is disabled. > > Test: Run PublicSuffixesTest.java Arun Joseph has updated the pull request incrementally with one additional commit since the last revision: C

  1   2   >