svn commit: r1529618 - in /jmeter/trunk: bin/jmeter.properties src/protocol/http/org/apache/jmeter/protocol/http/parser/HTMLParser.java

[pm...@apache.org]

Author: pmouawad
Date: Sun Oct  6 13:34:37 2013
New Revision: 1529618

URL: http://svn.apache.org/r1529618
Log:
Bug 55632 - Have a new implementation of htmlParser for embedded resources parsing with better performances
Rollback default for now
Comment on performances
Bugzilla Id: 55632

Modified:
    jmeter/trunk/bin/jmeter.properties
    jmeter/trunk/src/protocol/http/org/apache/jmeter/protocol/http/parser/HTMLParser.java

Modified: jmeter/trunk/bin/jmeter.properties
URL: http://svn.apache.org/viewvc/jmeter/trunk/bin/jmeter.properties?rev=1529618&r1=1529617&r2=1529618&view=diff
==============================================================================
--- jmeter/trunk/bin/jmeter.properties (original)
+++ jmeter/trunk/bin/jmeter.properties Sun Oct  6 13:34:37 2013
@@ -575,14 +575,16 @@ HTTPResponse.parsers=htmlParser wmlParse
 
 # Define the HTML parser to be used.
 # Default parser:
-htmlParser.className=org.apache.jmeter.protocol.http.parser.LagartoBasedHtmlParser
+#htmlParser.className=org.apache.jmeter.protocol.http.parser.HtmlParserHTMLParser
 # Other parsers:
 #htmlParser.className=org.apache.jmeter.protocol.http.parser.JTidyHTMLParser
 # Note that Regexp extractor may detect references that have been commented out.
 # In many cases it will work OK, but you should be aware that it may generate 
 # additional references.
 #htmlParser.className=org.apache.jmeter.protocol.http.parser.RegexpHTMLParser
-#htmlParser.className=org.apache.jmeter.protocol.http.parser.HtmlParserHTMLParser
+# This new parser (since 2.10) should perform better than all others
+# see https://issues.apache.org/bugzilla/show_bug.cgi?id=55632
+#htmlParser.className=org.apache.jmeter.protocol.http.parser.LagartoBasedHtmlParser
 #
 
 htmlParser.types=text/html application/xhtml+xml application/xml text/xml

Modified: jmeter/trunk/src/protocol/http/org/apache/jmeter/protocol/http/parser/HTMLParser.java
URL: http://svn.apache.org/viewvc/jmeter/trunk/src/protocol/http/org/apache/jmeter/protocol/http/parser/HTMLParser.java?rev=1529618&r1=1529617&r2=1529618&view=diff
==============================================================================
--- jmeter/trunk/src/protocol/http/org/apache/jmeter/protocol/http/parser/HTMLParser.java (original)
+++ jmeter/trunk/src/protocol/http/org/apache/jmeter/protocol/http/parser/HTMLParser.java Sun Oct  6 13:34:37 2013
@@ -67,7 +67,7 @@ public abstract class HTMLParser {
     public static final String PARSER_CLASSNAME = "htmlParser.className"; // $NON-NLS-1$
 
     public static final String DEFAULT_PARSER =
-        "org.apache.jmeter.protocol.http.parser.LagartoBasedHtmlParser"; // $NON-NLS-1$
+        "org.apache.jmeter.protocol.http.parser.HtmlParserHTMLParser"; // $NON-NLS-1$
 
     /**
      * Protected constructor to prevent instantiation except from within