From c0b86fcbd8355d9498d6d34a90a9ea4cf35dbbd9 Mon Sep 17 00:00:00 2001
From: "mrbkap%gmail.com"
 <mrbkap%gmail.com@18797224-902f-48f8-a5cc-f745e15eee43>
Date: Sun, 22 May 2005 00:57:37 +0000
Subject: [PATCH] bug 294984: Make sure to create text and end tokens for tags
 such as <iframe>. This fixes a crash where CNavDTD assumed an end token was
 coming, but it wasn't. r+sr=bzbarsky a=chofmann

git-svn-id: svn://10.0.0.236/trunk@173675 18797224-902f-48f8-a5cc-f745e15eee43
---
 .../parser/htmlparser/src/nsHTMLTokenizer.cpp  | 18 +++++++++---------
 1 file changed, 9 insertions(+), 9 deletions(-)

diff --git a/mozilla/parser/htmlparser/src/nsHTMLTokenizer.cpp b/mozilla/parser/htmlparser/src/nsHTMLTokenizer.cpp
index 32fb8752ab3..3732b689385 100644
--- a/mozilla/parser/htmlparser/src/nsHTMLTokenizer.cpp
+++ b/mozilla/parser/htmlparser/src/nsHTMLTokenizer.cpp
@@ -793,17 +793,17 @@ nsresult nsHTMLTokenizer::ConsumeStartTag(PRUnichar aChar,
       if (NS_FAILED(result)) {
         aToken->SetInError(PR_TRUE);
 
-        // Note: We know here that the scanner is not incremental since if
-        // this peek fails, then we've already masked over a kEOF coming from
-        // the Consume() call above.
-        return NS_OK;
-      }
-
-      if(kGreaterThan != aChar) { // Look for a '>'
-        result = ConsumeAttributes(aChar, aToken, aScanner);
+        // Don't return early here so we can create a text and end token for
+        // the special <iframe>, <script> and similar tags down below.
+        result = NS_OK;
       }
       else {
-        aScanner.GetChar(aChar);
+        if(kGreaterThan != aChar) { // Look for a '>'
+          result = ConsumeAttributes(aChar, aToken, aScanner);
+        }
+        else {
+          aScanner.GetChar(aChar);
+        }
       }
 
       /*  Now that that's over with, we have one more problem to solve.