Handle CDATA unintentionally treated as a conditional comment

Fixes #1161

Author: wataash

src/htmlparser.js

@@ -140,6 +140,15 @@ function HTMLParser(html, handler) {
           }
         }
 
+        // <![CDATA[...]]> as a html node is incorrect, but just leave it for backwords compatibility (#TODO)
+        if (html.startsWith('<![CDATA[')) {
+          var cdataEnd = html.indexOf(']]>');
+          handler.comment(html.substring(2, cdataEnd + 2), true /* non-standard */);
+          html = html.substring(cdataEnd + 3);
+          prevTag = '';
+          continue;
+        }
+
         // https://en.wikipedia.org/wiki/Conditional_comment#Downlevel-revealed_conditional_comment
         if (/^<!\[/.test(html)) {
           var conditionalEnd = html.indexOf(']>');

tests/minifier.js

@@ -623,7 +623,7 @@ QUnit.test('collapsing space in conditional comments', function(assert) {
   }), output);
 });
 
-QUnit.test('(bug) CDATA parsed as conditional comments', function(assert) {
+QUnit.test('treat CDATA like a comments for backward compatibility', function(assert) {
   var input;
 
   input = '<![CDATA[line 1\nline 2]]>';
@@ -633,21 +633,13 @@ QUnit.test('(bug) CDATA parsed as conditional comments', function(assert) {
   assert.equal(minify(input), input);
   assert.equal(minify(input, { removeComments: true }), '<p></p>');
 
-  // https://github.com/kangax/html-minifier/issues/1161
+  // // https://github.com/kangax/html-minifier/issues/1161
   input = '<![CDATA[___]><-___]]>';
-  assert.throws(function() {
-    minify(input);
-  }, '"]>" treated as end instead of "]]>" (bug)');
-  assert.throws(function() {
-    minify(input, { removeComments: true });
-  }, '"]>" treated as end instead of "]]>" (bug)');
+  assert.equal(minify(input), input);
+  assert.equal(minify(input, { removeComments: true }), '');
   input = '<p><![CDATA[___]><-___]]></p>';
-  assert.throws(function() {
-    minify(input);
-  }, '"]>" treated as end instead of "]]>" (bug)');
-  assert.throws(function() {
-    minify(input, { removeComments: true });
-  }, '"]>" treated as end instead of "]]>" (bug)');
+  assert.equal(minify(input), input);
+  assert.equal(minify(input, { removeComments: true }), '<p></p>');
 });
 
 QUnit.test('remove comments from scripts', function(assert) {