Changeset 60801 in webkit

Timestamp:

Jun 7, 2010 2:40:03 PM (14 years ago)

Author:

eric@webkit.org

Message:

2010-06-07 Eric Seidel <​eric@webkit.org>

Reviewed by Adam Barth.

HTML5 parser hits ASSERT in http/tests/misc/window-dot-stop.html
​https://bugs.webkit.org/show_bug.cgi?id=40264

• html/HTML5Tokenizer.cpp: (WebCore::HTML5Tokenizer::executeScript):

Location:

trunk/WebCore

Files:

• ChangeLog (modified) (1 diff)
• html/HTML5Tokenizer.cpp (modified) (1 diff)

trunk/WebCore/ChangeLog

@@ +1 @@
+2010-06-07  Eric Seidel  <eric@webkit.org>
+
+        Reviewed by Adam Barth.
+
+        HTML5 parser hits ASSERT in http/tests/misc/window-dot-stop.html
+        https://bugs.webkit.org/show_bug.cgi?id=40264
+
+        * html/HTML5Tokenizer.cpp:
+        (WebCore::HTML5Tokenizer::executeScript):
+
 2010-06-07  Kinuko Yasuda  <kinuko@chromium.org>
 

trunk/WebCore/html/HTML5Tokenizer.cpp

@@ -165 +165 @@
     // oldInsertionPoint.prepent(m_source) as that would ASSERT if
     // m_source.escaped() (it had characters pushed back onto it).
-    m_source.append(oldInsertionPoint);
+    // If m_source was closed, then the tokenizer was stopped, and we discard
+    // any pending data as though an EOF character was inserted into the stream.
+    if (!m_source.isClosed())
+        m_source.append(oldInsertionPoint);
 }