Re: RFR: 8263788: JavaFX application freezes completely after some time when using the WebView

2021-04-20 Thread Guru Hb
On Mon, 12 Apr 2021 13:10:53 GMT, Arun Joseph wrote: > Issue: Java application (with WebView) will completely freeze after using it > for a while. > > Fix: Use native isMainThread functions instead of JNI call. Looks good to me . Tested on Windows with Modified WebView which loads multiple UR

Re: RFR: 8260165: CSSFilterTest.testCSSFilterRendering system test fails

2021-02-26 Thread Guru Hb
On Mon, 22 Feb 2021 11:55:00 GMT, Arun Joseph wrote: > Issue: Initial layout delay was removed and layout() is called from > layoutTimer instead of WebPage::prePaint(). > > Fix: Re-introduce the initial layout delay. Tested on Linux and Windows, Looks good to me. - Marked as revi

Re: RFR: 8260257: [Linux] WebView no longer reacts to some mouse events

2021-02-22 Thread Guru Hb
On Wed, 17 Feb 2021 14:14:35 GMT, Arun Joseph wrote: > Timer in RunLoopGeneric has an open bug in WebKit > (https://bugs.webkit.org/show_bug.cgi?id=189335) causing the timer to remain > active even after firing. > > Reverting back to WebCore Timer for ScrollAnimation in Linux. Looks good to m

Re: RFR: 8261927: WebKit build fails with Visual Studio 2017

2021-02-21 Thread Guru Hb
On Thu, 18 Feb 2021 13:25:14 GMT, Arun Joseph wrote: > The WebKit build fails with Visual Studio 2017. > > Issue: Visual Studio 2017 doesn't support if constexpr in lambda > > Test: Build webkit with the VS2017 compiler with and without this fix. It > should fail without the fix and build with

Re: RFR: 8259635: Update to 610.2 version of WebKit [v2]

2021-01-22 Thread Guru Hb
On Thu, 21 Jan 2021 15:57:04 GMT, Arun Joseph wrote: >> Update JavaFX WebKit to GTK WebKit 2.30 (610.2) > > Arun Joseph has updated the pull request incrementally with one additional > commit since the last revision: > > Fix color in DefaultPlugin Changes looks good to me. Did a local build

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

2020-12-02 Thread Guru Hb
On Tue, 1 Dec 2020 03:12:10 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/classes/sun/securi

Re: RFR: 8247963: Update SQLite to version 3.32.3

2020-07-02 Thread Guru Hb
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 Looks good to me - Marked as reviewed by ghb (Reviewer). PR: ht

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

2020-05-21 Thread Guru Hb
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 SetTimer() call

Re: RFR: 8242505: Some WebKit tests might fail because Microsoft libraries are not loaded

2020-04-28 Thread Guru Hb
On Wed, 22 Apr 2020 18:07:09 GMT, Kevin Rushforth wrote: > This PR will allow javafx.web unit tests to run to completion and pass on > Windows with Visual Studio 2019. > > Two of the WebKit tests load the native WebKit library without initializing > the JavaFX runtime. This will lead to test >

Re: RFR: 8242077: Add information about HTTP/2 and HttpClient usage in WebEngine

2020-04-28 Thread Guru Hb
On Sun, 19 Apr 2020 08:12:07 GMT, Abhinay Agarwal wrote: > Update WebEngine's Javadoc to add information on how it switches to > HttpClient instead of URLConnection in JavaFX 14 > when used with JDK 12 or later. > Identification of the correct client is important as both these clients may > of

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

2020-04-17 Thread Guru Hb
On Wed, 15 Apr 2020 15:14:49 GMT, Arun Joseph wrote: >> 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 val

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

2020-04-10 Thread Guru Hb
On Wed, 8 Apr 2020 11:55:03 GMT, Arun Joseph wrote: >> 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 valu

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

2020-04-10 Thread Guru Hb
On Mon, 6 Apr 2020 11:57:54 GMT, Arun Joseph wrote: > 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(

Re: [Rev 01] RFR: 8240694: [macos 10.15] JavaFX Media hangs on some video files on Catalina

2020-04-09 Thread Guru Hb
On Thu, 9 Apr 2020 01:45:27 GMT, Alexander Matveev wrote: >> https://bugs.openjdk.java.net/browse/JDK-8240694 >> >> - Original fix JDK-8236832 was reverted. >> - Timestamp will be queried on event loop thread when spectrum event is >> received by event loop. >> - FIx only enabled for macOS when

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

2020-03-26 Thread Guru Hb
On Wed, 25 Mar 2020 18:12:41 GMT, Arun Joseph wrote: >> 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 `pixelScal

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

2020-03-05 Thread Guru Hb
On Thu, 5 Mar 2020 14:51:03 GMT, Arun Joseph wrote: > 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

Re: RFR: 8236832: [macos 10.15] JavaFX Application hangs on video play on Cata…

2020-02-26 Thread Guru Hb
On Wed, 26 Feb 2020 03:39:49 GMT, Alexander Matveev wrote: > https://bugs.openjdk.java.net/browse/JDK-8236832 > > - This issue most likely caused by changes in AVFoundation APIs on macOS > 10.15. > - Getting currentTime from AVPlayer was blocked if it is done from > ProcessAudioTap callback (

Re: [Integrated] RFR: 8239109: Update SQLite to version 3.31.1

2020-02-23 Thread Guru Hb
Changeset: 4eaff0de Author:Guru Hb Date: 2020-02-24 04:08:43 + URL: https://git.openjdk.java.net/jfx/commit/4eaff0de 8239109: Update SQLite to version 3.31.1 Reviewed-by: ajoseph, kcr, jvos ! modules/javafx.web/src/main/native/Source/ThirdParty/sqlite/VERSION ! modules

RFR: 8239109: Update SQLite to version 3.31.1

2020-02-19 Thread Guru Hb
Updating SQLite to version 3.31.1 (Currently used version is 3.30.1). - Commits: - d4e17340: 8239109: Update SQLite to version 3.31.1 Changes: https://git.openjdk.java.net/jfx/pull/119/files Webrev: https://webrevs.openjdk.java.net/jfx/119/webrev.00 Issue: https://bugs.openjdk.ja

Re: [Integrated] RFR: 8239454: LLIntData : invalid opcode returned for 16 and 32 bit wide instructions

2020-02-19 Thread Guru Hb
Changeset: d8e7f850 Author:Arun Joseph Committer: Guru Hb Date: 2020-02-20 05:33:04 + URL: https://git.openjdk.java.net/jfx/commit/d8e7f850 8239454: LLIntData : invalid opcode returned for 16 and 32 bit wide instructions Reviewed-by: ghb, kcr ! modules/javafx.web/src/main

RFR: 8239513: Update SQLite to version 3.31.1

2020-02-19 Thread Guru Hb
Updating SQlite to 3.31.1 (Currently used 3.30.1). - Commits: - e8312460: 8239513: Update SQLite to version 3.31.1 Changes: https://git.openjdk.java.net/jfx/pull/118/files Webrev: https://webrevs.openjdk.java.net/jfx/118/webrev.00 Issue: https://bugs.openjdk.java.net/browse/JDK-8

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

2020-02-19 Thread Guru Hb
On Wed, 19 Feb 2020 12:09:50 GMT, Arun Joseph wrote: > 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

Re: [Integrated] RFR: 8238526: Cherry pick GTK WebKit 2.26.3 changes

2020-02-06 Thread Guru Hb
Changeset: e224e549 Author:Guru Hb Date: 2020-02-07 02:13:15 + URL: https://git.openjdk.java.net/jfx/commit/e224e549 8238526: Cherry pick GTK WebKit 2.26.3 changes Reviewed-by: kcr, jvos ! modules/javafx.web/src/main/native/Source/JavaScriptCore/dfg

RFR: 8238526: Cherry pick GTK WebKit 2.26.3 changes

2020-02-06 Thread Guru Hb
Fix contains https://webkitgtk.org/2020/01/22/webkitgtk2.26.3-released.html - Commits: - 025373db: 8238526: Cherry pick GTK WebKit 2.26.3 changes Changes: https://git.openjdk.java.net/jfx/pull/107/files Webrev: https://webrevs.openjdk.java.net/jfx/107/webrev.00 Issue: https://bug

Re: [Integrated] RFR: 8237944: webview native cl "-m32" unknown option for windows 32-bit build

2020-02-03 Thread Guru Hb
Changeset: aa91ebbb Author:Guru Hb Date: 2020-02-03 08:51:25 + URL: https://git.openjdk.java.net/jfx/commit/aa91ebbb 8237944: webview native cl "-m32" unknown option for windows 32-bit build Reviewed-by: kcr ! modules/javafx.web/src/main/native/Tools/Scripts/webkitdirs.pm

Re: [Integrated] RFR: 8237003: Remove hardcoded WebAnimationsCSSIntegrationEnabled flag in DumpRenderTree

2020-01-29 Thread Guru Hb
Changeset: b96bc52b Author:Guru Hb Date: 2020-01-29 16:44:56 + URL: https://git.openjdk.java.net/jfx/commit/b96bc52b 8237003: Remove hardcoded WebAnimationsCSSIntegrationEnabled flag in DumpRenderTree Reviewed-by: kcr ! modules/javafx.web/src/main/java/com/sun/javafx/webkit

Re: [Rev 01] RFR: 8237944: webview native cl "-m32" unknown option for windows 32-bit build

2020-01-29 Thread Guru Hb
On Wed, 29 Jan 2020 12:57:41 GMT, Kevin Rushforth wrote: >> The pull request has been updated with 1 additional commit. > > Marked as reviewed by kcr (Lead). > the -m32 option seems to be ignored by the compiler: > cl : Command line warning D9002 : ignoring unknown option '-m32' > > However, I

RFR: 8237003: Remove hardcoded WebAnimationsCSSIntegrationEnabled flag in DumpRenderTree

2020-01-29 Thread Guru Hb
AnimationBase and WebAnimation are two different abstract animation API provider. By default KeyFrameAnimation (which is a sub class of AnimationBase) is used for controlling and rendering the CSS animation. Enabling "WebAnimationsCSSIntegrationEnabled" overrides the CSSAnimationController to u

Re: [Rev 01] RFR: 8237944: webview native cl "-m32" unknown option for windows 32-bit build

2020-01-29 Thread Guru Hb
> cl : Command line warning D9002 : ignoring unknown option '-m32' > > post fix for "https://trac.webkit.org/changeset/242724/webkit"; makes use of > cross compiling 32 bit JSC in a 64 bit and its holds good only for Linux. > '-m32' flag is gcc specifc and on windows cl.exe (visual studio) doesn

Re: [Rev 01] RFR: 8237944: webview native cl "-m32" unknown option for windows 32-bit build

2020-01-29 Thread Guru Hb
On Tue, 28 Jan 2020 23:42:08 GMT, Kevin Rushforth wrote: >> The pull request has been updated with 1 additional commit. > > modules/javafx.web/src/main/native/Tools/Scripts/webkitdirs.pm line 2300: > >> 2299: >> 2300: if (architecture() eq "x86_64" && shouldBuild32Bit() && (isJava() >> &&

RFR: 8237944: webview native cl "-m32" unknown option for windows 32-bit build

2020-01-28 Thread Guru Hb
cl : Command line warning D9002 : ignoring unknown option '-m32' post fix for "https://trac.webkit.org/changeset/242724/webkit"; makes use of cross compiling 32 bit JSC in a 64 bit and its holds good only for Linux. '-m32' flag is gcc specifc and on windows cl.exe (visual studio) doesn't recogn

Re: [Rev 03] RFR: 8236912: NullPointerException when clicking in WebView with Button 4 or Button 5

2020-01-24 Thread Guru Hb
On Fri, 24 Jan 2020 14:41:11 GMT, Robert Lichtenberger wrote: >> As documented in JDK-8236912, WebView did not check whether the idMap really >> contained a mapping for the given button, making it prone to errors, when >> things are extended (as has happened here). >> >> The fix consists of t

Re: [Rev 03] RFR: 8236912: NullPointerException when clicking in WebView with Button 4 or Button 5

2020-01-24 Thread Guru Hb
On Fri, 17 Jan 2020 23:18:28 GMT, Kevin Rushforth wrote: >> Previous commits in this pull request have been removed, probably due to a >> force push. The incremental views will show differences compared to the >> previous content of the PR. > > Looks good to me. > > @guruhb can you re-review

RFR: 8233942: Update to 609.1 version of WebKit

2020-01-21 Thread Guru Hb
Updated GTK Webkit 2.26 (609.1) into jfx. - Commits: - 8b48b1e0: 8233942: Update to 609.1 version of WebKit Changes: https://git.openjdk.java.net/jfx/pull/91/files Webrev: https://webrevs.openjdk.java.net/jfx/91/webrev.00 Issue: https://bugs.openjdk.java.net/browse/JDK-8233942

Re: [Rev 02] RFR: 8236912: NullPointerException when clicking in WebView with Button 4 or Button 5

2020-01-14 Thread Guru Hb
On Tue, 14 Jan 2020 14:13:54 GMT, Robert Lichtenberger wrote: >> As documented in JDK-8236912, WebView did not check whether the idMap really >> contained a mapping for the given button, making it prone to errors, when >> things are extended (as has happened here). >> >> The fix consists of t

Re: RFR: 8236912: NullPointerException when clicking in WebView with Button 4 or Button 5

2020-01-14 Thread Guru Hb
On Tue, 14 Jan 2020 12:01:11 GMT, Robert Lichtenberger wrote: > As documented in JDK-8236912, WebView did not check whether the idMap really > contained a mapping for the given button, making it prone to errors, when > things are extended (as has happened here). > > The fix consists of two te

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

2020-01-05 Thread Guru Hb
On Mon, 9 Dec 2019 13:09:04 GMT, Arun Joseph wrote: > 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

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

2020-01-05 Thread Guru Hb
On Mon, 6 Jan 2020 04:32:34 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 >> causes the JVM to crash when retrieving the value. >> >> Fix: Return from the function if

Re: [Approved] RFR: 8211308: Support HTTP/2 in WebView

2019-11-28 Thread Guru Hb
On Wed, 16 Oct 2019 17:57:58 GMT, Arunprasad Rajkumar wrote: > The pull request has been updated with additional changes. > > > > Added commits: > - 1832c2db: Incorporate fixes provided by @kcr > > Changes: > - all: https://git.openjdk.java.net/jfx/pull/14/files > - new:

Re: [Integrated] RFR: 8218640: Update ICU4C to version 64.2

2019-10-15 Thread Guru Hb
Changeset: b6e53f4f Author:Arun Joseph Committer: Guru Hb Date: 2019-10-15 17:41:33 + URL: https://git.openjdk.java.net/jfx/commit/b6e53f4f 8218640: Update ICU4C to version 64.2 Reviewed-by: kcr, ghb, jvos ! modules/javafx.web/src/main/legal/icu_web.md ! modules/javafx.web

Re: [Approved] RFR: 8218640: Update ICU4C to version 64.2

2019-10-09 Thread Guru Hb
On Wed, 9 Oct 2019 13:25:54 GMT, Arun Joseph wrote: > 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.

Re: [Approved] RFR: 8231854: Change Mercurial to git in various README files

2019-10-04 Thread Guru Hb
On Thu, 3 Oct 2019 23:15:56 GMT, Kevin Rushforth wrote: > https://bugs.openjdk.java.net/browse/JDK-8231854 > > Follow-on fix to > [JDK-8231590](https://bugs.openjdk.java.net/browse/JDK-8231590) -- PR #3 -- > to change Mercurial or hg to git in a few README files where appropriate. > > ---

[10] Review request: 8191035: WebView Canvas Graphics2D arc renders incorrectly

2017-11-19 Thread Guru Hb
Hi Kevin, Arun, Please review JBS : https://bugs.openjdk.java.net/browse/JDK-8191035 Webrev : http://cr.openjdk.java.net/~ghb/8191035/webrev.00/ RC and solution update in JBS. Thanks, Guru

Re: CFV: New OpenJFX Committer: Ambarish Rapte

2017-11-01 Thread Guru Hb
Vote: yes > On 01-Nov-2017, at 1:48 AM, Kevin Rushforth > wrote: > > I hereby nominate Ambarish Rapte [1] to OpenJFX Committer. > > Ambarish is a member of JavaFX team at Oracle, who has contributed 10 > changesets [2][3] to OpenJFX. > > Votes are due by November 14, 2017. > > Only current

[10] Review request: 8185654: Fix copyright notice on WebKit porting layer files in platform/java

2017-10-24 Thread Guru Hb
Please review the fix for https://bugs.openjdk.java.net/browse/JDK-8185654 http://cr.openjdk.java.net/~ghb/8185654/8185654_0.diff Thanks, Guru

Re: [webkit][10] Review request: 8187568: JavaFX crash in libjfxwebkit.so

2017-10-11 Thread Guru Hb
Resending with correct Bug id in Subject line. > On 11-Oct-2017, at 6:57 PM, Guru Hb wrote: > > Hi Arun, Murali & Kevin, > > Please review the fix for > JBS : https://bugs.openjdk.java.net/browse/JDK-8187568 > <https://bugs.openjdk.java.net/browse

[webkit][10] Review request: 8170955: JavaFX crash in libjfxwebkit.so

2017-10-11 Thread Guru Hb
Hi Arun, Murali & Kevin, Please review the fix for JBS : https://bugs.openjdk.java.net/browse/JDK-8187568 Webrev: http://cr.openjdk.java.net/~ghb/8187568/webrev.00/ RC and solution updated i

Re: CFV: New OpenJFX Committer: Prasanta Sadhukhan

2017-09-28 Thread Guru Hb
Vote: YES > On 28-Sep-2017, at 6:40 AM, Kevin Rushforth > wrote: > > I hereby nominate Prasanta Sadhukhan [1] to OpenJFX Committer. > > Prasanta is a member of JavaFX team at Oracle, who has contributed 12 > changesets [2] to OpenJFX. > > Votes are due by October 11, 2017. > > Only current

[10] Review request: 8187361: Revert non reviewed commit done for the fix to JDK-8087528

2017-09-07 Thread Guru Hb
Hi Murali & Arun, Please do the review for JBS : https://bugs.openjdk.java.net/browse/JDK-8187361 Webrev : http://cr.openjdk.java.net/~ghb/8187361/webrev.00/ Thanks, Guru

[webkit][10] Review request: 8170955: HTML5 canvas shadow properties should not be affected by transform

2017-08-22 Thread Guru Hb
Hi Arun & Murali, Please review the fix for : JBS : https://bugs.openjdk.java.net/browse/JDK-8170955 webrev : http://cr.openjdk.java.net/~ghb/8170955/webrev.00/ Thanks, Guru

[webkit][10] Review request: 8186061: libjfxwebkit.dylib ld warning libicudata.a was built for newer OSX version

2017-08-10 Thread Guru Hb
Hi Arun, Murali, Please review the fix for https://bugs.openjdk.java.net/browse/JDK-8186061 webrev : http://cr.openjdk.java.net/~ghb/8186061/webrev.00/ Thanks, Guru

[webkit][10] 8185940: Web native compiled files not removed during gradle clean

2017-08-09 Thread Guru Hb
Hi Arun, Murali, Please review the fix for JBS : https://bugs.openjdk.java.net/browse/JDK-8185940 WebRev : http://cr.openjdk.java.net/~ghb/8185940/webrev.00 Thanks, Guru

[webkit][10] 8185315: Update user agent string to JavaFX/10

2017-07-28 Thread Guru Hb
Hi Arun & Murali, Please review the fix for JBS : https://bugs.openjdk.java.net/browse/JDK-8185315 Webrev : http://cr.openjdk.java.net/~ghb/8185315/webrev.00/ Thanks, Guru

[10] Review request: 8184797: Copy DumpRenderTreeJava.dll from bin to lib directory

2017-07-18 Thread Guru Hb
Hi Arun & Murali, Please review the fix for: JBS : https://bugs.openjdk.java.net/browse/JDK-8184797 Webrev : http://cr.openjdk.java.net/~ghb/8184797/webrev.00/ Thanks, Guru

Re: Support for JavaScript FileReader object in JavaFX WebView

2017-07-17 Thread Guru Hb
Hi Jelurida, You are correct, We are not yet (in Java 9) supporting File Api in our javafx.WebView. Will update once we have this enhancement is implemented https://bugs.openjdk.java.net/browse/JDK-8184785 . Thanks, Guru > On 17-Jul-2017, at

[10] Review request: 8178360: Build and integrate ICU from source

2017-06-29 Thread Guru Hb
Hi Kevin, Arun and Murali, Please review the fix for : JBS : https://bugs.openjdk.java.net/browse/JDK-8178360 Webrev : http://cr.openjdk.java.net/~ghb/8178360/webrev.07 (Modified Source) patch

[10] Review request: 8179673: JVM Crash in WebPage.setBackgroundColor() during webpage navigation (Non Public API)

2017-05-11 Thread Guru Hb
Hi Arun, Murali & Kevin, Please review the fix for : https://bugs.openjdk.java.net/browse/JDK-8179673 Webrev : http://cr.openjdk.java.net/~ghb/8179673/webrev.00/ Thanks, Guru

[9] [8u] Review request for 8177284: HTMLEditor fails to load resources from modena.css if security manager is present

2017-03-21 Thread Guru Hb
Hi Kevin & Jonathan, Please review the fix for: JBS: https://bugs.openjdk.java.net/browse/JDK-8177284 Webrev: http://cr.openjdk.java.net/~ghb/8177284/webrev.00/ RC and solution updated in J

[9] Review request: 8174919: SocketException no longer handled by WebView when processing web pages

2017-02-14 Thread Guru Hb
Hi Kevin & Murali, Please review the fix for JBS : https://bugs.openjdk.java.net/browse/JDK-8174919 Webrev : http://cr.openjdk.java.net/~ghb/8174919/webrev.00/ Thanks, Guru

Re: CFV: New OpenJFX Committer: Semyon Sadetsky

2017-01-27 Thread Guru Hb
Vote: YES > On 26-Jan-2017, at 1:09 AM, David Hill wrote: > > > I hereby nominate Semyon Sadetsky to OpenJFX Committer. > > Semyon Sadetsky is part of the JavaFX team focusing on glass. > > A list of Semyon's commits and reviews is available by the following links > > http://hg.openjdk.java.

[9] Review request: 8158196: WebView Form Post fails if connection is closed before keepAlive-Timeout

2017-01-23 Thread Guru Hb
Hi Kevin, Arun & Murali, Please review the fix for: JBS : https://bugs.openjdk.java.net/browse/JDK-8158196 Webrev : http://cr.openjdk.java.net/~ghb/8158196/webrev.00/ RC and solution update

Re: CFV: New OpenJFX Committer: Ramesh Gangadhar

2017-01-23 Thread Guru Hb
Vote: yes > On 19-Jan-2017, at 3:23 AM, Kevin Rushforth > wrote: > > I hereby nominate Ramesh Gangadhar [1] to OpenJFX Committer. > > Ramesh is a member of JavaFX SQE team at Oracle working on test development > for the Java packager, who has contributed 20 changesets to OpenJFX, at least > 8

Re: [WebView] Single page application - relative resource in a different jar

2016-11-18 Thread Guru Hb
Hi Steve, Could you please share the layout of jar file (two jars) so i can re-create the problem and let you know the failure reason. Thanks, Guru > On 16-Nov-2016, at 2:37 PM, Hruda, Steve > wrote: > > Hi, > short question from my side. I’ve submitted the following bug report which is > fi

[9] Review request for 8157686: JavaFX WebView fails to track URL changes for PJAX websites

2016-10-28 Thread Guru Hb
Hi Arun, Kevin & Murali, Please review the fix for JBS: https://bugs.openjdk.java.net/browse/JDK-8157686 Webrev: http://cr.openjdk.java.net/~ghb/8157686/webrev.00/ RC and Solution updated in

Re: SVG

2016-10-11 Thread Guru Hb
WebView renders SVG ( WebView.getWebEngine().load("PathToSvg.svg"); ). Could you please let us know which content (SVG) failed to rendered as expected. Please file a JBS with simplified test content , will analyze. Thanks, Guru On 11/10/16 6:56 PM, Scott Palmer wrote: On Oct 11, 2016, at 8:36

Re: [9] Review request for 8165853: Loading "https://www.windyty.com" with JavaFX WebView crashes JVM.

2016-09-19 Thread Guru Hb
x27; Bottom Right). b. Launch dev tool before launching windyty.com and use the debugger tabs. -Guru On 20/9/16 4:31 AM, Dan Howard wrote: I can't even right-click view source. Is this Flash or something? On 9/19/2016 6:15 PM, Guru Hb wrote: Hi Kevin, Arun & Murali, Please review the

[9] Review request for 8165853: Loading "https://www.windyty.com" with JavaFX WebView crashes JVM.

2016-09-19 Thread Guru Hb
Hi Kevin, Arun & Murali, Please review the fix for (RC and fix updated in JBS) JBS : https://bugs.openjdk.java.net/browse/JDK-8165853 Webrev : http://cr.openjdk.java.net/~ghb/8165853/webrev.00/index.html Thanks, Guru

[9] Review Request for 8165508: Incorrect Bug ID in comment for JDK-8164076

2016-09-06 Thread Guru Hb
Hi Ankit, Murali & Kevin, Please review the fix for https://bugs.openjdk.java.net/browse/JDK-8165508 Webrev : http://cr.openjdk.java.net/~ghb/8165508/webrev.00/index.html Thanks,

[9] Review request for 8164076: [Windows] JavaFX crash in WebPage.twkOpen in 8u112 when closing WebView while debugging

2016-08-31 Thread Guru Hb
Hi Kevin, Murali & Ankit, Please review the fix for JBS : https://bugs.openjdk.java.net/browse/JDK-8164076 Webrev : http://cr.openjdk.java.net/~ghb/8164960/webrev.00/ RC and Solution updated in JBS. Thanks, Guru

Review request for 8144263 : [WebView, OS X] Webkit rendering artifacts with inertia scrolling

2016-08-29 Thread Guru Hb
Hi Kevin, Arun & Murali, Please review the fix for https://bugs.openjdk.java.net/browse/JDK-8144263 Webrev : http://cr.openjdk.java.net/~ghb/8144263/webrev.00/ RC and Solution updated in JBS

[9][tests]Review request for 8164498 : Backout changeset c9f45449d053 which has wrong bug id and commit message

2016-08-19 Thread Guru Hb
Hi Kevin, Please review the change for (Backing out the changes done for JDK-8163084 with wrong bug ID and commit message). Webrev : http://cr.openjdk.java.net/~ghb/rbgangad/8164498/webrev.00/ JBS : https://bugs.openjdk.java.net/browse/JDK-8164498 Thanks, Guru

[9] Review request 8163971: Revert "MiscellaneousTest.testRT26306 and testWebViewWithoutSceneGraph" Indentation done while JDK-8163582 fix

2016-08-12 Thread Guru Hb
Hi Kevin, Please review the fix for JBS : https://bugs.openjdk.java.net/browse/JDK-8163971 Webrev : http://cr.openjdk.java.net/~ghb/8163971/webrev.00/ Accidentally Indent two other Unit test case while merging JDK-8163582. Thanks, Guru

[9] Review request 8163582: JavaFX browser can get stuck in an infinite loop when calling path.getTotalLength()

2016-08-11 Thread Guru Hb
Hi Kevin, Arun & Murali, Please review the fix for : JBS : https://bugs.openjdk.java.net/browse/JDK-8163582 Webrev : http://cr.openjdk.java.net/~ghb/8163582/webrev.00/ RC : Merged back JDK-8090035 and added Unit test case. Thanks, Guru

[9] Review request: 8163183: [test] Correct assertEquals ("", actual, expected) in LoadTest.loadJarFile

2016-08-08 Thread Guru Hb
Hi Kevin & Arun, Please review the fix for JBS : https://bugs.openjdk.java.net/browse/JDK-8163183 Webrev : http://cr.openjdk.java.net/~ghb/8163183/webrev.00/ Thanks, Guru

[9] Review request 8153681: WebView needs to resolve resources relative to "jrt:" URLs

2016-07-29 Thread Guru Hb
Hi Kevin, Arun & Murali, Please review the fix for : https://bugs.openjdk.java.net/browse/JDK-8153681 Webrev : http://cr.openjdk.java.net/~ghb/8153681/webrev.00/ Solution updated in JBS. Thanks, Guru

Re: Problem building WebView with latest 8u-dev and 9-dev

2016-07-24 Thread Guru Hb
webview-deps (version 1.3.1) has been uploaded to maven central. Please revert the work around suggested earlier. -- Guru On 20/7/16 3:52 AM, Kevin Rushforth wrote: With the recently-integrated changes from 8u102, the WebKit build requires a new version of the webview-deps bundle. This new bund

Re: CFV: New OpenJFX Committer: Andrey Rusakov

2016-07-24 Thread Guru Hb
Vote: YES On 23/7/16 9:05 PM, Murali Billa wrote: Vote: YES Kevin Rushforth wrote: I hereby nominate Andrey Rusakov [1] to OpenJFX Committer. Andrey is a member of JavaFX SQE team at Oracle working on test development, who has contributed 19 changesets [5] to OpenJFX, at least 8 of which are

Re: CFV: New OpenJFX Committer: Ankit Srivastava

2016-07-19 Thread Guru Hb
Vote: YES On 19/7/16 7:07 PM, David Hill wrote: I hereby nominate Ankit Srivastava to OpenJFX Committer. Ankit Srivastava is part of the JavaFX team focusing on Web. A list of Ankit's commits and reviews is available by the following links http://hg.openjdk.java.net/openjfx/8u-dev/rt/log?re

Re: [9] Review request 8160837: WebEngine doesn't handle html5 color picker

2016-07-18 Thread Guru Hb
Hi Kevin, Arun & Alexander Z, Please review the updated webrev : http://cr.openjdk.java.net/~ghb/8160837/webrev.02/ Solution updated in JBS. Thanks, Guru On 14/7/16 10:30 AM, Guru Hb wrote: Hi Jonathan, Arun & Alexander Z, Please review the fix for: https://bugs.openjdk.java.ne

[9] Review request 8160837: WebEngine doesn't handle html5 color picker

2016-07-13 Thread Guru Hb
Hi Jonathan, Arun & Alexander Z, Please review the fix for: https://bugs.openjdk.java.net/browse/JDK-8160837 Webrev : http://cr.openjdk.java.net/~ghb/8160837/webrev.00/ Thanks, Guru

Re: Fix for test runner review request.

2016-07-07 Thread Guru Hb
http://hg.openjdk.java.net/openjfx/8u-dev/tests/rev/7562f73c8642 http://hg.openjdk.java.net/openjfx/9-dev/tests/rev/30fbc7690076 On 7/7/16 1:51 AM, Andrey Rusakov wrote: Could anyone commit this fix to open fx tests, 8 and 9? It's important for testing process. 05.07.2016 12:56, Andrey Rusako

Re: [webkit][9] Review request 8160563: jvm crash at javafx com.sun.webkit.WebPage.twkPrePaint (GFlag + Heap verification)

2016-07-04 Thread Guru Hb
Please ref webrev : http://cr.openjdk.java.net/~ghb/8160563/webrev.00 On 5/7/16 9:43 AM, Guru Hb wrote: Hi Alexander & Arun, Please review the fix for: JBS : https://bugs.openjdk.java.net/browse/JDK-8160563 webrev : file:///D:/ws/patch/8160563/webrev.00 RC and Solution updated in

[webkit][9] Review request 8160563: jvm crash at javafx com.sun.webkit.WebPage.twkPrePaint (GFlag + Heap verification)

2016-07-04 Thread Guru Hb
Hi Alexander & Arun, Please review the fix for: JBS : https://bugs.openjdk.java.net/browse/JDK-8160563 webrev : file:///D:/ws/patch/8160563/webrev.00 RC and Solution updated in JBS. Thanks, Guru

Re: [9] Review request : 8159868 :the JVM for our Swing application crashes, once we login into our application server

2016-06-22 Thread Guru Hb
Please ignore my previous email and ref https://bugs.openjdk.java.net/browse/JDK-8159868 http://cr.openjdk.java.net/~ghb/8159868/webrev.00/ Thanks, Guru On 22/6/16 10:34 AM, Guru Hb wrote: Hi Arun & Kevin, Could you please review the fix for : https://bugs.openjdk.java.net/browse/JDK-815

[9] Review request : 8159868 :the JVM for our Swing application crashes, once we login into our application server

2016-06-21 Thread Guru Hb
Hi Arun & Kevin, Could you please review the fix for : https://bugs.openjdk.java.net/browse/JDK-8159868 http://cr.openjdk.java.net/~ghb/8159868/webrev.00/ RC and solution update in JBS. Thanks, Guru

[9] Review request: 8157145: DRT crash at fast/css-generated-content/initial-letter-basic.html

2016-05-22 Thread Guru Hb
Hi Arun & Kevin, Please review the fix : JBS : https://bugs.openjdk.java.net/browse/JDK-8157145 Webrev : http://cr.openjdk.java.net/~ghb/8157145/webrev.00/ RC and Solution updated in JBS. Thanks, Guru

Re: Problems with multiple WebView

2016-05-19 Thread Guru Hb
Hi Matthieu Brouillard, Please file a bug at http://bugreport.java.com/bugreport/ along with JDK/JRE version and on which platform Windows / Linux / OS X details. Instructions : https://wiki.openjdk.java.net/display/OpenJFX/Submitting+a+Bug+Report Thanks, Guru On 19/5/16 2:55 PM, Matthieu

Re: Review request for 8152420 : [WebView] Icon font doesn't work if single page application will be loaded from jar

2016-04-27 Thread Guru Hb
forth wrote: Yes, please open a new bug. -- Kevin David Hill wrote: On 4/27/16, 9:56 AM, Guru Hb wrote: Hi Dave & Kevin, Could you please review updated Unit test case http://cr.openjdk.java.net/~ghb/8152420/webrev.01/ Solution updated in JBS. Hi Guru, JDK-8154186 is closed, so I su

Re: [9] Review request: 8154186: JVM crash while using JavaFW WebView and colgroup/cols html feature

2016-04-27 Thread Guru Hb
: On 4/27/16, 9:56 AM, Guru Hb wrote: Hi Dave & Kevin, Could you please review updated Unit test case http://cr.openjdk.java.net/~ghb/8152420/webrev.01/ Solution updated in JBS. Hi Guru, JDK-8154186 is closed, so I suspect you will need to open a new bug. Kevin will say for sure bu

Re: [9] Review request: 8154186: JVM crash while using JavaFW WebView and colgroup/cols html feature

2016-04-27 Thread Guru Hb
Hi Dave & Kevin, Could you please review updated Unit test case http://cr.openjdk.java.net/~ghb/8152420/webrev.01/ Solution updated in JBS. Thanks, Guru On 20/4/16 7:25 PM, Guru Hb wrote: Hi Alexander Z , Kevin & Arunprasad, Please review the fix : https://bugs.openjdk.java.net/br

[9] Review request: 8154186: JVM crash while using JavaFW WebView and colgroup/cols html feature

2016-04-20 Thread Guru Hb
Hi Alexander Z , Kevin & Arunprasad, Please review the fix : https://bugs.openjdk.java.net/browse/JDK-8154186 http://cr.openjdk.java.net/~ghb/8154186/webrev.00/ Root Cause and solution updated in JBS. Thanks, Guru

Re: status behind JDK-8149738

2016-04-13 Thread Guru Hb
problem not only on 32bits versions but also on 64bits 8u74 & 8u77. Thanks Matthieu On Wed, Apr 13, 2016 at 5:07 AM, Guru Hb <mailto:guru...@oracle.com>> wrote: Hi Matthieu, If you could update the re-producer html/js in the JBS would be very help full. I cou

Re: status behind JDK-8149738

2016-04-12 Thread Guru Hb
filled. Tomorrow I'll build a clean GWT reproducer app and will send it to you. I'll also do the exact same in pure html/js without GWT being involved. Matthieu On Tue, Apr 12, 2016 at 11:18 AM, Guru Hb <mailto:guru...@oracle.com>> wrote: Thank you Matthieu, Let me cre

Re: status behind JDK-8149738

2016-04-12 Thread Guru Hb
uce the crash with a simple app consisting of the 2 lines above. On Tue, Apr 12, 2016 at 9:41 AM, Guru Hb <mailto:guru...@oracle.com>> wrote: Hi Matthieu, Crash is due to Mouse event is not handled correctly from WebView context. Could you please provide these deta

Re: status behind JDK-8149738

2016-04-12 Thread Guru Hb
Hi Matthieu, Crash is due to Mouse event is not handled correctly from WebView context. Could you please provide these details. 1. What Content is loaded to WebEngine. (Live webpage or custom content) 2. which mouse event lead to crash (Right / left click, Double click, Scroll) 3. W.r.t 2nd po

Review request for 8152420 : [WebView] Icon font doesn't work if single page application will be loaded from jar

2016-04-04 Thread Guru Hb
Hi Alexander, Arunprasad & Kevin, JBS : https://bugs.openjdk.java.net/browse/JDK-8152420 Webrev : http://cr.openjdk.java.net/~ghb/8152420/webrev.00/ Root cause and fix description updated in JBS. Thanks, Guru

Re: [9] Review request for 8152315 : Don't server / request the Cached resource for if its Main resource is explicitly cancelled

2016-03-30 Thread Guru Hb
Updated webrev : http://cr.openjdk.java.net/~ghb/8152315/webrev.01/ Which address heap corruption detected by using GFlag On 22/3/16 12:54 AM, Guru Hb wrote: Hi Alexander, Arunprasad & Kevin, JBS : https://bugs.openjdk.java.net/browse/JDK-8152315 Webrev : http://cr.openjdk.java.net/

[9] Review request for 8153049 : Merge "Frame and Page lifetime fixes in WebCore::createwindows" and "FrameLoader::checkCompleted can hit the "ref'ing while destroyed" assertion"

2016-03-29 Thread Guru Hb
Hi Alexander, Arunprasad & Kevin, JBS : https://bugs.openjdk.java.net/browse/JDK-8153049 Webrev : http://cr.openjdk.java.net/~ghb/8153049/webrev.00/ Merging webkit change set, Root cause and description are updated in JBS. Thanks, Guru

[9] Review request for 8152315 : Don't server / request the Cached resource for if its Main resource is explicitly cancelled

2016-03-21 Thread Guru Hb
Hi Alexander, Arunprasad & Kevin, JBS : https://bugs.openjdk.java.net/browse/JDK-8152315 Webrev : http://cr.openjdk.java.net/~ghb/8152315/webrev.00/ RC and Solution updated in JIRA, fix tested on Windows. Thanks, Guru

[9] Review request for 8139114: WebView crashes on Yahoo login page

2016-02-20 Thread Guru Hb
Hi Alexander, Kevin and Arun JBS : https://bugs.openjdk.java.net/browse/JDK-8139114 Webrev : http://cr.openjdk.java.net/~ghb/8139114/webrev.00/ RC and Solution updated in JIRA. Thanks, Guru

  1   2   >