RE: RFR: 8214119: Update to 607.1 version of WebKit

2018-12-19 Thread Murali Billa
uot;, str, str.length())); Thanks, Murali -Original Message- From: Kevin Rushforth Sent: Wednesday, December 19, 2018 7:01 PM To: Arunprasad Rajkumar ; Murali Billa ; Johan Vos ; Joeri Sykora Cc: Praveen Srivastava ; Victor D'yakov ; openjfx-dev@openjdk.java.net List Subject: Re: RFR: 8

[12] RFR: JDK-8215231: Remove the redundant file readAsText.html

2018-12-11 Thread Murali Billa
Hi Kevin, Please review the simple fix. JBS: https://bugs.openjdk.java.net/browse/JDK-8215231 Webrev: http://cr.openjdk.java.net/~mbilla/8215231/webrev.01/ Thanks, Murali

RE: issue with WebView in 8u202

2018-11-20 Thread Murali Billa
Hi Arun, Regarding Cc: openjfx-dev@openjdk.java.net Subject: Re: issue with WebView in 8u202 Hi Anton, Looks like this is mostly related to Content Security Policy(CSP)[1] which have got strengthened after recent WebKit upgrades. I have reduced the given html further(PSB) & I could reproduc

[8u] RFR: JDK-8212014: JavaScriptBridgeTest::testBridgeExplicitOverloading test fails when run with newer JDK 8u

2018-11-02 Thread Murali Billa
Hi Kevin, Arun, Please review the patch for the following bug: JBS: https://bugs.openjdk.java.net/browse/JDK-8212014 Webrev: http://cr.openjdk.java.net/~mbilla/8212014/webrev.00/ Thanks, Murali

[12] RFR: JDK-8213139: TooltipFXTest. testTooltipLeak failing intermittently

2018-11-02 Thread Murali Billa
Hi Kevin, Arun, Please review the patch for the following bug: JBS: https://bugs.openjdk.java.net/browse/JDK-8213139 Webrev: http://cr.openjdk.java.net/~mbilla/8213139/webrev.00/ Thanks, Murali

[RFR] [12] JDK-8207772: File API and FileReader should be supported in WebView

2018-10-30 Thread Murali Billa
  Hi Kevin, Johan, Arun, Please review the below enhancement for File Reader support in JavaFX WebView. JIRA: https://bugs.openjdk.java.net/browse/JDK-8207772   Github PR: https://github.com/javafxports/openjdk-jfx/pull/250   Thanks, Murali

RE: [12,8u] RFR: JDK-8212158: FX: Update copyright year in docs, readme files to 2019

2018-10-25 Thread Murali Billa
Hi Kevin, +1 for 12 and 8u. Giving +1 in mail since currently not able to login to JBS. Thanks Murali -Original Message- From: Kevin Rushforth Sent: Thursday, October 25, 2018 5:44 PM To: Murali Billa ; openjfx-dev@openjdk.java.net Subject: [12,8u] RFR: JDK-8212158: FX: Update

RE: CFV: New OpenJFX Committer: Prem Balakrishnan

2018-07-11 Thread Murali Billa
Vote: YES > On 11-Jul-2018, at 5:11 AM, Kevin Rushforth > wrote: > > I hereby nominate Prem Balakrishnan [1] to OpenJFX Committer. > > Prem is a member of JavaFX team at Oracle, who has contributed 10 changesets > [2] to OpenJFX. > > Votes are due by July 24, 2018. > > Only current OpenJFX

[11] Review request for 8203698: JavaFX WebView crashes when visiting certain web sites

2018-05-25 Thread Murali Billa
  Hi Kevin, Arun Please review the below fix. JIRA: https://bugs.openjdk.java.net/browse/JDK-8203698   webrev:  http://cr.openjdk.java.net/~mbilla/8203698/webrev.00/ Thanks, Murali

RE: JDK-8195974: Replace use of java.util.logging in javafx with System logger

2018-05-17 Thread Murali Billa
anks, Murali From: Nir Lisker mailto:nlis...@gmail.com"nlis...@gmail.com> Sent: Saturday, April 28, 2018 1:06 AM To: Murali Billa mailto:murali.bi...@oracle.com"murali.bi...@oracle.com> Cc: HYPERLINK "mailto:openjfx-dev@openjdk.java.net"openjfx-dev@openjdk.java.net M

RE: JDK-8195974: Replace use of java.util.logging in javafx with System logger

2018-05-02 Thread Murali Billa
drt) and will get back to you.   Please let me know, if you have any queries for 1. Thanks, Murali From: Nir Lisker Sent: Saturday, April 28, 2018 1:06 AM To: Murali Billa Cc: openjfx-dev@openjdk.java.net Mailing Subject: JDK-8195974: Replace use of java.util.logging in javafx with Sy

[11] Review request for 8200418: webPage.executeCommand("removeFormat", null) removes the style of the body element

2018-04-17 Thread Murali Billa
  Hi Kevin, Arun Please review the below simple fix. JIRA: https://bugs.openjdk.java.net/browse/JDK-8200418   webrev:  http://cr.openjdk.java.net/~mbilla/8200418/webrev.00/ Thanks, Murali

RE: WebView: in jfx9 the ENTER key from keyboard does not add new lines on textareas (but works correctly on jfx8)

2018-04-15 Thread Murali Billa
Hi Mirco,   Yes..Issue is present in JDK 10 (verified  with  JDK 10+46).   I tested with JDK 11 and issue is not reproducible.   Thanks, Murali   From: Mirco Sent: Sunday, April 15, 2018 8:41 PM To: Murali Billa Cc: openjfx-dev@openjdk.java.net Subject: Re: WebView: in jfx9 the ENTER

RE: WebView: in jfx9 the ENTER key from keyboard does not add new lines on textareas (but works correctly on jfx8)

2018-04-14 Thread Murali Billa
Hi Mirco, I tried the below sample code with JDK9+180 in Ubuntu and able to reproduce this issue. I tried with latest JDK build in Ubuntu (and in windows too) and not able to reproduce this issue (i can see new lines are being added upon entering ENTER key) I suggest you can verify with late

[11] Review request for 8201176: gradle :web:test should print a warning if COMPILE_WEBKIT is false

2018-04-06 Thread Murali Billa
Hi Kevin, Please review the below simple fix. JIRA: https://bugs.openjdk.java.net/browse/JDK-8201176   webrev:  http://cr.openjdk.java.net/~mbilla/8201176/webrev.00/ Thanks, Murali

RE: gradle :web:test fails

2018-04-04 Thread Murali Billa
e webkit dll will be picked up from your source code repo (not from JDK). Thanks, Murali -Original Message- From: Murali Billa Sent: Thursday, April 05, 2018 10:19 AM To: Nir Lisker ; openjfx-dev@openjdk.java.net Mailing Subject: RE: gradle :web:test fails Hi Lisker, Can you

RE: gradle :web:test fails

2018-04-04 Thread Murali Billa
Hi Lisker, Can you print the value of useragentString ? Looks like you are using fxversion as 11 (you can print this value from System.getProperty("javafx.runtime.version");) but the useragentstring does not contain 11. final WebView webView = new WebView(); final WebEngine webEngine = webV

RE: CFV: New OpenJFX Committer: Rajath Kamath

2018-03-29 Thread Murali Billa
VOTE: YES > On 29-Mar-2018, at 10:12 PM, Kevin Rushforth > wrote: > > I hereby nominate Rajath Kamath [1] to OpenJFX Committer. > > Rajath is a member of JavaFX team at Oracle, who has contributed 17 > changesets [2][3] to OpenJFX. > > Votes are due by April 12, 2018. > > Only current Open

[11] Review request for 8198609: [TestBug] Some of the system tests need a message about exceptions being expected

2018-03-01 Thread Murali Billa
  Hi Kevin, Please review the below simple fix. JIRA:  https://bugs.openjdk.java.net/browse/JDK-8198609   webrev: http://cr.openjdk.java.net/~mbilla/8198609/webrev.00/ Thanks, Murali

[11] Review request for 8198610: [TestBug] test.javafx.scene.web.UserDataDirectoryTest logs on stderr

2018-02-26 Thread Murali Billa
  Hi Kevin, Please review the below simple fix. JIRA: https://bugs.openjdk.java.net/browse/JDK-8198610   webrev: http://cr.openjdk.java.net/~mbilla/8198610/webrev.00/ Thanks, Murali

[11] Review request for 8198587: Increase test timeout values to make test suite more stable

2018-02-26 Thread Murali Billa
  Hi Kevin, Please review the below simple fix. JIRA: https://bugs.openjdk.java.net/browse/JDK-8198587   webrev: http://cr.openjdk.java.net/~mbilla/8198587/webrev.00/ Thanks, Murali

[11] Review request for 8193207: [TESTBUG] Unit test case to check user agent string

2018-02-23 Thread Murali Billa
  Hi Kevin, Please review the below simple fix. JIRA: https://bugs.openjdk.java.net/browse/JDK-8193207   webrev: http://cr.openjdk.java.net/~mbilla/8193207/webrev.00/ Thanks, Murali

Re: CFV: New OpenJFX Committer: Rajath Kamath

2018-02-12 Thread Murali Billa
Vote: YES Kevin Rushforth wrote: > I hereby nominate Rajath Kamath [1] to OpenJFX Committer. > > Rajath is a member of JavaFX team at Oracle, who has contributed 11 > changesets [2][3] to OpenJFX. > > Votes are due by February 26, 2018. > > Only current OpenJFX Committers [4] are eligible to vo

[11] Review request for 8196592: Mark unstable tests so they will be skipped until fixed

2018-02-06 Thread Murali Billa
  Hi Kevin, Please review the below simple fix. JIRA: https://bugs.openjdk.java.net/browse/JDK-8196592   webrev: http://cr.openjdk.java.net/~mbilla/8196592/webrev.00/ Thanks, Murali

[11] Review request for 8196605: Robot tests fail on Windows platforms if terminal windows are in the way

2018-02-02 Thread Murali Billa
  Hi Kevin, Please review the below simple fix. JIRA: https://bugs.openjdk.java.net/browse/JDK-8196605     webrev: http://cr.openjdk.java.net/~mbilla/8196605/webrev.00/ Thanks, Murali

[11] Review request for 8195804: Remove unused qualified export of sun.net.www from java.base to javafx.web

2018-01-29 Thread Murali Billa
  Hi Kevin, Arun Please review the below simple fix. JIRA: https://bugs.openjdk.java.net/browse/JDK-8195804   Webrev: http://cr.openjdk.java.net/~mbilla/8195804/webrev.00/ Thanks, Murali

[11] Review request for 8193368: [OS X] Remove redundant files

2018-01-10 Thread Murali Billa
  Hi, Please review the below simple fix. JIRA: https://bugs.openjdk.java.net/browse/JDK-8193368   Webrev: http://cr.openjdk.java.net/~mbilla/8193368/webrev.00/   Thanks, Murali

[11] Review request for 8193590: Memory leak when using WebView with Tooltip

2018-01-09 Thread Murali Billa
  Hi, Please review the below simple fix. JIRA: https://bugs.openjdk.java.net/browse/JDK-8193590   Webrev: http://cr.openjdk.java.net/~mbilla/8193590/webrev.01/   Thanks, Murali

[11] Review request for 8193131: Need to remove SharedBuffer unit Test cases

2018-01-04 Thread Murali Billa
  Hi, Please review the below simple fix. JIRA: https://bugs.openjdk.java.net/browse/JDK-8193131   Webrev: http://cr.openjdk.java.net/~mbilla/8193131/webrev.00/   Thanks, Murali

[8u] Review request for 8193450: Port changes to java and native files from JDK-8192806 to FX 8u

2017-12-13 Thread Murali Billa
Hi Kevin, Please review the below simple fix. JIRA: https://bugs.openjdk.java.net/browse/JDK-8193450 Webrev: http://cr.openjdk.java.net/~mbilla/8193450/webrev.00/ Thanks, Murali

[10] Review request for 8193458: Incorrect cmake platform string for 32-bit Linux

2017-12-13 Thread Murali Billa
    Hi Kevin, Please review the below simple fix. JIRA: https://bugs.openjdk.java.net/browse/JDK-8193458   Webrev: http://cr.openjdk.java.net/~mbilla/8193458/webrev.00/ Thanks, Murali

RE: New OpenJFX Committer: Laurent Bourgès

2017-12-13 Thread Murali Billa
Vote: yes Thanks, Murali On 12/12/17, 1:25 PM, "openjfx-dev on behalf of Kevin Rushforth" wrote: I hereby nominate Laurent Bourgès [1] to OpenJFX Committer. Laurent is the author of the Marlin rasterizer, which is used as the default rasterizer for both Java2D and JavaFX.

[8u-Backport] Review request: 8187483: Update to 605.1 version of WebKit

2017-12-11 Thread Murali Billa
Hi All, Please review the following patch to merge the GTK WebKit 2.18 into FX 8u : http://cr.openjdk.java.net/~mbilla/8187483/8u-dev-bp/webrev.01/ Above link has one webrev and one changeset file: 1. rt-non-webkit — Contains changes other than "modules/web/src/main/native", it will be usef

[10] Review request: 8187483: Update to 605.1 version of WebKit

2017-12-07 Thread Murali Billa
Hi All, Following webrev contains changes to upgrade to newer WebKit which is based on GTK WebKit 2.18 [1]. http://cr.openjdk.java.net/~mbilla/8187483/webrev.01/ Above link has one webrev and one patchset file. 1. rt-non-webkit — Contains changes other than "modules/javafx.web/src/main/n

[10] Review request for 8191621: Windows build fails if VS150COMNTOOLS is set

2017-11-28 Thread Murali Billa
  Hi Kevin, Please review the below simple fix. JIRA: https://bugs.openjdk.java.net/browse/JDK-8191621   Webrev: http://cr.openjdk.java.net/~mbilla/8191621/webrev.00/   Thanks, Murali

[10] Review request for 8190349: Tooltip based on graphic contented only show less than 1 out of 10 times

2017-11-05 Thread Murali Billa
  Hi Kevin, Arun, Please review the below fix. JIRA: https://bugs.openjdk.java.net/browse/JDK-8190349   Webrev: http://cr.openjdk.java.net/~mbilla/8190349/webrev.00/   Thanks, Murali

[10] Review request for 8189420: Crash in :web:test in debug build

2017-10-17 Thread Murali Billa
  Hi Kevin, Arun Please review the below fix. JIRA: https://bugs.openjdk.java.net/browse/JDK-8189420   Webrev: http://cr.openjdk.java.net/~mbilla/8189420/webrev.00/   Thanks, Murali

[10] Review request for 8189350: Crash due to ASSERT(url == m_string) fail while loading URL

2017-10-16 Thread Murali Billa
  Hi Kevin, Arun Please review the below simple fix. JIRA: https://bugs.openjdk.java.net/browse/JDK-8189350   Webrev: http://cr.openjdk.java.net/~mbilla/8189350/webrev.00/   Thanks, Murali

[10] Review request for 8187366: Remove hardcoded MSVC Version in win.gradle

2017-09-28 Thread Murali Billa
  Hi Kevin, Arun Please review the below simple fix. JIRA: https://bugs.openjdk.java.net/browse/JDK-8187366   Webrev: http://cr.openjdk.java.net/~mbilla/8187366/webrev.01/     Thanks, Murali

[10] Review request for 8185970: Possible crash due to use-after-free

2017-08-08 Thread Murali Billa
  Hi, Please review the below fix. JIRA: https://bugs.openjdk.java.net/browse/JDK-8185970   Webrev: http://cr.openjdk.java.net/~mbilla/8185970/webrev.00/   Thanks, Murali

[10] Review request for 8182977: NullPointerException with HTMLEditor when changing the scene graph

2017-08-07 Thread Murali Billa
  Hi, Please review the below simple fix. JIRA: https://bugs.openjdk.java.net/browse/JDK-8182977   Webrev: http://cr.openjdk.java.net/~mbilla/8182977/webrev.00/   Thanks, Murali

[10] Review request for 8183928: [Linux] Remove Warnings [-Wunused-parameter]

2017-08-07 Thread Murali Billa
  Hi, Please review the below simple fix. JIRA: https://bugs.openjdk.java.net/browse/JDK-8183928   Webrev: http://cr.openjdk.java.net/~mbilla/8183928/webrev.00/   Thanks, Murali

[10] Review request for 8184453: [Win] DRT crashes for imported/blink/fast/shapes/crash-caused-by-dirtying-a-shape-while-computing-it-requires-a-long-filename-to-crash.html

2017-07-30 Thread Murali Billa
  Hi, Please review the below fix. JIRA: https://bugs.openjdk.java.net/browse/JDK-8184453 Webrev: http://cr.openjdk.java.net/~mbilla/8184453/webrev.00/ Thanks, Murali

[10] Review request for 8185138: Build hangs and fills up disk on Windows with VS 2013 or 2010

2017-07-25 Thread Murali Billa
  Hi Kevin, Arun, Please review the below simple fix. JIRA: https://bugs.openjdk.java.net/browse/JDK-8185138 Webrev: http://cr.openjdk.java.net/~mbilla/8185138/webrev.00/   Thanks, Murali

RE: CFV: New OpenJFX Committer: Ajit Ghaisas

2017-05-26 Thread Murali Billa
Vote: YES Kevin Rushforth wrote: > I hereby nominate Ajit Ghaisas [1] to OpenJFX Committer. > > Ajit is a member of JavaFX team at Oracle, who has contributed 10 > changesets to OpenJFX. A list of these changesets is available by the > following link: > > http://hg.openjdk.java.net/openjfx/10-d

[10] Review request for 8176729: com.sun.webkit.dom.NodeImpl#SelfDisposer is not called

2017-04-17 Thread Murali Billa
    Hi Kevin, Arun, Guru Please review the fix for ­ JDK-8176729. JIRA: https://bugs.openjdk.java.net/browse/JDK-8176729   Webrev: http://cr.openjdk.java.net/~mbilla/8176729/webrev.00/   Thanks, Murali

[9] Review request for 8174866: User Agent String for JavaFX is not correct

2017-02-13 Thread Murali Billa
  Hi Kevin, Arun, Guru Please review the simple fix for ­ JDK-8174866 JIRA: https://bugs.openjdk.java.net/browse/JDK-8174866   Webrev: http://cr.openjdk.java.net/~mbilla/8174866/webrev.00/   Thanks, Murali

RE: CFV: New OpenJFX Committer: Semyon Sadetsky

2017-01-25 Thread Murali Billa
Vote: YES 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.net/openjfx/9-dev/rt/log?rev=auth

RE: CFV: New OpenJFX Committer: Ramesh Gangadhar

2017-01-18 Thread Murali Billa
Vote: YES -Original Message- From: Kevin Rushforth Sent: Thursday, January 19, 2017 3:24 AM To: Ramesh Gangadhar; openjfx-dev@openjdk.java.net Subject: CFV: New OpenJFX Committer: Ramesh Gangadhar I hereby nominate Ramesh Gangadhar [1] to OpenJFX Committer. Ramesh is a member of JavaFX

[9] Review request for 8162922: JavaFx WebView canvas doesn't support dash within strokeRec

2017-01-17 Thread Murali Billa
  Hi Kevin, Arun, Guru, Please review the fix for ­ JDK-8162922. JIRA: https://bugs.openjdk.java.net/browse/JDK-8162922   Webrev: http://cr.openjdk.java.net/~mbilla/8162922/webrev.00/   Thanks, Murali

[9] Review request for 8172694: JSCallbackMemoryTest fails intermittently

2017-01-16 Thread Murali Billa
  Hi Kevin, Arun, Guru Please review the fix for ­ JDK-8172694 JIRA: https://bugs.openjdk.java.net/browse/JDK-8172694   Webrev: http://cr.openjdk.java.net/~mbilla/8172694/webrev.00/   Thanks, Murali

[9] Review request for 8164914: Validation of WebSQL Support in Webkit

2017-01-16 Thread Murali Billa
  Hi Kevin, Arun, Guru Please review the fix for JDK-8164914 JIRA: https://bugs.openjdk.java.net/browse/JDK-8164914   Webrev: http://cr.openjdk.java.net/~mbilla/8164914/webrev.02/   Thanks, Murali

[9] Review request for 8170610: Validation of JSObject Memory leak

2017-01-05 Thread Murali Billa
    Hi Kevin, Arun, Guru Please review the fix for JDK-8170610. JIRA: https://bugs.openjdk.java.net/browse/JDK-8170610     Webrev: http://cr.openjdk.java.net/~mbilla/8170610/webrev.00/   Thanks, Murali

[9] Review request for 8170938: Memory leak in JavaFX WebView

2016-12-09 Thread Murali Billa
  Hi Kevin, Arun, Guru Please review the simple fix for JDK-8170938. JIRA: https://bugs.openjdk.java.net/browse/JDK-8170938 Webrev: http://cr.openjdk.java.net/~mbilla/8170938/webrev.00/   Thanks, Murali

[9] Review request for 8170930: WebView unit test crashes in 32-bit mode

2016-12-08 Thread Murali Billa
  Hi Kevin, Please review the backout fix for JDK-8164792. JIRA: https://bugs.openjdk.java.net/browse/JDK-8170930 Webrev: http://cr.openjdk.java.net/~mbilla/8170930/webrev.00/   Thanks, Murali

[9] Review request for 8164792: Memory leak in JavaFX WebView

2016-11-25 Thread Murali Billa
  Hi Kevin, Arun, Guru, Please review the below leak fix. JIRA: https://bugs.openjdk.java.net/browse/JDK-8164792 Webrev: http://cr.openjdk.java.net/~mbilla/8164792/webrev.00/   Thanks, Murali

[9] Review request for 8169204: Need to document JSObject Call and setSlot APIs to use weak references

2016-11-04 Thread Murali Billa
Hi Kevin, Please review the below simple fix. JIRA: https://bugs.openjdk.java.net/browse/JDK-8169204 Webrev: http://cr.openjdk.java.net/~mbilla/8169204/webrev.00/   Thanks, Murali

[9] Review request for 8166677: HTMLEditor freezes after restoring previously maximized window

2016-10-06 Thread Murali Billa
Hi Kevin, David Please review the following fix. JIRA: https://bugs.openjdk.java.net/browse/JDK-8166677 Webrev: http://cr.openjdk.java.net/~mbilla/8166677/webrev.02/   Thanks, Murali

[9] Review request for 8166677: HTMLEditor freezes after restoring previously maximized window

2016-09-29 Thread Murali Billa
  Hi Kevin, Arun, Guru, Please review the following fix. JIRA: https://bugs.openjdk.java.net/browse/JDK-8166677 Webrev: http://cr.openjdk.java.net/~mbilla/8166677/webrev.00/   Thanks, Murali

[9] Review request for 8164914: Add Test Case for JDK-8162979

2016-08-30 Thread Murali Billa
  Hi Kevin, Arun, Guru, Please review the following fix. JIRA: https://bugs.openjdk.java.net/browse/JDK-8164914 Webrev: http://cr.openjdk.java.net/~mbilla/8164914/webrev.00/ Thanks, Murali

[9] Review request for 8162979: Website weibo.com cannot be loaded

2016-08-24 Thread Murali Billa
  Hi Kevin, Arun, Guru, Please review the following fix. JIRA: https://bugs.openjdk.java.net/browse/JDK-8162979 Webrev: http://cr.openjdk.java.net/~mbilla/8162979/webrev.00/   Thanks, Murali

[9] Review request for 8162977: General sibling selector is broken for selected input boxes in WebView

2016-08-03 Thread Murali Billa
    Hi Kevin, Arun, Guru, Please review the following fix. JIRA: https://bugs.openjdk.java.net/browse/JDK-8162977 Webrev: http://cr.openjdk.java.net/~mbilla/8162977/webrev.00/ Thanks, Murali

[9] Review request for 8090216: HTMLEditor: font bold doesn't work when an indent is set

2016-08-02 Thread Murali Billa
  Hi Kevin, Arun, Guru, Please review the following fix. JIRA: https://bugs.openjdk.java.net/browse/JDK-8090216 Webrev: http://cr.openjdk.java.net/~mbilla/8090216/webrev.00/ Thanks, Murali

RE: CFV: New OpenJFX Committer: Andrey Rusakov

2016-07-23 Thread Murali Billa
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 significant. > > Votes are due by August 6,

[9] Review request for 8161724: [Win] Exceptions while browsing

2016-07-19 Thread Murali Billa
Hi Kevin, Arun, Ankit Please review below fix. JIRA: https://bugs.openjdk.java.net/browse/JDK-8161724 Webrev:  http://cr.openjdk.java.net/~mbilla/8161724/webrev.00/   Thanks, Murali

[9] Review request for 8161699: [Win] Fix Compilation warnings

2016-07-19 Thread Murali Billa
Hi Kevin, Arun, Ankit Please review below fix. JIRA: https://bugs.openjdk.java.net/browse/JDK-8161699 Webrev:  http://cr.openjdk.java.net/~mbilla/8161699/webrev.00/   Thanks, Murali

RE: CFV: New OpenJFX Committer: Ankit Srivastava

2016-07-19 Thread Murali Billa
Vote: YES 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?rev=a.ank

[9] Review request for 8161053: Passing objects between JavaScript (JavaFX / WebKit) and Java causes a memory leak

2016-07-13 Thread Murali Billa
Hi Arun, Guru, Ankit Please review below fix. JIRA: https://bugs.openjdk.java.net/browse/JDK-8161053 Webrev: http://cr.openjdk.java.net/~mbilla/8161053/webrev.00/ Thanks, Murali

[9] Review request for 8160400: WebView can't alert from a timer

2016-06-28 Thread Murali Billa
Hi Kevin, Arun, Please review below fix: JBS : https://bugs.openjdk.java.net/browse/JDK-8160400 Webrev : http://cr.openjdk.java.net/~mbilla/8160400/webrev.00/ Thanks, Murali

[9] Review request for 8160388: Test Case Failure in CallBackTest

2016-06-27 Thread Murali Billa
Hi Kevin, Arun, Please review below fix: JBS : https://bugs.openjdk.java.net/browse/JDK-8160388 Webrev : http://cr.openjdk.java.net/~mbilla/8160388/webrev.00/ Thanks, Murali

[9] Review request for 8154127: Need to document that JavaScript to Java bindings use weak references

2016-06-27 Thread Murali Billa
Hi Kevin, Arun, Please review below fix: JBS : https://bugs.openjdk.java.net/browse/JDK-8154127 Webrev : http://cr.openjdk.java.net/~mbilla/8154127/webrev.00/ Thanks, Murali

[9] Review request for 8146211: WebView can't alert from a timer

2016-06-23 Thread Murali Billa
Hi Kevin, Arun, Please review below fix: JBS : https://bugs.openjdk.java.net/browse/JDK-8146211 Webrev : http://cr.openjdk.java.net/~mbilla/8146211/webrev.00/ Thanks, Murali

[9] Review request for 8150800: NullPointer exception in WebView

2016-05-09 Thread Murali Billa
Hi Kevin, Alexander Please review below fix. JBS : https://bugs.openjdk.java.net/browse/JDK-8150800 Webrev : http://cr.openjdk.java.net/~mbilla/8150800/webrev.00/ Thanks, Murali

RE: CFV: New OpenJFX Committer: Guru Hb

2016-05-02 Thread Murali Billa
VOTE: YES -- Murali On 4/28/16 8:16 AM, Kevin Rushforth wrote: > I hereby nominate Guru Hb [1] to OpenJFX Committer. > > Guru is a member of JavaFX team at Oracle working on WebKit, who has > contributed 10 changesets [5] to OpenJFX, at least 8 of which are > significant. > > Votes are due by

[9] Review request for 8152393: SQL Server Reporting Services in WebViews shows 401

2016-04-28 Thread Murali Billa
Hi Kevin, Alexander & Arun, Please review below fix. JBS : https://bugs.openjdk.java.net/browse/JDK-8152393 Webrev : http://cr.openjdk.java.net/~mbilla/8152393/webrev.00/ This issue is a regression of 8u40 fix for https://bugs.openjdk.java.net/browse/JDK-8093859 Rootcause and Fix updated in

RE: CFV: New OpenJFX Committer: Arunprasad Rajkumar

2016-04-21 Thread Murali Billa
Vote: YES -- Murali -Original Message- From: Kevin Rushforth Sent: Thursday, April 21, 2016 10:18 PM To: Arunprasad Rajkumar; openjfx-dev@openjdk.java.net Subject: CFV: New OpenJFX Committer: Arunprasad Rajkumar I hereby nominate Arunprasad Rajkumar [1] to OpenJFX Committer. Arunpras

[9] Review request for 8154161: Webkit: Conversion from string literal loses const qualifier

2016-04-14 Thread Murali Billa
Hi Kevin, Alexander & Arun, Can you please review below fix. JBS : https://bugs.openjdk.java.net/browse/JDK-8154161 Webrev : http://cr.openjdk.java.net/~mbilla/8154161/webrev.00/ Thanks, Murali

RE: Windows 7 WebView Text Rendering

2016-01-18 Thread Murali Billa
Hi Michael, Can you mention what is the resolution/scale under which you are loading the content in webview? Thanks, Murali -Original Message- From: Michael Ennen [mailto:mike.en...@gmail.com] Sent: Friday, January 15, 2016 10:52 AM To: openjfx-dev@openjdk.java.net Subject: Windows 7

[9] review request: JDK-8147410 Crash with long selector list

2016-01-14 Thread Murali Billa
Hi Alexander, kevin, Please review the fix for below issue : JIRA: https://bugs.openjdk.java.net/browse/JDK-8147410 Webrev: http://cr.openjdk.java.net/~mbilla/8147410/ Issue: This crash is webkit regression in 8u60 and fixed in JDK 9 version. Thanks, Murali

[9] review request: JDK-8090061 [WebView] replace "char*" with "const char*" where appropriate

2016-01-12 Thread Murali Billa
Hi Arun, Alexander, kevin, Please review the fix for below issue : JIRA: https://bugs.openjdk.java.net/browse/JDK-8090061 Webrev: http://cr.openjdk.java.net/~mbilla/8090061/webrev.00/ Issue: This issue is related to changing JNI functions signature(BackForwardListJava.cpp

[9] review request: JDK-8146715 Input fields accepting characters beyond maxlength attribute

2016-01-11 Thread Murali Billa
Hi Arun, Alexander, kevin, Please review the fix for below issue : JIRA: https://bugs.openjdk.java.net/browse/JDK-8146715 Webrev: http://cr.openjdk.java.net/~mbilla/8146715/webrev.00/ Issue: This issue is webkit regression from 8u60 and fixed with webkit patch http://trac.w

[9] review request: JDK-8146512 Table Borders are not drawn properly

2016-01-05 Thread Murali Billa
Hi Kevin, Alexander, Guru, Please review the fix for below issue : JIRA: https://bugs.openjdk.java.net/browse/JDK-8146512 Webrev: http://cr.openjdk.java.net/~arajkumar/murali/8146512/webrev.00/ Root Cause: Table Borders are not drawn properly in web sites like wikipedia. The

[9] review request: JDK-8145937 Not able to enter text and pop menu disappearing on Mouseover Menu

2015-12-21 Thread Murali Billa
Hi Kevin, Alexander, Please review the fix for below issue : JIRA: https://bugs.openjdk.java.net/browse/JDK-8145937 Webrev: http://cr.openjdk.java.net/~ghb/murali/8145937/webrev.00/ Root Cause: There is a regression with r157328 (http://trac.webkit.org/changeset/157328) from 8u

[9] review request: 8145682 topDocument() returns an incorrect reference for cached Documents

2015-12-18 Thread Murali Billa
Hi Kevin, Alexander, Please review the fix for below : JIRA: https://bugs.openjdk.java.net/browse/JDK-8145682 Webrev: http://cr.openjdk.java.net/~ghb/murali/8145682/webrev.00/ Root Cause: There is a regression with r162947 which changed the way the top document is determined, eff

RE: [9] review request: 8144472 [Webcore] DRT test fast/dom/HTMLInputElement/input-line-height.html fails

2015-12-04 Thread Murali Billa
line-height which is introduced in html.css. Thanks, Murali -Original Message- From: Victor D'yakov Sent: Thursday, December 03, 2015 11:21 PM To: Murali Billa; Alexander Zvegintsev; Kevin Rushforth; Guru Hb; Arunprasad Rajkumar Cc: openjfx-dev@openjdk.java.net Subject: Re: [9

[9] review request: 8144472 [Webcore] DRT test fast/dom/HTMLInputElement/input-line-height.html fails

2015-12-03 Thread Murali Billa
Hi Kevin, Alexander, Please review the fix related to DumpRenderTree CSS line-height issue. JIRA: https://bugs.openjdk.java.net/browse/JDK-8144472 Webrev: http://cr.openjdk.java.net/~ghb/murali/8144472/webrev.00/ Thanks, Murali