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.
Updated webrev : http://cr.openjdk.java.net/~ghb/8178360/webrev.08/
Updated patch : http://cr.openjdk.java.net/~ghb/8178360/rt.patch_08.tar.gz
[~arajkumar] I have fixed all the above review comments, Below are the one with description.
MiscellaneousTest.java : It was a typo mistake and its corrected.
modules/javafx.web/src/main/native/Source/ThirdParty/icu/CMakeLists.txt : Except ICU specific declaration , others were not reqired and its removed now.
ThirdParty/icu/java/data/CMakeLists.txt : Corrected now with set_target_properties(${ICU_JAVA_ICU_DATA_LIB_NAME} PROPERTIES IMPORTED_LOCATION ${ICU_JAVA_DATA_LIB}).
03-07-2017
Looks good.
+1
29-06-2017
Webrev : http://cr.openjdk.java.net/~ghb/8178360/webrev.07/ (Only modified source)
Patch : http://cr.openjdk.java.net/~ghb/8178360/rt.patch_07.tar.gz (Complete change set for testing)
ICU Source used : http://download.icu-project.org/files/icu4c/51.2/icu4c-51_2-src.tgz