Changeset 139217 in webkit

Timestamp:

Jan 9, 2013 12:01:46 PM (11 years ago)

Author:

tonyg@chromium.org

Message:

REGRESSION(r139141): Assertion failure in WebCore::HTMLConstructionSite::HTMLConstructionSite
​https://bugs.webkit.org/show_bug.cgi?id=106412

Reviewed by Darin Adler.

This corrects the assertion added in r139141. The assertion was designed to make sure that we are using the
HTMLDocument class. XHTML documents also use HTMLDocument.

No new tests because no new functionality.

• html/parser/HTMLConstructionSite.cpp:

(WebCore::HTMLConstructionSite::HTMLConstructionSite):

Location:

trunk/Source/WebCore

Files:

• ChangeLog (modified) (1 diff)
• html/parser/HTMLConstructionSite.cpp (modified) (2 diffs)

trunk/Source/WebCore/ChangeLog

@@ +1 @@
+2013-01-09  Tony Gentilcore  <tonyg@chromium.org>
+
+        REGRESSION(r139141): Assertion failure in WebCore::HTMLConstructionSite::HTMLConstructionSite
+        https://bugs.webkit.org/show_bug.cgi?id=106412
+
+        Reviewed by Darin Adler.
+
+        This corrects the assertion added in r139141. The assertion was designed to make sure that we are using the
+        HTMLDocument class. XHTML documents also use HTMLDocument.
+
+        No new tests because no new functionality.
+
+        * html/parser/HTMLConstructionSite.cpp:
+        (WebCore::HTMLConstructionSite::HTMLConstructionSite):
+
 2013-01-08  Ojan Vafai  <ojan@chromium.org>
 

trunk/Source/WebCore/html/parser/HTMLConstructionSite.cpp

@@ -147 +147 @@
     , m_inQuirksMode(document->inQuirksMode())
 {
-    ASSERT(m_document->isHTMLDocument());
+    ASSERT(m_document->isHTMLDocument() || m_document->isXHTMLDocument());
 }
 
@@ -159 +159 @@
     , m_inQuirksMode(fragment->document()->inQuirksMode())
 {
-    ASSERT(m_document->isHTMLDocument());
+    ASSERT(m_document->isHTMLDocument() || m_document->isXHTMLDocument());
 }