DOMParser.parseFromString() should support creating HTML Document with mimetype text/html

Source/core/xml/DOMParser.cpp

 /*
  *  Copyright (C) 2003, 2006, 2008 Apple Inc. All rights reserved.
  *
  *  This library is free software; you can redistribute it and/or
  *  modify it under the terms of the GNU Lesser General Public
  *  License as published by the Free Software Foundation; either
  *  version 2 of the License, or (at your option) any later version.
  *
  *  This library is distributed in the hope that it will be useful,
  *  but WITHOUT ANY WARRANTY; without even the implied warranty of
  *  MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the GNU
  *  Lesser General Public License for more details.
  *
  *  You should have received a copy of the GNU Lesser General Public
  *  License along with this library; if not, write to the Free Software
  *  Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA  02110-1301  USA
  */
 
 #include "config.h"
 #include "core/xml/DOMParser.h"
 
 #include "core/dom/DOMImplementation.h"
 #include "core/dom/Document.h"
 #include "wtf/text/WTFString.h"
 
 namespace WebCore {
 
 PassRefPtr<Document> DOMParser::parseFromString(const String& str, const String& contentType)
 {
-    if (!DOMImplementation::isXMLMIMEType(contentType))
+    if (!(DOMImplementation::isXMLMIMEType(contentType) || contentType == "text/html"))

[abarth-chromium, 2013/09/05 17:25:31]

Can you distribute the ! inside the || ?  I think that will make this line of
code read better:

if (!DOMImplementation::isXMLMIMEType(contentType) && contentType !=
"text/html")
..

[eseidel, 2013/09/05 17:26:28]

I'm concerned that we support more mime types than the spec allows us to.

I would like us to see a test of what mimetypes we support and what happens when
we don't support them. It's OK for such a test to document (with FAIL) our
current spec-divergent behavior, but I'd like to see it tested.

         return 0;
 
     RefPtr<Document> doc = DOMImplementation::createDocument(contentType, 0, KURL(), false);
     doc->setContent(str);
     return doc.release();
 }
 
 } // namespace WebCore