From 5ebdab7cedbcf0c7b99cad2a4b9c08c092e5ed03 Mon Sep 17 00:00:00 2001
From: "mrbkap%gmail.com"
 <mrbkap%gmail.com@18797224-902f-48f8-a5cc-f745e15eee43>
Date: Mon, 21 May 2007 18:09:54 +0000
Subject: [PATCH] Don't call DidBuildModel twice if Terminate is called twice
 since consumers aren't expecting it. bug 378982, r+sr=bzbarsky

git-svn-id: svn://10.0.0.236/trunk@226741 18797224-902f-48f8-a5cc-f745e15eee43
---
 mozilla/parser/htmlparser/src/nsParser.cpp | 6 ++++++
 1 file changed, 6 insertions(+)

diff --git a/mozilla/parser/htmlparser/src/nsParser.cpp b/mozilla/parser/htmlparser/src/nsParser.cpp
index 727b32a8f85..b160f2253b4 100644
--- a/mozilla/parser/htmlparser/src/nsParser.cpp
+++ b/mozilla/parser/htmlparser/src/nsParser.cpp
@@ -1038,6 +1038,12 @@ nsParser::GetRootContextKey()
 NS_IMETHODIMP
 nsParser::Terminate(void)
 {
+  // We should only call DidBuildModel once, so don't do anything if this is
+  // the second time that Terminate has been called.
+  if (mInternalState == NS_ERROR_HTMLPARSER_STOPPARSING) {
+    return NS_OK;
+  }
+
   nsresult result = NS_OK;
   // XXX - [ until we figure out a way to break parser-sink circularity ]
   // Hack - Hold a reference until we are completely done...