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.
[~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.