JDK-8164914 : Validation of WebSQL Support in Webkit
  • Type: Bug
  • Component: javafx
  • Sub-Component: web
  • Affected Version: 9
  • Priority: P4
  • Status: Resolved
  • Resolution: Fixed
  • Submitted: 2016-08-28
  • Updated: 2017-01-17
  • Resolved: 2017-01-17
The Version table provides details related to the release that this issue/RFE will be addressed.

Unresolved : Release in which this issue/RFE will be addressed.
Resolved: Release in which this issue/RFE has been resolved.
Fixed : Release in which this issue/RFE has been fixed. The release containing this fix may be available for download as an Early Access Release or a General Availability Release.

To download the current JDK release, click here.
JDK 8 JDK 9
8u152Fixed 9Fixed
Related Reports
Relates :  
Comments
Changeset: aba9d077a391 Author: mbilla Date: 2017-01-17 19:56 +0530 URL: http://hg.openjdk.java.net/openjfx/9-dev/rt/rev/aba9d077a391
17-01-2017

[~arajkumar] thanks for the comments.. [~kcr] Regarding your earlier query, This new test checks for the specific reason we failed to redirect -- the partially implemented Websql -- rather than testing the actual ability to redirect in the weibo.com case. It is fine as far as it goes, but would it also be possible to also add a test case that actually tests that the redirect works? -> I think it is good to add the above use case as part of reliability suite as we are not testing live websites as part of unit test cases (since it involves VPN settings, network speed etc..)
17-01-2017

Thanks for the explanation. +1
17-01-2017

+1 with nit: + //This test case will be removed once we implement Websql feature. Give a space between // and comment statement. e.g. // This test case will be removed once we implement WebSQL feature
17-01-2017

webrev: http://cr.openjdk.java.net/~mbilla/8164914/webrev.03/ Note: This issue is test bug and risk free. Also good to have this test case after webkit upgrade to catch regressions.
16-01-2017

The Rampdown phase start [1] milestone is approaching, it means we are not targeting "9" to fix P4s anymore, unless the fix is ready and not risky, safe enough do not introduce P2-P3 regressions; if this is a case you can retarget back to "9" right before the push the reviewed code to repo. [1] http://openjdk.java.net/projects/jdk8/milestones#Rampdown_start
01-09-2016

testWebKitDatabaseUndefined / testWebSQLUndefined instead of testWebSQLSupport.
01-09-2016

Also, the expect and actual values in the assertEquals call are backwards.
30-08-2016

Please add a comment to this test indicating that it should be removed when Websql is implemented. This new test checks for the specific reason we failed to redirect -- the partially implemented Websql -- rather than testing the actual ability to redirect in the weibo.com case. It is fine as far as it goes, but would it also be possible to also add a test case that actually tests that the redirect works?
30-08-2016

webrev: http://cr.openjdk.java.net/~mbilla/8164914/webrev.00/ Tested in windows 7, 64 bit with JDK 9.
30-08-2016