Bug 1462272: Remove an unneeded and ugly reinterpret_cast. r?dholbert
MozReview-Commit-ID: Evip43nwNii
--- a/layout/generic/nsImageFrame.cpp
+++ b/layout/generic/nsImageFrame.cpp
@@ -209,17 +209,17 @@ nsImageFrame::DestroyFrom(nsIFrame* aDes
if (imageLoader) {
// Notify our image loading content that we are going away so it can
// deregister with our refresh driver.
imageLoader->FrameDestroyed(this);
imageLoader->RemoveNativeObserver(mListener);
}
- reinterpret_cast<nsImageListener*>(mListener.get())->SetFrame(nullptr);
+ mListener->SetFrame(nullptr);
}
mListener = nullptr;
// If we were displaying an icon, take ourselves off the list
if (mDisplayingIcon)
gIconLoad->RemoveIconObserver(this);
--- a/layout/generic/nsImageFrame.h
+++ b/layout/generic/nsImageFrame.h
@@ -324,17 +324,17 @@ private:
* @param aFrameInvalidRect The area to invalidate in frame space. If null, the
* entire frame will be invalidated.
*/
void InvalidateSelf(const nsIntRect* aLayerInvalidRect,
const nsRect* aFrameInvalidRect);
RefPtr<nsImageMap> mImageMap;
- nsCOMPtr<imgINotificationObserver> mListener;
+ RefPtr<nsImageListener> mListener;
nsCOMPtr<imgIContainer> mImage;
nsCOMPtr<imgIContainer> mPrevImage;
nsSize mComputedSize;
mozilla::IntrinsicSize mIntrinsicSize;
nsSize mIntrinsicRatio;
bool mDisplayingIcon;