8151921: Improved page resolution jdk8u111-b04

Wed, 06 Jul 2016 20:07:09 -0700

author
bpatel
date
Wed, 06 Jul 2016 20:07:09 -0700
changeset 3258
fd9b6417c521
parent 3257
f74918365a72
child 3259
5710d574a99a

8151921: Improved page resolution
Reviewed-by: jjg, ksrini

src/share/classes/com/sun/tools/doclets/formats/html/markup/HtmlWriter.java file | annotate | diff | comparison | revisions
test/com/sun/javadoc/testJavascript/TestJavascript.java file | annotate | diff | comparison | revisions
     1.1 --- a/src/share/classes/com/sun/tools/doclets/formats/html/markup/HtmlWriter.java	Tue Jul 05 15:03:58 2016 -0700
     1.2 +++ b/src/share/classes/com/sun/tools/doclets/formats/html/markup/HtmlWriter.java	Wed Jul 06 20:07:09 2016 -0700
     1.3 @@ -380,11 +380,12 @@
     1.4          HtmlTree script = new HtmlTree(HtmlTag.SCRIPT);
     1.5          script.addAttr(HtmlAttr.TYPE, "text/javascript");
     1.6          String scriptCode = DocletConstants.NL +
     1.7 -                "    targetPage = \"\" + window.location.search;" + DocletConstants.NL +
     1.8 -                "    if (targetPage != \"\" && targetPage != \"undefined\")" + DocletConstants.NL +
     1.9 -                "        targetPage = targetPage.substring(1);" + DocletConstants.NL +
    1.10 -                "    if (targetPage.indexOf(\":\") != -1 || (targetPage != \"\" && !validURL(targetPage)))" + DocletConstants.NL +
    1.11 -                "        targetPage = \"undefined\";" + DocletConstants.NL +
    1.12 +                "    tmpTargetPage = \"\" + window.location.search;" + DocletConstants.NL +
    1.13 +                "    if (tmpTargetPage != \"\" && tmpTargetPage != \"undefined\")" + DocletConstants.NL +
    1.14 +                "        tmpTargetPage = tmpTargetPage.substring(1);" + DocletConstants.NL +
    1.15 +                "    if (tmpTargetPage.indexOf(\":\") != -1 || (tmpTargetPage != \"\" && !validURL(tmpTargetPage)))" + DocletConstants.NL +
    1.16 +                "        tmpTargetPage = \"undefined\";" + DocletConstants.NL +
    1.17 +                "    targetPage = tmpTargetPage;" + DocletConstants.NL +
    1.18                  "    function validURL(url) {" + DocletConstants.NL +
    1.19                  "        try {" + DocletConstants.NL +
    1.20                  "            url = decodeURIComponent(url);" + DocletConstants.NL +
     2.1 --- a/test/com/sun/javadoc/testJavascript/TestJavascript.java	Tue Jul 05 15:03:58 2016 -0700
     2.2 +++ b/test/com/sun/javadoc/testJavascript/TestJavascript.java	Wed Jul 06 20:07:09 2016 -0700
     2.3 @@ -23,7 +23,7 @@
     2.4  
     2.5  /*
     2.6   * @test
     2.7 - * @bug      4665566 4855876 7025314 8012375 8015997 8016328 8024756
     2.8 + * @bug      4665566 4855876 7025314 8012375 8015997 8016328 8024756 8151921
     2.9   * @summary  Verify that the output has the right javascript.
    2.10   * @author   jamieh
    2.11   * @library  ../lib/
    2.12 @@ -50,11 +50,12 @@
    2.13              "<a href=\"index.html?TestJavascript.html\" target=\"_top\">Frames</a>"},
    2.14          {BUG_ID + FS + "index.html",
    2.15              "<script type=\"text/javascript\">" + NL +
    2.16 -                        "    targetPage = \"\" + window.location.search;" + NL +
    2.17 -            "    if (targetPage != \"\" && targetPage != \"undefined\")" + NL +
    2.18 -            "        targetPage = targetPage.substring(1);" + NL +
    2.19 -            "    if (targetPage.indexOf(\":\") != -1 || (targetPage != \"\" && !validURL(targetPage)))" + NL +
    2.20 -            "        targetPage = \"undefined\";" + NL +
    2.21 +            "    tmpTargetPage = \"\" + window.location.search;" + NL +
    2.22 +            "    if (tmpTargetPage != \"\" && tmpTargetPage != \"undefined\")" + NL +
    2.23 +            "        tmpTargetPage = tmpTargetPage.substring(1);" + NL +
    2.24 +            "    if (tmpTargetPage.indexOf(\":\") != -1 || (tmpTargetPage != \"\" && !validURL(tmpTargetPage)))" + NL +
    2.25 +            "        tmpTargetPage = \"undefined\";" + NL +
    2.26 +            "    targetPage = tmpTargetPage;" + NL +
    2.27              "    function validURL(url) {" + NL +
    2.28              "        try {" + NL +
    2.29              "            url = decodeURIComponent(url);" + NL +

mercurial