svn commit: r1179261 - /incubator/wookie/trunk/src/org/apache/wookie/util/html/HtmlCleaner.java

[psharples]

Author: psharples
Date: Wed Oct  5 15:27:01 2011
New Revision: 1179261

URL: http://svn.apache.org/viewvc?rev=1179261&view=rev
Log:
The missing doctype problem had returned. See WOOKIE-236. It would appear that 
we have to set the omitdoctypedeclaration(false) call. This now appears to 
remedy the problem.

Modified:
    incubator/wookie/trunk/src/org/apache/wookie/util/html/HtmlCleaner.java

Modified: 
incubator/wookie/trunk/src/org/apache/wookie/util/html/HtmlCleaner.java
URL: 
http://svn.apache.org/viewvc/incubator/wookie/trunk/src/org/apache/wookie/util/html/HtmlCleaner.java?rev=1179261&r1=1179260&r2=1179261&view=diff
==============================================================================
--- incubator/wookie/trunk/src/org/apache/wookie/util/html/HtmlCleaner.java 
(original)
+++ incubator/wookie/trunk/src/org/apache/wookie/util/html/HtmlCleaner.java Wed 
Oct  5 15:27:01 2011
@@ -48,11 +48,11 @@ public class HtmlCleaner implements IHtm
                // set cleaner properties       
                properties  = cleaner.getProperties();
                //
-               // The following setting has been removed as it can cause 
issues with IE going
-               // into "quirks mode" - see WOOKIE-236
-               //
-               // properties.setOmitDoctypeDeclaration(true);
+               // The following setting has been re-inserted. Set to true as 
it can cause issues with IE going
+               // into "quirks mode" - see WOOKIE-236. 
+               // However, it appears we explicitly have to declare it as 
false to work correctly.
                //
+               properties.setOmitDoctypeDeclaration(false);
                properties.setOmitXmlDeclaration(true);
                properties.setUseCdataForScriptAndStyle(true);
                properties.setUseEmptyElementTags(false);