Bommels05/Mozilla
Bommels05/Mozilla
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

#=29862

Browse Source 
r=vidur
a=jar
d=Fixes the memory leak problem with JavaScript preloading animated gifs.


git-svn-id: svn://10.0.0.236/trunk@62413 18797224-902f-48f8-a5cc-f745e15eee43
This commit is contained in:
waqar%netscape.com 2000-03-09 00:20:55 +00:00
parent aca6b0bba2
commit 1d5c814149
1 changed files with 17 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

17
mozilla/layout/base/src/nsFrameImageLoader.cpp
View File

@ -278,6 +278,17 @@ nsFrameImageLoader::IsSameImageRequest(const nsString& aURL,
return NS_OK;
}
// XXX Temporary fix to deal with the case of animated
// images that are preloaded. If we've completed the image
// and there aren't any frames associated with it, then we
// interrupted the load of the image. The good things is that
// the image is still in the image cache. The bad thing is that
// we don't share the image loader.
if ((nsnull == mFrames) && (mImageLoadStatus & NS_IMAGE_LOAD_STATUS_IMAGE_READY)) {
*aResult = PR_FALSE;
return NS_OK;
}
// Background colors must match
if (aBackgroundColor) {
if (!mHaveBackgroundColor) {
@ -420,6 +431,12 @@ nsFrameImageLoader::Notify(nsIImageRequest *aImageRequest,
break;
case nsImageNotification_kImageComplete:
// XXX Temporary fix to deal with the case of animated
// images that are preloaded. If there are no frames
// registered, then stop loading this image.
if (nsnull == mFrames) {
AbortImageLoad();
}
if ((nsnull == mImage) && (nsnull != aImage)) {
mImage = aImage;
NS_ADDREF(aImage);