From 2cca325f53af844823c836efe921323115447510 Mon Sep 17 00:00:00 2001 From: David Zbarsky Date: Sun, 21 Apr 2013 21:25:29 -0400 Subject: [PATCH] Remove TitleChangedListener dead code, no bug, r=jlebar over irc --- content/base/public/nsINode.h | 2 +- .../content/src/nsGenericHTMLFrameElement.h | 22 ------------------- 2 files changed, 1 insertion(+), 23 deletions(-) diff --git a/content/base/public/nsINode.h b/content/base/public/nsINode.h index 69c45fa11224..46d34b203c17 100644 --- a/content/base/public/nsINode.h +++ b/content/base/public/nsINode.h @@ -276,7 +276,7 @@ public: // measurement of the following members may be added later if DMD finds it is // worthwhile: // - nsGenericHTMLElement: mForm, mFieldSet - // - nsGenericHTMLFrameElement: mFrameLoader (bug 672539), mTitleChangedListener + // - nsGenericHTMLFrameElement: mFrameLoader (bug 672539) // - HTMLBodyElement: mContentStyleRule // - HTMLDataListElement: mOptions // - HTMLFieldSetElement: mElements, mDependentElements, mFirstLegend diff --git a/content/html/content/src/nsGenericHTMLFrameElement.h b/content/html/content/src/nsGenericHTMLFrameElement.h index 0224d692c7c5..40b6e2d488cf 100644 --- a/content/html/content/src/nsGenericHTMLFrameElement.h +++ b/content/html/content/src/nsGenericHTMLFrameElement.h @@ -71,28 +71,6 @@ public: void SwapFrameLoaders(nsXULElement& aOtherOwner, mozilla::ErrorResult& aError); protected: - /** - * Listens to titlechanged events from the document inside the iframe and - * forwards them along to the iframe so it can fire a mozbrowsertitlechange - * event if appropriate. - */ - class TitleChangedListener MOZ_FINAL : public nsIDOMEventListener - { - public: - TitleChangedListener(nsGenericHTMLFrameElement *aElement, - mozilla::dom::EventTarget *aChromeHandler); - - /* Unregister this listener. */ - void Unregister(); - - NS_DECL_ISUPPORTS - NS_DECL_NSIDOMEVENTLISTENER - - private: - nsWeakPtr mElement; /* nsGenericHTMLFrameElement */ - nsWeakPtr mChromeHandler; /* EventTarget */ - }; - // This doesn't really ensure a frame loade in all cases, only when // it makes sense. void EnsureFrameLoader();