Bug 720124 - Remove dead code from HTMLContentSink: mNumOpenIFRAMES; r=hsivonen
authorMs2ger <ms2ger@gmail.com>
Wed, 25 Jan 2012 08:50:03 +0100
changeset 86424 2bb09ec057e6e16f3831117e474596ab964e2dea
parent 86423 90ce75a0083fe9594a04b055cb35caee10ed843b
child 86425 8232ce31f2219647a28bf2d897f4bcc239aa54db
push id805
push userakeybl@mozilla.com
push dateWed, 01 Feb 2012 18:17:35 +0000
treeherdermozilla-aurora@6fb3bf232436 [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
reviewershsivonen
bugs720124
milestone12.0a1
Bug 720124 - Remove dead code from HTMLContentSink: mNumOpenIFRAMES; r=hsivonen
content/html/document/src/nsHTMLContentSink.cpp
--- a/content/html/document/src/nsHTMLContentSink.cpp
+++ b/content/html/document/src/nsHTMLContentSink.cpp
@@ -244,17 +244,16 @@ protected:
   nsRefPtr<nsGenericHTMLElement> mBody;
   nsRefPtr<nsGenericHTMLElement> mHead;
 
   nsRefPtr<nsGenericHTMLElement> mCurrentForm;
 
   nsAutoTArray<SinkContext*, 8> mContextStack;
   SinkContext* mCurrentContext;
   SinkContext* mHeadContext;
-  PRInt32 mNumOpenIFRAMES;
 
   // depth of containment within <noembed>, <noframes> etc
   PRInt32 mInsideNoXXXTag;
 
   // Boolean indicating whether we've seen a <head> tag that might have had
   // attributes once already.
   bool mHaveSeenHead;
 
@@ -749,20 +748,16 @@ SinkContext::OpenContainer(const nsIPars
       MOZ_NOT_REACHED("Must not use HTMLContentSink for frames.");
       break;
 
     case eHTMLTag_noembed:
     case eHTMLTag_noframes:
       mSink->mInsideNoXXXTag++;
       break;
 
-    case eHTMLTag_iframe:
-      mSink->mNumOpenIFRAMES++;
-      break;
-
     case eHTMLTag_script:
     case eHTMLTag_style:
       MOZ_NOT_REACHED("Must not use HTMLContentSink for styles and scripts.");
       break;
 
     case eHTMLTag_button:
 #ifdef MOZ_MEDIA
     case eHTMLTag_audio:
@@ -885,20 +880,16 @@ SinkContext::CloseContainer(const nsHTML
       // of invalid form nesting. When the end FORM tag comes through,
       // we'll ignore it.
       if (aTag != nodeType) {
         result = CloseContainer(aTag);
       }
     }
 
     break;
-  case eHTMLTag_iframe:
-    mSink->mNumOpenIFRAMES--;
-
-    break;
 
 #ifdef MOZ_MEDIA
   case eHTMLTag_video:
   case eHTMLTag_audio:
 #endif
   case eHTMLTag_select:
   case eHTMLTag_textarea:
   case eHTMLTag_object: