RESOLVED FIXED 22189
Track CachedScript decoded data 
https://bugs.webkit.org/show_bug.cgi?id=22189
Summary Track CachedScript decoded data
Sam Weinig
Reported 2008-11-11 15:46:33 PST
Patch forthcoming.
Attachments
patch (3.37 KB, patch)
2008-11-11 15:53 PST, Sam Weinig 		mitz: review+
DetailsFormatted DiffDiff
View All Add attachment proposed patch, testcase, etc.
Sam Weinig
Comment 1 2008-11-11 15:53:48 PST
Created attachment 25078 [details] patch
mitz
Comment 2 2008-11-11 15:58:44 PST
Comment on attachment 25078 [details] patch > Index: ChangeLog > =================================================================== > --- ChangeLog (revision 38320) > +++ ChangeLog (working copy) > @@ -1,3 +1,19 @@ > +2008-11-11 Sam Weinig <sam@webkit.org> > + > + Reviewed by NOBODY (OOPS!). > + > + Fix for https://bugs.webkit.org/show_bug.cgi?id=22189 > + Track CachedScript decoded data > + > + * loader/CachedResource.h: > + (WebCore::CachedResource::allClientsRemoved): > + (WebCore::CachedResource::destroyDecodedData): > + * loader/CachedScript.cpp: > + (WebCore::CachedScript::script): > + (WebCore::CachedScript::data): > + (WebCore::CachedScript::destroyDecodedData): > + * loader/CachedScript.h: > + You could explain the code changes. > + if (m_script.isNull() && m_data) { You can write this as (!m_script && m_data). r=me
Sam Weinig
Comment 3 2008-11-11 16:37:37 PST
Fixed in r38321.
Note You need to log in before you can comment on or make changes to this bug.