From: <asa...@us...> - 2017-06-07 06:47:11
|
Revision: 14567 http://sourceforge.net/p/htmlunit/code/14567 Author: asashour Date: 2017-06-07 06:47:08 +0000 (Wed, 07 Jun 2017) Log Message: ----------- Remove old comments Modified Paths: -------------- trunk/htmlunit/src/main/java/com/gargoylesoftware/htmlunit/BrowserVersion.java trunk/htmlunit/src/main/java/com/gargoylesoftware/htmlunit/WebClient.java trunk/htmlunit/src/main/java/com/gargoylesoftware/htmlunit/WebClientOptions.java trunk/htmlunit/src/main/java/com/gargoylesoftware/htmlunit/attachment/AttachmentHandler.java trunk/htmlunit/src/main/java/com/gargoylesoftware/htmlunit/html/HtmlCheckBoxInput.java trunk/htmlunit/src/main/java/com/gargoylesoftware/htmlunit/html/HtmlPage.java trunk/htmlunit/src/main/java/com/gargoylesoftware/htmlunit/html/HtmlRadioButtonInput.java trunk/htmlunit/src/main/java/com/gargoylesoftware/htmlunit/javascript/host/html/HTMLDocument.java trunk/htmlunit/src/test/java/com/gargoylesoftware/htmlunit/WebClient3Test.java trunk/htmlunit/src/test/java/com/gargoylesoftware/htmlunit/html/HTMLParser4Test.java trunk/htmlunit/src/test/java/com/gargoylesoftware/htmlunit/html/HtmlFrame2Test.java trunk/htmlunit/src/test/java/com/gargoylesoftware/htmlunit/javascript/JavaScriptEngine2Test.java trunk/htmlunit/src/test/java/com/gargoylesoftware/htmlunit/javascript/JavaScriptEngineTest.java trunk/htmlunit/src/test/java/com/gargoylesoftware/htmlunit/javascript/NativeFunctionTest.java trunk/htmlunit/src/test/java/com/gargoylesoftware/htmlunit/javascript/host/Window2Test.java trunk/htmlunit/src/test/java/com/gargoylesoftware/htmlunit/javascript/host/Window3Test.java trunk/htmlunit/src/test/java/com/gargoylesoftware/htmlunit/javascript/host/WindowTest.java trunk/htmlunit/src/test/java/com/gargoylesoftware/htmlunit/javascript/host/event/Event2Test.java trunk/htmlunit/src/test/java/com/gargoylesoftware/htmlunit/javascript/host/event/EventTest.java trunk/htmlunit/src/test/java/com/gargoylesoftware/htmlunit/javascript/host/html/HTMLDocumentWrite2Test.java trunk/htmlunit/src/test/java/com/gargoylesoftware/htmlunit/javascript/host/html/HTMLElementTest.java trunk/htmlunit/src/test/java/com/gargoylesoftware/htmlunit/javascript/host/html/HTMLFormElementTest.java trunk/htmlunit/src/test/java/com/gargoylesoftware/htmlunit/javascript/host/html/HTMLHtmlElementTest.java trunk/htmlunit/src/test/java/com/gargoylesoftware/htmlunit/javascript/host/html/HTMLInputElementTest.java trunk/htmlunit/src/test/java/com/gargoylesoftware/htmlunit/javascript/host/xml/XMLHttpRequestTest.java trunk/htmlunit/src/test/java/com/gargoylesoftware/htmlunit/javascript/regexp/mozilla/MozillaTestGenerator.java trunk/htmlunit/src/test/java/com/gargoylesoftware/htmlunit/util/DebuggingWebConnectionTest.java Modified: trunk/htmlunit/src/main/java/com/gargoylesoftware/htmlunit/BrowserVersion.java =================================================================== --- trunk/htmlunit/src/main/java/com/gargoylesoftware/htmlunit/BrowserVersion.java 2017-06-07 06:31:04 UTC (rev 14566) +++ trunk/htmlunit/src/main/java/com/gargoylesoftware/htmlunit/BrowserVersion.java 2017-06-07 06:47:08 UTC (rev 14567) @@ -410,7 +410,7 @@ * @param userAgent the user agent string that will be sent to the server * @param javaScriptVersion the version of JavaScript * @param browserVersionNumeric the floating number version of the browser - * @param nickname the short name of the browser (like "FF3", "IE6", ...) + * @param nickname the short name of the browser (like "FF52", "IE", ...) * @param features the browser features */ private BrowserVersion(final String applicationName, final String applicationVersion, Modified: trunk/htmlunit/src/main/java/com/gargoylesoftware/htmlunit/WebClient.java =================================================================== --- trunk/htmlunit/src/main/java/com/gargoylesoftware/htmlunit/WebClient.java 2017-06-07 06:31:04 UTC (rev 14566) +++ trunk/htmlunit/src/main/java/com/gargoylesoftware/htmlunit/WebClient.java 2017-06-07 06:47:08 UTC (rev 14567) @@ -1526,7 +1526,7 @@ /** * Returns the current refresh handler. - * The default refresh handler is a {@link NiceRefreshHandler NiceRefreshHandler(2)} (since HtmlUnit-2.12). + * The default refresh handler is a {@link NiceRefreshHandler NiceRefreshHandler(2)}. * @return the current RefreshHandler */ public RefreshHandler getRefreshHandler() { Modified: trunk/htmlunit/src/main/java/com/gargoylesoftware/htmlunit/WebClientOptions.java =================================================================== --- trunk/htmlunit/src/main/java/com/gargoylesoftware/htmlunit/WebClientOptions.java 2017-06-07 06:31:04 UTC (rev 14566) +++ trunk/htmlunit/src/main/java/com/gargoylesoftware/htmlunit/WebClientOptions.java 2017-06-07 06:47:08 UTC (rev 14567) @@ -244,10 +244,9 @@ /** * Enables/disables Applet support. By default, this property is disabled.<br> * <p> - * Note: as of HtmlUnit-2.4, Applet support is experimental and minimal + * Note: Applet support is experimental and minimal * </p> * @param enabled {@code true} to enable Applet support - * @since HtmlUnit-2.4 */ public void setAppletEnabled(final boolean enabled) { appletEnabled_ = enabled; @@ -447,7 +446,7 @@ /** * Gets the timeout value for the {@link WebConnection}. - * The default timeout is 90 seconds (it was 0 up to HtmlUnit-2.11). + * The default timeout is 90 seconds. * @return the timeout value in milliseconds * @see WebClientOptions#setTimeout(int) */ Modified: trunk/htmlunit/src/main/java/com/gargoylesoftware/htmlunit/attachment/AttachmentHandler.java =================================================================== --- trunk/htmlunit/src/main/java/com/gargoylesoftware/htmlunit/attachment/AttachmentHandler.java 2017-06-07 06:31:04 UTC (rev 14566) +++ trunk/htmlunit/src/main/java/com/gargoylesoftware/htmlunit/attachment/AttachmentHandler.java 2017-06-07 06:47:08 UTC (rev 14567) @@ -29,8 +29,7 @@ * {@link com.gargoylesoftware.htmlunit.WebClient#setAttachmentHandler(AttachmentHandler)}. When * no attachment handler has been registered with the <tt>WebClient</tt>, the semantics described * above to not apply, and attachments are loaded inline. By default, <tt>AttachmentHandler</tt>s - * are not registered with new <tt>WebClient</tt> instances, in order to maintain backwards - * compatibility with HtmlUnit 2.1 and earlier. This will likely change in the future.</p> + * are not registered with new <tt>WebClient</tt> instances.</p> * * @author Bruce Chapman * @author Sudhan Moghe Modified: trunk/htmlunit/src/main/java/com/gargoylesoftware/htmlunit/html/HtmlCheckBoxInput.java =================================================================== --- trunk/htmlunit/src/main/java/com/gargoylesoftware/htmlunit/html/HtmlCheckBoxInput.java 2017-06-07 06:31:04 UTC (rev 14566) +++ trunk/htmlunit/src/main/java/com/gargoylesoftware/htmlunit/html/HtmlCheckBoxInput.java 2017-06-07 06:47:08 UTC (rev 14567) @@ -51,7 +51,7 @@ /** * Creates an instance. - * If no value is specified, it is set to "on" as browsers do (e.g. IE6 and Mozilla 1.7) + * If no value is specified, it is set to "on" as browsers do * even if spec says that it is not allowed * (<a href="http://www.w3.org/TR/REC-html40/interact/forms.html#adef-value-INPUT">W3C</a>). * @@ -61,7 +61,6 @@ */ HtmlCheckBoxInput(final String qualifiedName, final SgmlPage page, final Map<String, DomAttr> attributes) { - // default value for both IE6 and Mozilla 1.7 even if spec says it is unspecified super(qualifiedName, page, addValueIfNeeded(page, attributes)); // fix the default value in case we have set it @@ -167,8 +166,7 @@ } /** - * Both IE and Mozilla will first update the internal state of checkbox - * and then handle "onclick" event. + * First update the internal state of checkbox and then handle "onclick" event. * {@inheritDoc} */ @Override Modified: trunk/htmlunit/src/main/java/com/gargoylesoftware/htmlunit/html/HtmlPage.java =================================================================== --- trunk/htmlunit/src/main/java/com/gargoylesoftware/htmlunit/html/HtmlPage.java 2017-06-07 06:31:04 UTC (rev 14566) +++ trunk/htmlunit/src/main/java/com/gargoylesoftware/htmlunit/html/HtmlPage.java 2017-06-07 06:47:08 UTC (rev 14567) @@ -853,7 +853,7 @@ * * The HTML specification seems to indicate that one accesskey cannot be used * for multiple elements however Internet Explorer does seem to support this. - * It's worth noting that Mozilla does not support multiple elements with one + * It's worth noting that Firefox does not support multiple elements with one * access key so you are making your HTML browser specific if you rely on this * feature.<p> * Modified: trunk/htmlunit/src/main/java/com/gargoylesoftware/htmlunit/html/HtmlRadioButtonInput.java =================================================================== --- trunk/htmlunit/src/main/java/com/gargoylesoftware/htmlunit/html/HtmlRadioButtonInput.java 2017-06-07 06:31:04 UTC (rev 14566) +++ trunk/htmlunit/src/main/java/com/gargoylesoftware/htmlunit/html/HtmlRadioButtonInput.java 2017-06-07 06:47:08 UTC (rev 14567) @@ -53,7 +53,7 @@ /** * Creates an instance. - * If no value is specified, it is set to "on" as browsers do (eg IE6 and Mozilla 1.7) + * If no value is specified, it is set to "on" as browsers do * even if spec says that it is not allowed * (<a href="http://www.w3.org/TR/REC-html40/interact/forms.html#adef-value-INPUT">W3C</a>). * @@ -63,7 +63,6 @@ */ HtmlRadioButtonInput(final String qualifiedName, final SgmlPage page, final Map<String, DomAttr> attributes) { - // default value for both IE6 and Mozilla 1.7 even if spec says it is unspecified super(qualifiedName, page, addValueIfNeeded(page, attributes)); // fix the default value in case we have set it Modified: trunk/htmlunit/src/main/java/com/gargoylesoftware/htmlunit/javascript/host/html/HTMLDocument.java =================================================================== --- trunk/htmlunit/src/main/java/com/gargoylesoftware/htmlunit/javascript/host/html/HTMLDocument.java 2017-06-07 06:31:04 UTC (rev 14566) +++ trunk/htmlunit/src/main/java/com/gargoylesoftware/htmlunit/javascript/host/html/HTMLDocument.java 2017-06-07 06:47:08 UTC (rev 14567) @@ -197,7 +197,6 @@ /** * JavaScript function "write" may accept a variable number of arguments. - * It's not documented by W3C, Mozilla or MSDN but works with Mozilla and IE. * @param context the JavaScript context * @param thisObj the scriptable * @param args the arguments passed into the method @@ -226,7 +225,6 @@ /** * JavaScript function "writeln" may accept a variable number of arguments. - * It's not documented by W3C, Mozilla or MSDN but works with Mozilla and IE. * @param context the JavaScript context * @param thisObj the scriptable * @param args the arguments passed into the method Modified: trunk/htmlunit/src/test/java/com/gargoylesoftware/htmlunit/WebClient3Test.java =================================================================== --- trunk/htmlunit/src/test/java/com/gargoylesoftware/htmlunit/WebClient3Test.java 2017-06-07 06:31:04 UTC (rev 14566) +++ trunk/htmlunit/src/test/java/com/gargoylesoftware/htmlunit/WebClient3Test.java 2017-06-07 06:47:08 UTC (rev 14567) @@ -115,8 +115,7 @@ } /** - * Was causing a "java.net.URISyntaxException: Malformed escape pair" - * as of HtmlUnit-2.12-SNAPSHOT on Nov. 29, 2012. + * Was causing a "java.net.URISyntaxException: Malformed escape pair". * HtmlUnit now escapes the "%%" to "%25%25" to build a valid URL but FF doesn't care * and sends the invalid "%%" sequence as it. * This will be quite difficult to simulate FF here as HttpClient's HttpRequestBase @@ -135,8 +134,7 @@ } /** - * Was causing a "java.net.URISyntaxException: Malformed escape pair" - * as of HtmlUnit-2.12-SNAPSHOT on Nov. 29, 2012. + * Was causing a "java.net.URISyntaxException: Malformed escape pair". * This is a simplified version of {@link #escapeRequestQuery2a()} only testing * that no exception is thrown. The request performed is not fully correct. * This test can be removed once {@link #escapeRequestQuery2a()} runs correctly. @@ -402,7 +400,6 @@ /** * Test "deflate" encoding without ZLIB header and checksum fields. - * This was failing as of HtmlUnit-2.10. * @throws Exception if the test fails */ @Test Modified: trunk/htmlunit/src/test/java/com/gargoylesoftware/htmlunit/html/HTMLParser4Test.java =================================================================== --- trunk/htmlunit/src/test/java/com/gargoylesoftware/htmlunit/html/HTMLParser4Test.java 2017-06-07 06:31:04 UTC (rev 14566) +++ trunk/htmlunit/src/test/java/com/gargoylesoftware/htmlunit/html/HTMLParser4Test.java 2017-06-07 06:47:08 UTC (rev 14567) @@ -601,7 +601,6 @@ /** * Test for a case where complete HTML page is set in innerHTML of HTML tag. - * IE8 throws JavaScript error as innerHTML of HTML is read only. * Others replace the current content of the HTML node by the new one. * * @throws Exception failure Modified: trunk/htmlunit/src/test/java/com/gargoylesoftware/htmlunit/html/HtmlFrame2Test.java =================================================================== --- trunk/htmlunit/src/test/java/com/gargoylesoftware/htmlunit/html/HtmlFrame2Test.java 2017-06-07 06:31:04 UTC (rev 14566) +++ trunk/htmlunit/src/test/java/com/gargoylesoftware/htmlunit/html/HtmlFrame2Test.java 2017-06-07 06:47:08 UTC (rev 14567) @@ -238,7 +238,6 @@ } /** - * Was failing as of HtmlUnit-2.11. * @see <a href="sourceforge.net/p/htmlunit/bugs/1443/">Bug #1443</a> * @throws Exception if the test fails */ Modified: trunk/htmlunit/src/test/java/com/gargoylesoftware/htmlunit/javascript/JavaScriptEngine2Test.java =================================================================== --- trunk/htmlunit/src/test/java/com/gargoylesoftware/htmlunit/javascript/JavaScriptEngine2Test.java 2017-06-07 06:31:04 UTC (rev 14566) +++ trunk/htmlunit/src/test/java/com/gargoylesoftware/htmlunit/javascript/JavaScriptEngine2Test.java 2017-06-07 06:47:08 UTC (rev 14567) @@ -48,11 +48,7 @@ public class JavaScriptEngine2Test extends WebDriverTestCase { /** - * All browsers except Opera seem to have a single JS execution thread for all windows, - * but it is not the case of HtmlUnit-2.6! - * Tested browsers (21.09.09): - * - single JS thread: FF2, FF3.1, FF3.5, IE6 (and for info: Konqueror 4.2.2, Chrome Linux dev build) - * - multiple JS threads: none of HtmlUnit's simulated browsers (for info Opera 10.00) + * All browsers except Opera seem to have a single JS execution thread for all windows. * @throws Exception if the test fails */ @Test Modified: trunk/htmlunit/src/test/java/com/gargoylesoftware/htmlunit/javascript/JavaScriptEngineTest.java =================================================================== --- trunk/htmlunit/src/test/java/com/gargoylesoftware/htmlunit/javascript/JavaScriptEngineTest.java 2017-06-07 06:31:04 UTC (rev 14566) +++ trunk/htmlunit/src/test/java/com/gargoylesoftware/htmlunit/javascript/JavaScriptEngineTest.java 2017-06-07 06:47:08 UTC (rev 14567) @@ -169,9 +169,6 @@ /** * If a reference has been hold on a page and the page is not * anymore the one contained in "its" window, JavaScript should not be executed. - * Up to HtmlUnit-2.10 it could work a little bit without any guarantee but this - * has been removed to fix a problem where background JS tasks were still executed - * once a page has been unloaded * (see {@link com.gargoylesoftware.htmlunit.javascript.host.WindowConcurrencyTest#cleanSetTimeout}). * @throws Exception if the test fails */ @@ -242,9 +239,7 @@ /** * An odd case, if two external scripts are referenced and the <script> element * of the first contain a comment which contain an apostrophe, then the second script - * is ignored. - * https://sourceforge.net/tracker/?func=detail&atid=448266&aid=1990198&group_id=47038 - * This works fine in IE6 and FF 2.0. Remove the apostrophe from "shouldn't" to make it work here. + * is ignored. Bug #632. * @throws Exception if the test fails */ @Test Modified: trunk/htmlunit/src/test/java/com/gargoylesoftware/htmlunit/javascript/NativeFunctionTest.java =================================================================== --- trunk/htmlunit/src/test/java/com/gargoylesoftware/htmlunit/javascript/NativeFunctionTest.java 2017-06-07 06:31:04 UTC (rev 14566) +++ trunk/htmlunit/src/test/java/com/gargoylesoftware/htmlunit/javascript/NativeFunctionTest.java 2017-06-07 06:47:08 UTC (rev 14567) @@ -144,7 +144,7 @@ } /** - * Function properties "arguments" and "caller" were wrongly enumerated as of HtmlUnit-2.11. + * Function properties "arguments" and "caller" were wrongly enumerated. * @throws Exception if the test fails */ @Test Modified: trunk/htmlunit/src/test/java/com/gargoylesoftware/htmlunit/javascript/host/Window2Test.java =================================================================== --- trunk/htmlunit/src/test/java/com/gargoylesoftware/htmlunit/javascript/host/Window2Test.java 2017-06-07 06:31:04 UTC (rev 14566) +++ trunk/htmlunit/src/test/java/com/gargoylesoftware/htmlunit/javascript/host/Window2Test.java 2017-06-07 06:47:08 UTC (rev 14567) @@ -1130,8 +1130,6 @@ } /** - * This was causing HtmlUnit to hang as of HtmlUnit-2.12 snapshot from 24.01.2013 and probably since a very long - * time. * The reason was that "top" evaluate to WindowProxy and "Object(top)" was setting the top scope as parentScope * of the WindowProxy which was setting it on the Window where it should always be null. * @throws Exception if the test fails @@ -1186,7 +1184,7 @@ } /** - * As of 2.12-SNAPSHOT on 19.02.2013, a task started by setTimeout in an event handler could be executed before + * As of 19.02.2013, a task started by setTimeout in an event handler could be executed before * all events handlers have been executed due to a missing synchronization. * @throws Exception if the test fails */ Modified: trunk/htmlunit/src/test/java/com/gargoylesoftware/htmlunit/javascript/host/Window3Test.java =================================================================== --- trunk/htmlunit/src/test/java/com/gargoylesoftware/htmlunit/javascript/host/Window3Test.java 2017-06-07 06:31:04 UTC (rev 14566) +++ trunk/htmlunit/src/test/java/com/gargoylesoftware/htmlunit/javascript/host/Window3Test.java 2017-06-07 06:47:08 UTC (rev 14567) @@ -1065,12 +1065,12 @@ } /** - * Tests Mozilla viewport properties. + * Tests viewport properties. * @throws Exception if the test fails */ @Test @Alerts({"number", "number", "number", "number"}) - public void mozillaViewport() throws Exception { + public void viewport() throws Exception { final String html = HtmlPageTest.STANDARDS_MODE_PREFIX_ + "<html>\n" + "<head></head>\n" Modified: trunk/htmlunit/src/test/java/com/gargoylesoftware/htmlunit/javascript/host/WindowTest.java =================================================================== --- trunk/htmlunit/src/test/java/com/gargoylesoftware/htmlunit/javascript/host/WindowTest.java 2017-06-07 06:31:04 UTC (rev 14566) +++ trunk/htmlunit/src/test/java/com/gargoylesoftware/htmlunit/javascript/host/WindowTest.java 2017-06-07 06:47:08 UTC (rev 14567) @@ -1451,7 +1451,7 @@ */ @Test @Alerts({"10", "20", "30", "40"}) - public void mozillaViewportSetters() throws Exception { + public void viewportSetters() throws Exception { final String html = "<html>\n" + "<head></head>\n" + "<body>\n" Modified: trunk/htmlunit/src/test/java/com/gargoylesoftware/htmlunit/javascript/host/event/Event2Test.java =================================================================== --- trunk/htmlunit/src/test/java/com/gargoylesoftware/htmlunit/javascript/host/event/Event2Test.java 2017-06-07 06:31:04 UTC (rev 14566) +++ trunk/htmlunit/src/test/java/com/gargoylesoftware/htmlunit/javascript/host/event/Event2Test.java 2017-06-07 06:47:08 UTC (rev 14567) @@ -516,7 +516,7 @@ } /** - * Test Mozilla DOMContentLoaded event. + * Test DOMContentLoaded event. * @throws Exception if the test fails */ @Test Modified: trunk/htmlunit/src/test/java/com/gargoylesoftware/htmlunit/javascript/host/event/EventTest.java =================================================================== --- trunk/htmlunit/src/test/java/com/gargoylesoftware/htmlunit/javascript/host/event/EventTest.java 2017-06-07 06:31:04 UTC (rev 14566) +++ trunk/htmlunit/src/test/java/com/gargoylesoftware/htmlunit/javascript/host/event/EventTest.java 2017-06-07 06:47:08 UTC (rev 14567) @@ -945,7 +945,6 @@ /** * Test that the function open resolves to document.open within a handler defined by an attribute. - * This was wrong (even in unit tests) up to HtmlUnit-2.12. * @throws Exception if the test fails */ @Test Modified: trunk/htmlunit/src/test/java/com/gargoylesoftware/htmlunit/javascript/host/html/HTMLDocumentWrite2Test.java =================================================================== --- trunk/htmlunit/src/test/java/com/gargoylesoftware/htmlunit/javascript/host/html/HTMLDocumentWrite2Test.java 2017-06-07 06:31:04 UTC (rev 14566) +++ trunk/htmlunit/src/test/java/com/gargoylesoftware/htmlunit/javascript/host/html/HTMLDocumentWrite2Test.java 2017-06-07 06:47:08 UTC (rev 14567) @@ -337,8 +337,8 @@ } /** - * This was causing a StackOverflowError in HtmlUnit-2.10-SNAPSHOT when simulating IE as of 14.10.2011 - * and probably in release 2.9 as well. + * This was causing a StackOverflowError. + * * @throws Exception if an error occurs */ @Test Modified: trunk/htmlunit/src/test/java/com/gargoylesoftware/htmlunit/javascript/host/html/HTMLElementTest.java =================================================================== --- trunk/htmlunit/src/test/java/com/gargoylesoftware/htmlunit/javascript/host/html/HTMLElementTest.java 2017-06-07 06:31:04 UTC (rev 14566) +++ trunk/htmlunit/src/test/java/com/gargoylesoftware/htmlunit/javascript/host/html/HTMLElementTest.java 2017-06-07 06:47:08 UTC (rev 14567) @@ -3146,7 +3146,7 @@ + "<script>\n" + "function test() {\n" + " var div1 = document.getElementById('div1');\n" - + " var defined = typeof(div1.filters) != 'undefined';\n" // "unknown" for IE6!!! + + " var defined = typeof(div1.filters) != 'undefined';\n" + " alert(defined ? 'defined' : 'undefined');\n" + "}\n" + "</script></head><body onload='test()'>\n" Modified: trunk/htmlunit/src/test/java/com/gargoylesoftware/htmlunit/javascript/host/html/HTMLFormElementTest.java =================================================================== --- trunk/htmlunit/src/test/java/com/gargoylesoftware/htmlunit/javascript/host/html/HTMLFormElementTest.java 2017-06-07 06:31:04 UTC (rev 14566) +++ trunk/htmlunit/src/test/java/com/gargoylesoftware/htmlunit/javascript/host/html/HTMLFormElementTest.java 2017-06-07 06:47:08 UTC (rev 14567) @@ -835,7 +835,6 @@ getMockWebConnection().setDefaultResponse(""); final WebDriver driver = loadPageWithAlerts2(html); driver.findElement(By.id("x")).click(); - // caution: IE7 doesn't put a trailing "?" assertEquals(URL_FIRST + expectedFile, driver.getCurrentUrl().replaceAll("\\?", "")); } @@ -1136,7 +1135,6 @@ /** * Ensure that Multipart form text fields are correctly encoded. - * This was a regression introduced in 2.12-SNAPSHOT after upgrading to HttpClient to 4.3. * @throws Exception if the test fails */ @Test Modified: trunk/htmlunit/src/test/java/com/gargoylesoftware/htmlunit/javascript/host/html/HTMLHtmlElementTest.java =================================================================== --- trunk/htmlunit/src/test/java/com/gargoylesoftware/htmlunit/javascript/host/html/HTMLHtmlElementTest.java 2017-06-07 06:31:04 UTC (rev 14566) +++ trunk/htmlunit/src/test/java/com/gargoylesoftware/htmlunit/javascript/host/html/HTMLHtmlElementTest.java 2017-06-07 06:47:08 UTC (rev 14567) @@ -120,7 +120,7 @@ @Alerts({"true", "true", "true", "true"}) public void clientWidth() throws Exception { final String html = "<!DOCTYPE HTML PUBLIC '-//W3C//DTD HTML 4.01 Transitional//EN'" - + " 'http://www.w3.org/TR/html4/loose.dtd'>" // important for IE6!!! + + " 'http://www.w3.org/TR/html4/loose.dtd'>" + "<html><head>\n" + "<script>\n" + "function test() {\n" Modified: trunk/htmlunit/src/test/java/com/gargoylesoftware/htmlunit/javascript/host/html/HTMLInputElementTest.java =================================================================== --- trunk/htmlunit/src/test/java/com/gargoylesoftware/htmlunit/javascript/host/html/HTMLInputElementTest.java 2017-06-07 06:31:04 UTC (rev 14566) +++ trunk/htmlunit/src/test/java/com/gargoylesoftware/htmlunit/javascript/host/html/HTMLInputElementTest.java 2017-06-07 06:47:08 UTC (rev 14567) @@ -875,7 +875,6 @@ /** * Inputs have properties not only from there own type. - * Works with Mozilla, Firefox and IE... but not with HtmlUnit now. * @throws Exception if the test fails */ @Test Modified: trunk/htmlunit/src/test/java/com/gargoylesoftware/htmlunit/javascript/host/xml/XMLHttpRequestTest.java =================================================================== --- trunk/htmlunit/src/test/java/com/gargoylesoftware/htmlunit/javascript/host/xml/XMLHttpRequestTest.java 2017-06-07 06:31:04 UTC (rev 14566) +++ trunk/htmlunit/src/test/java/com/gargoylesoftware/htmlunit/javascript/host/xml/XMLHttpRequestTest.java 2017-06-07 06:47:08 UTC (rev 14567) @@ -108,7 +108,7 @@ } /** - * Tests Mozilla and IE style object creation. + * Tests style object creation. * @throws Exception if the test fails */ @Test Modified: trunk/htmlunit/src/test/java/com/gargoylesoftware/htmlunit/javascript/regexp/mozilla/MozillaTestGenerator.java =================================================================== --- trunk/htmlunit/src/test/java/com/gargoylesoftware/htmlunit/javascript/regexp/mozilla/MozillaTestGenerator.java 2017-06-07 06:31:04 UTC (rev 14566) +++ trunk/htmlunit/src/test/java/com/gargoylesoftware/htmlunit/javascript/regexp/mozilla/MozillaTestGenerator.java 2017-06-07 06:47:08 UTC (rev 14567) @@ -23,7 +23,7 @@ import org.apache.commons.io.FileUtils; /** - * Utility for automatically transforming Moalla JavaScript tests to WebDriver test cases. + * Utility for automatically transforming Mozilla JavaScript tests to WebDriver test cases. * * @author Ahmed Ashour */ Modified: trunk/htmlunit/src/test/java/com/gargoylesoftware/htmlunit/util/DebuggingWebConnectionTest.java =================================================================== --- trunk/htmlunit/src/test/java/com/gargoylesoftware/htmlunit/util/DebuggingWebConnectionTest.java 2017-06-07 06:31:04 UTC (rev 14566) +++ trunk/htmlunit/src/test/java/com/gargoylesoftware/htmlunit/util/DebuggingWebConnectionTest.java 2017-06-07 06:47:08 UTC (rev 14567) @@ -91,7 +91,7 @@ /** * Ensures that Content-Encoding headers are removed when JavaScript is uncompressed. - * (was causing java.io.IOException: Not in GZIP format as of HtmlUnit-2.10). + * (was causing java.io.IOException: Not in GZIP format). * @throws Exception if the test fails */ @Test |