Backing out Bug 572290 due to leaks
authorKyle Huey <me@kylehuey.com>
Tue, 10 Aug 2010 21:11:15 -0700
changeset 49370 b89bd32f1245896103b93414f4841e3820fcb679
parent 49368 b1be7d4acee211e547389d0d0151c0ebfd774027 (current diff)
parent 49369 8c2da01383308cb39567e9ae7c85e6935cc55655 (diff)
child 49372 a675f24fbeb2605cd1c7fce53254c43e1537ca41
push id1
push userroot
push dateTue, 26 Apr 2011 22:38:44 +0000
treeherdermozilla-beta@bfdb6e623a36 [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
bugs572290
milestone2.0b4pre
first release with
nightly linux32
nightly linux64
nightly mac
nightly win32
nightly win64
last release without
nightly linux32
nightly linux64
nightly mac
nightly win32
nightly win64
Backing out Bug 572290 due to leaks
--- a/content/html/document/src/nsHTMLFragmentContentSink.cpp
+++ b/content/html/document/src/nsHTMLFragmentContentSink.cpp
@@ -1228,19 +1228,16 @@ nsHTMLParanoidFragmentSink::SanitizeStyl
 
 NS_IMETHODIMP
 nsHTMLParanoidFragmentSink::AddLeaf(const nsIParserNode& aNode)
 {
   NS_ENSURE_TRUE(mNodeInfoManager, NS_ERROR_NOT_INITIALIZED);
   
   nsresult rv = NS_OK;
 
-  // We need to explicitly skip adding leaf nodes in the paranoid sink,
-  // otherwise things like the textnode under <title> get appended to
-  // the fragment itself, and won't be popped off in CloseContainer.
   if (mSkip || mIgnoreNextCloseHead) {
     return rv;
   }
   
   if (aNode.GetTokenType() == eToken_start) {
     nsCOMPtr<nsIAtom> name;
     rv = NameFromNode(aNode, getter_AddRefs(name));
     NS_ENSURE_SUCCESS(rv, rv);